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

[GUI] Fix minor issues with Workbenches preference panel #4372

Merged
merged 1 commit into from Feb 5, 2021

Conversation

chennes
Copy link
Member

@chennes chennes commented Feb 5, 2021

This fixes three issues with the new Workbench Loader preferences panel reported by users:

  • Some icons were not being properly scaled to the new 48x48 size
  • Some text was being incorrectly cut off, despite fitting in the column
  • The sort order of the workbenches was incorrect when external workbenches were added

  • Branch rebased on latest master git pull --rebase upstream master
  • Unit tests confirmed to pass by running ./bin/FreeCAD --run-test 0
  • Commit message is well-written
  • No tracker tickets filed

This fixes three issues with the new Workbench Loader preferences panel reported by users:
* Some icons were not being properly scaled to the new 48x48 size
* Some text was being incorrectly cut off, despite fitting in the column
* The sort order of the workbenches was incorrect when external workbenches were added
chennes referenced this pull request Feb 5, 2021
A new group is added to the Preferences window that explains why
some preferences may appear to be missing due to the workbench being
unloaded. It lists the unloaded workbenches, and offers to load them,
updating the preferences dialog as necessary.

This at least partially resolves issue #4474.
@wwmayer wwmayer merged commit 7560a35 into FreeCAD:master Feb 5, 2021
@chennes chennes deleted the fixWorkbenchBugsA branch February 9, 2021 15:11
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.

None yet

2 participants