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

Clicking again the "Presets & Settings" button in an effect dialog won't close the dropdown menu (Windows) #6470

Open
dozzzzer opened this issue May 22, 2024 · 3 comments
Labels
bug An error, undesired behaviour, or missed functionality Effects (EGATs) Effects, Generators, Analyzers and Tools P4 Low priority Windows The issues affecting exclusively Windows

Comments

@dozzzzer
Copy link
Contributor

dozzzzer commented May 22, 2024

Steps to reproduce

  1. Open up any effect's dialog either from the realtime effects panel or from the main Effect/Generate menu
  2. Click "Presets & Settings"
  3. Observe: the dropdown menu expands
  4. Click "Presets & Settings" again

Expected behavior

The dropdown menu gets closed

Actual behavior

The dropdown menu remains open

Audacity Version

current master / alpha / nightly build

Operating system

Windows 10 & 11

Additional context

This goes all the way back to at least 2.4

@dozzzzer dozzzzer added bug An error, undesired behaviour, or missed functionality Windows The issues affecting exclusively Windows Effects (EGATs) Effects, Generators, Analyzers and Tools labels May 22, 2024
@dozzzzer dozzzzer changed the title Clicking again the "Presets & Settings" button in an effect dialog won't close the dropdown menu Clicking again the "Presets & Settings" button in an effect dialog won't close the dropdown menu (Windows) May 22, 2024
@petersampsonaudacity
Copy link

petersampsonaudacity commented May 22, 2024

@dozzzzer I do not believe that this is an issue - for me it works as expected ...

In fact it dates back to 2.1.1 when presets for effects were first introduced - but notice that back then (and indeed in 2.4.2 even) the presets were invoked with a Manage button rather than a Presets & Settings button:
image

And note that a second click on the Manage button never did close the dropdown presets menu.


However, having said that as a user I do not really expect a second press on Manage or Presets&Settings to close the menu.

The Presets dropdown menu can readily be dismissed by clicking anywhere else in the Audacity window - and even by clicking outside the Audacity window.

So for me the UI is working properly in this regard and I really don't think this is a bug (or design flaw) to be corrected.

@dozzzzer
Copy link
Contributor Author

I really don't think this is a bug (or design flaw) to be corrected

It definitely should be corrected one way or another, simply for the sake of consistency across platforms. Currently, both on macOS and Linux, clicking again the Presets & Settings does close the menu.

@LWinterberg LWinterberg added the P4 Low priority label May 22, 2024
@LWinterberg
Copy link
Member

Note that we don't need consistency across platforms, but consistency within platforms. This means stuff like #5963 is something we should do to get in line with the host OS (mac: Cancel/OK, windows: OK/Cancel), rather than getting the buttons to be in the same order across all OSes.

As for consistency within Audacity on Windows:

  • Clicking again on true dropdowns (like the one in the time signature) closes them
  • Clicking again on the fake dropdown we use for the snapping thing closes it.
  • Clicking again on the NumericTextControl... is actually impossible as the options overlap the control.
  • Clicking again on the Audio Setup button reopens it.
  • Clicking again on the timeline gear reopens it.
  • Clicking again on the Preferenes > Shortcuts > Defaults button reopens it (though the context menu tends to overlay).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error, undesired behaviour, or missed functionality Effects (EGATs) Effects, Generators, Analyzers and Tools P4 Low priority Windows The issues affecting exclusively Windows
Projects
None yet
Development

No branches or pull requests

3 participants