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

allow to export favorite symbols from style manager (fix #27315) #36793

Merged
merged 2 commits into from
May 28, 2020
Merged

allow to export favorite symbols from style manager (fix #27315) #36793

merged 2 commits into from
May 28, 2020

Conversation

alexbruy
Copy link
Contributor

@alexbruy alexbruy commented May 28, 2020

Description

When exporting symbols from Style manager one can select symbols with mouse or using "Select by group" button. The latter allows to select symbols which belong to some smart group or tag but it is not possible to select favorite symbols.

Proposed PR adds "Favorites" group to the "Select by group" dialog to allow selecting favorite symbols and make it consistent with group selector in the Style manager itself.

Fixes #27315.

Not sure about backport to 3.10.

@alexbruy alexbruy added GUI/UX Related to QGIS application GUI or User Experience Bug Either a bug report, or a bug fix. Let's hope for the latter! labels May 28, 2020
@alexbruy
Copy link
Contributor Author

While looking at this I also notices couple of other issues:

  1. Selecting by tag does not select color ramps which belongs to the tag
  2. We have dedicated "Select all" button at the bottom of the Export/Import dialog, but selecting all symbols also possible with "Select by group". Probably we can remove it.

@github-actions github-actions bot added this to the 3.14.0 milestone May 28, 2020
@alexbruy
Copy link
Contributor Author

Don't know why indentation test failed. I have run astyle.sh again and all files were left unchanged.

@nyalldawson
Copy link
Collaborator

Not sure about backport to 3.10.

Let's skip it -- it's not a high priority fix IMO.

@nyalldawson nyalldawson merged commit d2f72e8 into qgis:master May 28, 2020
@alexbruy alexbruy deleted the export-favorites branch May 29, 2020 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! GUI/UX Related to QGIS application GUI or User Experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Style Manager] Missing Favorites category in the "select by group" dialog from "Export symbols"
2 participants