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

Show possibly rotated canvas map in overview #1889

Closed
wants to merge 2 commits into from

Conversation

strk
Copy link
Contributor

@strk strk commented Feb 6, 2015

@strk
Copy link
Contributor Author

strk commented Feb 6, 2015

NOTE: includes the new visiblePolygon method submitted as #1888

@strk
Copy link
Contributor Author

strk commented Feb 6, 2015

\cc @nyalldawson @wonder-sk

@strk
Copy link
Contributor Author

strk commented Feb 6, 2015

travis failure depends on rounding failure in a test (not failing on my local machine) that's being handled in #1888 -- I guess it would be better to merge #1888 first, once the rounding issue is fixed

Sandro Santilli added 2 commits February 6, 2015 20:42
Centralizes code to return a possible rotated map area boundaries.
Makes use of the new method from the grid decoration.

Adds new testcase for QgsMapSettings, including test for visibleExtent,
mapUnitsPerPixel and the new visiblePolygon method.
@strk
Copy link
Contributor Author

strk commented Feb 6, 2015

Rebased again to #1888 after fixing that one to avoid the "breaking" change.

@strk
Copy link
Contributor Author

strk commented Feb 7, 2015

Pushed as 7d6b7cc

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.

1 participant