<div dir="ltr"><div><div>Sandro, Nyall,<br><br>Discussing this with Martin earlier today, he came up with a test patch to see what would happen if the code was "blindly" modified to use QgsMapSettings' setRotation() function. The result is interesting, it actually _mostly_ works (see attached patch):<br>- before the patch, rotating the map composer of 45 degrees looked like this: <a href="http://i.imgur.com/8e88VZf.png">http://i.imgur.com/8e88VZf.png</a><br>- applying the patch, rotating the map composer of 45 degrees now looks like this: <a href="http://i.imgur.com/VudbYrn.png">http://i.imgur.com/VudbYrn.png</a><br><br>The map rotation without the patch shows the current limitations, namely undesired rotated label, undesired rotated symbology, etc.<br><br>With the patch applied, the label is horizontal, and my centroid triangle symbology's orientation is respected. There is however an issue with the calculation of the rotated map extend leading to a slightly misplaced map overview polygon on the larger map as well as a slight misalignment of the grid. <br><br>Regarding the map overview issue, this got me to think that instead of having a map visibility extend calculation code in the composer map item, the QgsMapSettings object should have a function that returns a visible extend polygon (in WKT string, or whatever). That would simplify and bulletproof the code in many places beyond the composer map item.<br><br></div>Food for thoughts.<br><br></div>Math<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 16, 2015 at 9:00 AM, Mathieu Pellerin <span dir="ltr"><<a href="mailto:nirvn.asia@gmail.com" target="_blank">nirvn.asia@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sandro,<br>
<br>
The last few weeks has been a nice leap forward when it comes to your<br>
rotation feature, congratulation on that.<br>
<br>
IMO, based on the improvements, it's worth considering _removing the<br>
activation option altogether_ from the preference window. I can't see<br>
any stability and usability issue that would warrant disabling it.<br>
<br>
That said, in my view, a basic composer support (i.e. map item's<br>
rotation feature) should be moved at the top of your priority list.<br>
Right now, the benefits of your rotation work _cannot_ be exported<br>
outside of the QGIS canvas, which IMO is a serious limitation. In most<br>
of my QGIS workflows, the final cartographic products are exported<br>
through the composer to obtain an high resolution file that can be<br>
printed out (i.e. I can't just go save the canvas as an image :) ).<br>
Not having map items make use of your rotation feature is a<br>
significant restriction.<br>
<br>
Cheers, and thanks again for the work,<br>
<br>
Math<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Thu, Jan 15, 2015 at 9:59 PM, Sandro Santilli <<a href="mailto:strk@keybit.net">strk@keybit.net</a>> wrote:<br>
> I've spent more time on rotation (only partially funded), fixing:<br>
><br>
>  - Label pre-rotation (#11814)<br>
>  - North Arrow Decoration (#11818)<br>
>  - Grid decoration (#11817)<br>
><br>
> Leftover issues (open for funding):<br>
><br>
>  - Plugin layers (#11900) -- needs feedback<br>
>  - Overview map (#11862) -- minor, IMHO<br>
>  - Composer support (#11912) -- wish ?<br>
>  - Atlas doesn't consider map rotation when setting scale (#11954)<br>
><br>
> Please report any other issue that might arise with rotation enabled<br>
> and properly tag it with the "rotation" label.<br>
><br>
> I'd like to partecipate to the PSC meeting tomorrow but it is only<br>
> possible for me before 18:00 CET. I hadn't found the schedule here:<br>
> <a href="http://hub.qgis.org/wiki/17/PSC_Meeting_16_January_2015" target="_blank">http://hub.qgis.org/wiki/17/PSC_Meeting_16_January_2015</a><br>
> At what time do you plan to meet ?<br>
><br>
> --strk;<br>
><br>
>   ()   Free GIS & Flash consultant/developer<br>
>   /\   <a href="http://strk.keybit.net/services.html" target="_blank">http://strk.keybit.net/services.html</a><br>
> _______________________________________________<br>
> Qgis-developer mailing list<br>
> <a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</div></div></blockquote></div><br></div>