-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Expose choice of distance unit for elevation profiles #53235
Conversation
This pull request has been tagged for the changelog.
You can edit the description. Format available for credits
Thank you! |
@nyalldawson 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. Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good, thanks!
One thought: how about adding a space between numbers and distance units - "100 m" instead of "100m" ? It seems that's the correct way of writing according to SI - https://grammarhow.com/do-you-put-a-space-between-number-and-unit/
menu Allows users to change the distance unit for situations where the map's default distance unit isn't appropriate
2739ef0
to
160b256
Compare
Done! |
@nyalldawson |
This PR adds choices of distance units for elevation profiles. It's exposed both for the interactive elevation profiles (via the settings toolbar button) and for layout elevation profiles.
Users can pick to override the default (ie canvas map units) distance unit.
In layout profiles, there's also an option which controls where the distance unit suffixes should be placed. Options are to hide the suffixes, for all values, for the first value, last value or first and last values.
note it is currently NOT possible to expose unit selection for the vertical axis, as this is currently dimension-less and is blocked by missing vertical CRS handling in QGIS. See qgis/QGIS-Enhancement-Proposals#267