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

Unify UX behavior, pt 2: rotation #4646

Merged
merged 11 commits into from
May 31, 2017
Merged

Conversation

nyalldawson
Copy link
Collaborator

This PR unifies the behavior of controls for specifying rotation, so that all rotation is set in degrees clockwise, and accepts values from -360->360.

It also implements an important fix where label rotation is changed to operate in the same direction (clockwise), so that it matches symbology, map rotation, etc.

@nyalldawson nyalldawson added the API Break! Breaks stable API. Proceed with extreme caution!! label May 30, 2017
@pcav
Copy link
Member

pcav commented May 30, 2017

Great, it was SO annoying!
Thanks Nyall.

@nyalldawson nyalldawson merged commit 77fd7e6 into qgis:master May 31, 2017
@nyalldawson nyalldawson deleted the rotation branch May 31, 2017 02:54
@DelazJ
Copy link
Contributor

DelazJ commented Jun 2, 2017

@nyalldawson Thanks for the fix.

Any reason to have let the description of DD button beside rotation spinbox state that expected inputs are from 0 to 360 while it can also take -360 to 0 values?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Break! Breaks stable API. Proceed with extreme caution!!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants