<div dir="ltr"><div><div><div><div><div><div><div>Hi<br><br></div>I just made a small test and commented the line<br>currentLabel->adjustSizeToText();<br></div>Then recompile QGIS master and tested it with a classical text box (not html rendered) with a long text.<br>
<br></div>I personnaly prefer the behaviour when this method is not called : the text box keep the size set by the user in QGIS. And the text wraps when it reaches the right side of the Box. I even tried with "HTML rendered" on (and my fake X Server running), and it worked well too.<br>
<br></div>At the contrary, when adjustSizeToText is used, the box is transformed in a one line box, which goes to the right of the entire composition (out of it) : text does not wrap. <br><br></div>So for me, simply removing the adjustSizeToText call would be enough. Or we have to make sure this method does the job correctly, which may be a bit hard to implement : we need to avoid to have collision with other composition items, etc.<br>
<br></div>Regards<br></div>Michael<br><div><div><div><div><div><br></div></div></div></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/5/28 René-Luc D'Hont <span dir="ltr"><<a href="mailto:rldhont@gmail.com" target="_blank">rldhont@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Marco<br><div><div class="gmail_extra"><br><div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<br></div>
I guess yes. 'adjustSizeToText()' works for text, not html. Html support for composer label was added recently to the composer label class. Please adapt the relevant lines to work with html labels too. </blockquote>
<div><br></div></div><div>I'll adapt it<br></div><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
<br>
>Question for Marco, Why removed label, if the request doesn't define a value ? Why not used the default value ?<br>
<br></div>
The idea was that the web client has the possibility to remove an advertised text label completely if it is not used. E.g. if a composition defines a text item with id 'author', the web client can simply ignore it if it does not want to have author's name printed. Note that the labels without ids are not removed as they are considered to be fix, so not subject to client text replacement (original meaning of label id was that this text item is subject to text replacement by web client. Later, the id has been generalized as an id for every item type).<br>
<br>
Another possibility would be to remove the item only if the client gives an empty text and use the default text otherwise. What do you think?<br></blockquote><div><br><br></div></div><div>I prefer to remove the label only if the user gives an empty text.<br>
</div><div>Can I do it ?<br><br><br></div><div>Regards,<br></div><div>René-Luc<br></div><div><div class="h5"><div> <br><br>
On <a href="tel:27.05.2013%2022" value="+12705201322" target="_blank">27.05.2013 22</a>:23, rldhont wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
Hi Marco and other devs,<br>
<br>
In src/mapserver/qgsconfigparser.<u></u>cpp:530 we can read :<br>
<br>
//replace label text<br>
foreach ( QgsComposerLabel *currentLabel, composerLabels )<br>
{<br>
QString title = parameterMap.value( currentLabel->id().toUpper() );<br>
<br>
if ( title.isEmpty() )<br>
{<br>
//remove exported labels not referenced in the request<br>
if ( !currentLabel->id().isEmpty() )<br>
{<br>
c->removeItem( currentLabel );<br>
delete currentLabel;<br>
}<br>
continue;<br>
}<br>
<br>
currentLabel->setText( title );<br>
currentLabel-><u></u>adjustSizeToText();<br>
}<br>
<br>
This code is used to draw composer label with value defined by the QGIS WMS Server Request GetPrint.<br>
<br>
The bug defined in the issues #7894 <a href="http://hub.qgis.org/issues/7894" target="_blank">http://hub.qgis.org/issues/<u></u>7894</a> is probably du to the method adjustSizeToText()<br>
Someone can confim it ?<br>
<br>
Question for Marco, Why removed label, if the request doesn't define a value ? Why not used the default value ?<br>
<br>
Regards,<br>
René-Luc D'Hont<br>
3Liz<br></div></div>
______________________________<u></u>_________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/qgis-<u></u>developer</a><span><font color="#888888"><br>
</font></span></blockquote><span><font color="#888888">
<br>
<br>
-- <br>
Dr. Marco Hugentobler<br>
Sourcepole - Linux & Open Source Solutions<br>
Weberstrasse 5, CH-8004 Zürich, Switzerland<br>
<a href="mailto:marco.hugentobler@sourcepole.ch" target="_blank">marco.hugentobler@sourcepole.<u></u>ch</a> <a href="http://www.sourcepole.ch" target="_blank">http://www.sourcepole.ch</a><br>
Technical Advisor QGIS Project Steering Committee<br>
<br>
______________________________<u></u>_________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/qgis-<u></u>developer</a><br>
</font></span></blockquote></div></div></div><br></div></div></div>
<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>
<br></blockquote></div><br></div>