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

Scalebar label cleanups #57964

Merged
merged 2 commits into from
Jul 11, 2024
Merged

Conversation

nyalldawson
Copy link
Collaborator

A couple of small tweaks as a lead in to some later scalebar improvements:

  • Use a single combo box for scale bar horizontal/vertical label placement. Since we only have 2 choices for each, we can reduce the number of widgets by just exposing 4 choices in a single combo.
  • Promote scalebar enums to enum class, move to Qgis

Since we only have 2 choices for each, we can reduce the number
of widgets by just exposing 4 choices in a single combo
@nyalldawson nyalldawson added the Cleanup Code cleanup label Jul 3, 2024
@github-actions github-actions bot added this to the 3.40.0 milestone Jul 3, 2024
Copy link

github-actions bot commented Jul 3, 2024

🪟 Windows builds ready!

Windows builds of this PR are available for testing here. Debug symbols for this build are available here.

(Built from commit 82d2189)

@nyalldawson nyalldawson merged commit 711c75d into qgis:master Jul 11, 2024
36 checks passed
@nyalldawson nyalldawson deleted the scalebar_label_pos branch July 11, 2024 19:36
@agiudiceandrea agiudiceandrea added the Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo. label Sep 2, 2024
@qgis-bot
Copy link
Collaborator

qgis-bot commented Sep 2, 2024

@nyalldawson
This pull request has been tagged as requiring documentation.

A documentation ticket will be opened at https://github.com/qgis/QGIS-Documentation when this PR is merged.

Please update the description (not the comments) with helpful description and screenshot to help the work from documentors.
Also, any commit having [needs-doc] or [Needs Documentation] in will see its message pushed to the issue, so please be as verbose as you can.

Thank you!

@qgis-bot
Copy link
Collaborator

qgis-bot commented Sep 2, 2024

@nyalldawson
A documentation ticket has been opened at qgis/QGIS-Documentation#9225
It is your responsibility to visit this ticket and add as much detail as possible for the documentation team to correctly document this change.
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cleanup Code cleanup Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants