-
-
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
Fix map grid decoration bottom vertical labels placement and vertical labels background placement #44968
Fix map grid decoration bottom vertical labels placement and vertical labels background placement #44968
Conversation
Fixes the position of the bottom vertical label of a map grid decoration when a non zero "Distance to map frame" is set
8500630
to
4c657cb
Compare
The QGIS project highly values your contribution and would love to see this work merged! Unfortunately this PR has not had any activity in the last 14 days and is being automatically marked as "stale". If you think this pull request should be merged, please check
|
I think this bugfix should be merged and backported. |
I'd rather track this down instead -- we use QgsTextRenderer a LOT in qgis now, so the underlying issue would affect other code paths... |
The background issue is fixed in #45425 -- can you drop that commit so we can merge the other? |
4c657cb
to
cc4be55
Compare
Yes, of course. Commit dropped. Thanks a lot @nyalldawson! So, does your fix makes no longer needed a similar "patch" used for map layout grid in QgsLayoutItemMapGrid::drawCoordinateAnnotation? QGIS/src/core/layout/qgslayoutitemmapgrid.cpp Lines 1390 to 1395 in de20bca
|
Hmm... Probably. I suspect that's been in place there from before the grids were drawn using the text renderer. There's good tests coverage there -- you could try removing and see if everything passes |
Description
Fixes the position of the bottom vertical label of a map grid decoration when a non zero "Distance to map frame" is set (regression introduced with #37133 since 3.14).
Fixes the vertical labels background placement of a map grid decoration.Now fixed with #45425.(It seems to me the QgsTextRenderer::drawText function has an issue with a rotated background placement, but I couldn't find the cause)
Fixes #44907.
This PR needs to be backported to
3.20 and3.16 branches.Before:
![image](https://user-images.githubusercontent.com/16253859/132981889-bd44f69b-6b63-49e1-a288-e0e20e2bbe88.png)
After:
![image](https://user-images.githubusercontent.com/16253859/132981880-ddd403b6-9aed-4578-af36-079823f0428d.png)
Edited after #45425.