-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Manage 3D symbols through style manager #37952
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@nyalldawson , fantastic work! What about displaying some basic information - such as colors - with the preview icon. We could still use the 3D symbol, but tweak it to reflect part of the saved symbol. |
@nirvn it's an option I guess - still not a trivial inclusion, so I'd rather explore taking a snapshot of the real symbol first |
3efeade
to
207ebe8
Compare
822defb
to
3373b56
Compare
Nice! |
…try is populated Otherwise it's impossible to load these symbols
Because implementing proper thumbnails is a considerable task in itself...
(still not implement -- but a necessary prerequisite to proper icons)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
3D
Relates to QGIS' 3D engine or rendering
ChangelogHarvested
This PR description has been harvested in the Changelog already.
Feature
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds support for managing 3d symbols through the QGIS style manager, including support for importing and exporting libraries of 3d symbols, and completes the recent work in fully abstracting out 3d symbols and their widget handlers.
One notable omission is that there's no thumbnails generated for 3d symbols in the library, and everything gets a generic icon for now. Generating thumbnails is no trivial matter, so it's deffered for now...
Fixes #31479