-
Notifications
You must be signed in to change notification settings - Fork 192
Description
Hello! Thanks for starting and contributing to such an amazing project!
As a new user, I have an slight improvement idea on the UI design:
- add text "Noise Gate" for the switch at the bottom, so that all switches uniformly have text labels.
- replace text for the knob from "Gate" to "Threshold", so it indicates what the knob does (though the unit makes it pretty simple to infer)
The logic behind this change is to have better unity and clarity for the UI:
- all switches now have labels (unity); and the function of the knob is more clear
- relationship between texts above and below the separator line is more uniform: "Bass", "Middle", and "Treble" are parameters of "EQ", and now "Threshold" is a parameter for "Noise Gate".
Because the modification is so slight, I have updated it and it looks like this:
Also, I totally get why there's no text for the switch: the warning message! So, I've got another change: to move the warning message to the bottom near the model area and move the IR area a bit lower to leave space.
This actually makes the error message closer to where the error comes from, also making the hint slightly more intuitive; like this:
Since the modification is so little, I have already implemented all these changes on my local branch, so if this idea gets approved, I can push my branch and setup PR right away.
Thanks for your time and consideration!

