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

Turn theme into Preference Pack #3

Merged
merged 6 commits into from
Jul 14, 2022

Conversation

Chrismettal
Copy link
Contributor

See related issue: #2

Refactors the repo into easily installable PreferencePack, supported by FreeCAD version 0.20 and up.
Allows easier installation without overwriting/merging the user.cfg manually. After installing the PreferencePack to your FreeCAD installation, theme can be applied with one click in settings.

Installation instructions were also updated accordingly.

I will open up a PR to the addon manager repo, adding this PreferencePack there, so it can be installed from within FreeCAD, without manually downloading/installing anything. This is already referenced in the installation manual.

I have taken the liberty to create an SVG Icon showing representative colors for the addon manager, similar to existing PreferencePacks to help users to get a glance of themes when scrolling through the addon manager.

Additionaly, the "Fully constrained" color in the sketcher was still set to default, and has been set to the green defined in the Dracula palette.

@luzpaz
Copy link

luzpaz commented Jul 10, 2022

Awesome work here. Thanks @Chrismettal !

@luzpaz
Copy link

luzpaz commented Jul 11, 2022

testers, please report issues? We're waiting upstream for this PR to be merged

@luzpaz
Copy link

luzpaz commented Jul 13, 2022

Bumping

@eleanor-clifford
Copy link
Member

Great work @Chrismettal, thanks for the PR! Just one thing, can you leave the original instructions in INSTALL.md as well as the new ones? (perhaps under a subheading like "FreeCAD <0.20") Since it looks like not all distros have 0.20 yet.

@eleanor-clifford
Copy link
Member

Also if you'd like to @Chrismettal, feel free to add yourself to the "Team" section in README.md

@Chrismettal
Copy link
Contributor Author

[...] can you leave the original instructions in INSTALL.md as well as the new ones?

Good call, didn't think about that!

[...] feel free to add yourself to the "Team" section [...]

Gladly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants