-
-
Notifications
You must be signed in to change notification settings - Fork 261
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
Clarify enabled effects and presets #3142
Comments
Just to avoid any misunderstanding EasyEffects does not operate around the idea of "enabled presets". Imagine that you've loaded a preset and manually changed a parameter after this. EasyEffects database will remember the current value and use it from this point on. It does not matter that a preset was loaded before. Presets are just a convenient way to load a given set of values. Our database does not work based on the preset file. When I added the preset name in the menu the idea was more along the lines of "last used preset". Nothing more than that.
I never wanted to highlight the preset row for the same reasons I talked about above. EasyEffects does not keep track of loaded presets because they are just a fast way to load values. Our database does not use them for storage purposes. The preset selected there may not represent the current values depending on what the user is doing.
The original intention was having this icon representing the flow of audio buffers in the pipeline. Disabling the plugin through that toggle button puts it in passthrough mode. So audio is still flowing through it. It just does not modify the data. It is not a big deal to me to try to put the icon in passive mode as long as people do not also think that the plugin is totally removed from the pipeline. |
Please, if you can, try out the master branch, the preset menu is changed and now we show the "last loaded preset". As @wwmm said, effects are not enabled/disabled, but they are dimmed when bypassed. |
Okay, the reasons for presets make sense.
Okay, that is important background, but I think we should allow user keep set of effects on the side panel regardless are them disabled or not. And even without that, the status need to be more clear. I keep EQ always open, but I wan't to know is it on or not. Now the indicator is too unnoticeable. Could we make a dedicated indicator for the statuses of effects? |
If you look the indicator, you are looking at the wrong thing. You should look at the effect name. I can see in the screenshot that only the first Equalizer is not bypassed. Then, yes, maybe we can make also the arrow with dimmed style in bypass mode, but I don't think it's a good idea. When we added the player/device rows (they were missing initially) we wanted to give an hint of the "flow", but I made their icon dimmed because those rows are not interactive (if you click them, nothing happens). If you dim also the arrow, it seems not interactive too, which is not the case since it's an effect. |
For displaying the bypass/active state of the effects, I think that instead of dimming bits of the UI, we should simply have a different icon in place of the flow arrow that represents the bypass state. Perhaps simply an arrow with a dashed tail, instead of solid. This way the communication of "flow" still happens, but it includes the bypass state as well. To me, this seems much better than relying on the relatively low-contrast dimming of text. |
Assuming we find a good icon it will probably be the most reasonable thing to do. But I would still dim the label contrast. |
Exactly! I just want a more noticeable indicator; was it contrast, text or an icon doesn't matter.
I think that's also a good idea. |
@korpe1 I don't think there's an icon like that. Please suggest one that has the symbolic style (changes the color in light/dark theme). |
Maybe it's just better to change the style. The red |
What do you think? I picked these icons: CC0 license, we can use them. |
Could we make the UI more clear about which effects and presets are enabled? I suggest to use checkbox or slide switch elements to enable things.
And same problems in the preset selector where a small text is the only indicator:
The text was updated successfully, but these errors were encountered: