Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Surge XT - Filter Cutoff hit / hover region partly overlaps with F2 Offset and Link buttons #4855

Closed
VincyZed opened this issue Aug 17, 2021 · 5 comments
Labels
Bug Report Item submitted using the Bug Report template Rebuild With JUCE Issues pertaining to porting Surge from VSTGUI to JUCE UI Issues related to UI look&feel
Milestone

Comments

@VincyZed
Copy link
Collaborator

The hit / hover region for both filter cutoff sliders stretches too far to the right. It takes approximatively half of the freq Offset button on F2, and same for resonance link. When hovering to the left of those buttons and clicking, it's selecting the cutoff silder, so it's not just related to the visual hover.
This may be the case for all horizontal sliders but I'm not sure.

Hover Bug

@VincyZed VincyZed added UI Issues related to UI look&feel Bug Report Item submitted using the Bug Report template Rebuild With JUCE Issues pertaining to porting Surge from VSTGUI to JUCE labels Aug 17, 2021
@VincyZed VincyZed added this to the Surge XT 1.0 milestone Aug 17, 2021
@mkruselj
Copy link
Collaborator

It's exactly the case for all horizontal sliders. The solution would be placing those two buttons in a higher Z layer so they are on top of the slider. How? I don't know, but JUCE probably has a way.

@baconpaul
Copy link
Collaborator

oh man that's gonna need a special case. the sliders actually overlap with that button because they are bigger (so you can get the handle all the way over).

@mkruselj is right. We need to zorder them up. they layout in skin order now basically.

@VincyZed
Copy link
Collaborator Author

VincyZed commented Aug 17, 2021

Right, just tested and the range of slider grabbing hasn't really changed that much since 1.9, and is actually good as it is now imo.

@VincyZed VincyZed changed the title Surge XT - Filter Cutoff hit / hover region range stretches too far to the right Surge XT - Filter Cutoff hit / hover region partly overlaps with F2 Offset and Link buttons Aug 17, 2021
@baconpaul
Copy link
Collaborator

hmm I stuck this at the end of openOrRecreateEditor and it didn't fix it

The problem is these widgets actually are overlapping. I kinda wish they weren't

Ponder ponder

    /*
     * OK this is a gross hack but it's OK. The sliders are a bit 'long' so they hover over
     * the switches. Just fix this by brining all the switches to the front. See #4855
     */
    for (auto p : nonmod_param)
    {
        auto s = dynamic_cast<Surge::Widgets::Switch *>(p);
        if (s)
        {
            std::cout << "Got an S" << std::endl;
            s->toFront(false);
        }
    }

@VincyZed
Copy link
Collaborator Author

From what I can see, this one has been fixed.
Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Report Item submitted using the Bug Report template Rebuild With JUCE Issues pertaining to porting Surge from VSTGUI to JUCE UI Issues related to UI look&feel
Projects
None yet
Development

No branches or pull requests

3 participants