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

[Backport queued_ltr_backports] Fix inconcequent use of DPI at generating the WMS legend #52147

Merged
merged 11 commits into from
Mar 11, 2023

Conversation

signedav
Copy link
Contributor

@signedav signedav commented Mar 8, 2023

Manual backport of #52015
By @signedav

- Consider DPI of QgsWmsRenderContext what is the OGC default (0.28 mm per pixel) or the passed WMS parameter - this is done by creating the QgsRenderContext by the mapSettings (with BBOX) or applying the dotsPerMm to the scaleFactor.

- Additionally the image size needs to be calculated according to the QgsRenderContext now, what means it needs to be generated before.

- The QPainter needs to be applied after to the context (since it's not passed by creating the context anymore).
… context settings according to the mapSettings since it NEVER has a BBOX (since of RULE)
…correct CRS and care about meter based crs. This fixes #50366
…labels and having not yet a fix font with "LAYERFONTBOLD": "TRUE", "LAYERFONTSIZE": "12", "LAYERFONTFAMILY": self.fontFamily, "ITEMFONTBOLD": "TRUE", "ITEMFONTSIZE": "12", "ITEMFONTFAMILY": self.fontFamily
@github-actions github-actions bot added this to the 3.32.0 milestone Mar 8, 2023
@signedav signedav marked this pull request as draft March 9, 2023 16:20
@signedav signedav force-pushed the backport-52015-to-queued_ltr_backports branch from 799e25a to b29e490 Compare March 10, 2023 12:23
@signedav signedav marked this pull request as ready for review March 11, 2023 06:35
@signedav signedav merged commit 86e655d into queued_ltr_backports Mar 11, 2023
@signedav signedav deleted the backport-52015-to-queued_ltr_backports branch March 11, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants