-
-
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
add message to QGIS status bar on selection of vertices #58811
Conversation
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.
Nice!
Also keep in mind that once #58795 gets merged, QgsDistanceArea::measureLine()
might throw a QgsCsException
which will need to be handled.
{ | ||
QgsDistanceArea distArea = QgsDistanceArea(); | ||
distArea.setSourceCrs( mCurrentLayer->crs(), QgsProject::instance()->transformContext() ); | ||
distArea.setEllipsoid( QgsProject::instance()->ellipsoid() ); |
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.
I'd suggest to only set the ellipsoid for geographic crs
es, otherwise skip it so distArea
it will calculate a cartesian distance for projected crs
es
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.
No, I think this is correct as is. If users need purely cartesian distances then they should set the project ellipsoid to "none" accordingly.
use project distance units Co-authored-by: Stefanos Natsis <uclaros@gmail.com>
make const Co-authored-by: Stefanos Natsis <uclaros@gmail.com>
cons function Co-authored-by: Stefanos Natsis <uclaros@gmail.com>
Look ok for you @uclaros ? |
Description
Display message in status bar when vertices of Mesh are selected.