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 vertical labeling when rotation is 270° #54667
Conversation
2aa8c18
to
1582b90
Compare
This test mask doesn't look right -- it's negating the whole test |
c700fa7
to
950a890
Compare
Hi @nyalldawson,
Yes you're right, I updated the control image instead. Let me know if it suits you better. |
@pblottiere I agree that the updated test image is arguably correct. But I'm slightly concerned about the risk of regressions here -- the logic with upside down label handling is extremely complex 😱 Could you add new tests with rotation +/- 1 degree off vertical so that we've got a test which still covers a similar use case as the existing one with vertical text? |
Yes I understand. I won't be surprised if this PR is finally not merged :).
I'm taking a look. |
950a890
to
e1a3c94
Compare
ca65dd7
to
6c1fd7e
Compare
6c1fd7e
to
58e011a
Compare
Hello @nyalldawson, CI is green 🎉. Let me know what you think :). Thanks! |
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
|
While we hate to see this happen, this PR has been automatically closed because it has not had any activity in the last 21 days. If this pull request should be reconsidered, please follow the guidelines in the previous comment and reopen this pull request. Or, if you have any further questions, just ask! We love to help, and if there's anything the QGIS project can do to help push this PR forward please let us know how we can assist. |
Description
See #54598
The label cannot be put in this position on master when
Show upside-down labels
is set tonever
: