[FEATURE] UI Improvement Idea

[HuoDx]

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!

[sdatkinson]

Thanks for the suggestions.

The warning looks a bit squished down there. How about putting it up top below the "NEURAL AMP MODELER"?

If that looks good (I appreciate the demo pictures!) then yes, a PR would probably go quickly 👍🏻

[HuoDx]

Great suggestion! How about this:

[sdatkinson]

Works for me!

[HuoDx]

@sdatkinson I've posted the PR #387; Thanks!

[olilarkin]

I think that the "Thesh"/Gate label change is good! but what about a small red "i" in a circle in the bottom right hand corner for the samplerate mismatch? This could also be a button that gives you a little popup with info about the .nam file (.e.g training sample rate etc, model type) - could be white when there is no samplerate mismatch

[sdatkinson]

I'm resolving this since the PR is merged.

As far as sample rate etc, I'm thinking to put this all under some "advanced" page with a gear, an ellipsis, or some other icon in the corner. (#460).