Skip to content
Permalink
Browse files

allow numerical input in rotation map tool

* use a combo box to set the angle
* allow snapping to angles (directly enabled when shift + click)
* uses click-click behavior
* rotation anchor is defined on CTRL+click and not on mouse move (TODO: snapping for anchor placement)
  • Loading branch information
3nids committed Apr 28, 2015
1 parent 15c1dbc commit 96560c89b99c59a65ee20e3eb2f0e7656d5c6ea9
Showing with 345 additions and 162 deletions.
  1. +287 −150 src/app/qgsmaptoolrotatefeature.cpp
  2. +58 −12 src/app/qgsmaptoolrotatefeature.h

4 comments on commit 96560c8

@nyalldawson

This comment has been minimized.

Copy link
Collaborator

@nyalldawson nyalldawson replied Apr 28, 2015

@3nids nice work! One issue I've encountered testing this is that clicking the "x" button in the message bar will crash QGIS. I wonder if a new widget should be created for these types of interactive panels rather than abusing the messagebar? There's many other potential uses for this, including the processing commander, the offset tool spin box, adding spin boxes for the radius select, etc.... It would also leave the messagebar intact for displaying messages during these operations too.

@NathanW2

This comment has been minimized.

Copy link
Member

@NathanW2 NathanW2 replied Apr 28, 2015

@3nids

This comment has been minimized.

Copy link
Member Author

@3nids 3nids replied Apr 29, 2015

Thanks for the feedback.

I did not like the idea of the status bar (used in the offset tool).
The scene following the mouse is not working on linux and moreover you can't use the mouse (for the snapping).
What would you suggest? A toolbar at the bottom? A dock widget on the left?
I kind of like the idea of the toolbar.

My intention is then to make this coherent with the offset tool.

Thanks

@nyalldawson

This comment has been minimized.

Copy link
Collaborator

@nyalldawson nyalldawson replied Apr 29, 2015

@3nids I don't like the status bar either. I'd say a new toolbar at the bottom - it could be heavily based off the messagebar, but specifically for use by (non-message) widgets.

Please sign in to comment.
You can’t perform that action at this time.