https://issues.qgis.org/https://issues.qgis.org/favicon.ico2015-07-06T00:32:54ZQGIS Issue TrackingQGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=627832015-07-06T00:32:54ZNyall Dawson
<ul></ul><p>Can you share your project?</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=627852015-07-06T00:36:08ZGiovanni Manghigiovanni.manghi@gmail.com
<ul><li><strong>Status</strong> changed from <i>Open</i> to <i>Feedback</i></li></ul> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=627872015-07-06T00:43:13ZHolger Naumannholger.naumann@gmail.com
<ul><li><strong>File</strong> <a href="/attachments/download/8902/projekt.zip">projekt.zip</a> added</li><li><strong>File</strong> <a href="/attachments/download/8903/punktsymbole.zip">punktsymbole.zip</a> added</li><li><strong>File</strong> <a href="/attachments/download/8904/style.zip">style.zip</a> added</li></ul><p>No problem.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=627912015-07-06T01:32:28ZNyall Dawson
<ul><li><strong>Assignee</strong> set to <i>Vincent Mora</i></li></ul><p>Vincent - looks related to your legend related changes, mind taking a look?</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=630282015-07-31T00:38:49ZGiovanni Manghigiovanni.manghi@gmail.com
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Open</i></li></ul> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=630392015-07-31T03:50:10ZVincent Moravincent.mora@oslandia.com
<ul></ul><p>Nyall, it definitely looks like that, but I wasn't able to open the project when I tried (I think I forgot the symbols or something). I'm on a vacation at the moment, but I'll try and have a look asap.</p>
<p>Holger, what happens if you use mm instead of map units, does it look better, does the zoom level change anything ?</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=631042015-08-03T03:46:58ZHolger Naumannholger.naumann@gmail.com
<ul><li><strong>File</strong> <a href="/attachments/download/8959/example.png">example.png</a> added</li></ul><p>Hi Vincent,<br />if units are in mm, it is ok. But if one symbol is in mapunits, legend is broken.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=631322015-08-05T00:32:28ZVincent Moravincent.mora@oslandia.com
<ul></ul><p>Hi Holger,<br />Thanks for the update. The thing that seems to be broken seems to be the cropping of the symbol, the symbol size is right but the symbol takes way to much place. I'll have a look asap.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=631352015-08-05T07:02:50ZVincent Moravincent.mora@oslandia.com
<ul></ul><p>Nyall, I have a lead: when QgsSymbolV2LegendNode::minimumIconSize is called by QgsDefaultVectorLayerLegend::createLayerTreeModelLegendNodes, the model() is null, whereas it is not when QgsSymbolV2LegendNode::data() is called with decoration rôle.</p>
<p>For what I understand, it's a matter of setting the correct parent for the node in createLayerTreeModelLegendNodes. If you or Matthias could give me a hint, I'd be gratefull.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=631652015-08-08T00:21:33ZNyall Dawson
<ul></ul><p>Vmora - Martin will be the one to contact about this, he's got the most knowledge of how the legends work...</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=632062015-08-13T10:05:44ZVincent Moravincent.mora@oslandia.com
<ul><li><strong>Pull Request or Patch supplied</strong> changed from <i>No</i> to <i>Yes</i></li></ul><p>The fix is here <a class="external" href="https://github.com/qgis/QGIS/pull/2245">https://github.com/qgis/QGIS/pull/2245</a></p>
<p>Holger: thanks for the report.</p>
<p>Martin: thanks for the pointers.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=632272015-08-17T16:06:12ZMartin Dobiaswonder.sk@gmail.com
<ul><li><strong>Status</strong> changed from <i>Open</i> to <i>Closed</i></li></ul><p>Fixed in changeset <a class="changeset" href="https://issues.qgis.org/projects/qgis/repository/revisions/7d1a8d30199e2b6bf2669478f474c0911d04f6b1" title="Merge pull request #2245 from vmora/issue13078 Fix legend symbol size when using map units (fix ...">7d1a8d30199e2b6bf2669478f474c0911d04f6b1</a>.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=633602015-08-24T02:15:04ZHolger Naumannholger.naumann@gmail.com
<ul><li><strong>Status</strong> changed from <i>Closed</i> to <i>Reopened</i></li><li><strong>File</strong> <a href="/attachments/download/9018/Bildschirmfoto_von_%C2%BB2015-08-24_13-01-43%C2%AB.png">Bildschirmfoto_von_»2015-08-24_13-01-43«.png</a> added</li></ul><p>Hi Vincent, the problem is solved in the mapwindow. But in print composer, the problem exist.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=634242015-08-29T13:20:55ZNyall Dawson
<ul></ul><p>Please note that the scaling of legend shapes in composer legends is intentional and is the same beahviour as previous versions. So that's not an issue.</p>
<p>However, it seems like the size of the legend isn't calculated correctly on composer load when the legend contains shape sizes in map units - the legend is set to a massive width & height. Forcing a refresh of the legend fixes the size.</p> QGIS Application - Bug report #13078: Size of legend itemshttps://issues.qgis.org/issues/13078?journal_id=634252015-08-29T13:22:38ZNyall Dawson
<ul><li><strong>Status</strong> changed from <i>Reopened</i> to <i>Closed</i></li></ul><p>Actually - that's a unrelated issue. Closing this.</p>