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

Local Adjustments GUI improvements #5743

Merged
merged 6 commits into from
May 8, 2020

Conversation

Pandagrapher
Copy link
Collaborator

@Pandagrapher Pandagrapher commented May 8, 2020

@Desmis

Major Changes:

  • Tool complexity can now be changed dynamically without restarting RT;
  • Tool complexity can now be managed independently for each tool (a default choice when adding new tool can be chosen in Preferences panel);
  • Tool advice tooltips can now be shown/hidden dynamically without restarting RT (option in Preferences panel).

Other:

  • Some GUI code cleanups

Pierre

Pandagrapher and others added 6 commits May 3, 2020 13:48
Changes:
- For each Locallab tool, a complexity mode can now be chosen (by
defaut, "Expert") to reduce or not the number of tool parameters.

Fixed issues:
- Some tool parameters had their values forced to a default one in
"Normal" mode whereas it shouldn't.
Others:
- Improves "Local Contrast" tool GUI behavior
- Fixes label for default complexity mode choice in preferences windows
- Modified some Locallab tool default parameters
- Updates some labels
- Expanders visibility state is now kept when using history
- New added tool has now its visibily state resetted to default one
- Code cleanup
@Desmis
Copy link
Collaborator

Desmis commented May 8, 2020

@Pandagrapher

Ok, I have seen your "new" work...
Now you have solved the squaring of the circle. How to develop both a simple interface so as not to make the user flee, and preserve complexity.

Each user can choose the tools he wants, with the level of complexity he wants....

Jacques

@Desmis Desmis merged commit 2cfa6d6 into Beep6581:newlocallab May 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants