Feature request #11912

rotation: the composer map item should rely on the new map rotation feature

Added by Mathieu Pellerin - nIRV over 2 years ago. Updated about 2 years ago.

Status:Closed Start Date:12/23/2014
Priority:Normal Due date:
Assigned to:Nyall Dawson % Done:

90%

Category:Map Composer / Printing
Target version:Future Release - High Priority
Platform: Resolution:
Platform version: Pull Request or Patch supplied:No
Status info: Tag:rotation

Description

Right now, the newly coded rotation feature is only useable on qgis' main canvas, meaning composer sheets can't make use of its benefits (a rotation which doesn't rotate symbology, layer, etc.).

The composer map item has a map rotation input, but it is not as efficient (as it rotates the rendered map, instead of rotating the datasets prior to symbology and labelling). The composer map item rotation feature should be updated to rely on the new map rotation code.

Newly inserted maps should also by default set its rotation value to the one currently used in the main canvas (like the item does for background color upon insertion)

Associated revisions

Revision a40eca4a343eacff8c610e7f945e675a0edf0ec6
Added by Sandro Santilli about 2 years ago

Use core support for map rotation in composer

Includes patch by Martin Dobias
See http://lists.osgeo.org/pipermail/qgis-developer/2015-January/036309.html

Fix #11912

History

Updated by Sandro Santilli about 2 years ago

A contributed patch by Martin was proveded by Mathieu here: http://lists.osgeo.org/pipermail/qgis-developer/2015-January/036309.html

Updated by Sandro Santilli about 2 years ago

The problem with Martin patch is that the rendered image becomes larger as rotation is added. It is not clipped and position is always set to 0,0 upper-left, resulting in a weird shift maximum at 45 and multiple degrees

Updated by Sandro Santilli about 2 years ago

Martin's patch applied in a PR that'll be used for staging: https://github.com/qgis/QGIS/pull/1882

Updated by Sandro Santilli about 2 years ago

  • Assigned to changed from Sandro Santilli to Nyall Dawson
  • % Done changed from 0 to 90

PR is ready for merge. Travis passes and Nyall deeply tested manually. Re-assigning to him for the final merge.

Updated by Sandro Santilli about 2 years ago

  • Status changed from Open to Closed

Also available in: Atom