Review Request: KMessageWidget: Adapt height to text changes

[Matthias Fuchs]

> On May 2, 2011, 5:49 p.m., Aurélien Gâteau wrote:
> > Thanks for the patch Mattias! This is indeed an annoying bug but your fix does not work when the widget is resized. heightForWidth() needs to be implemented I think.

The problem is that resizing takes no effect yet either. Here without the patch enabling word wrap instantly increases the height, even if there is enough horizontal space. So the fix would not make matters worse.

I am not sure how to fix the height problem in general though. To me it appears that QLabel is lacking here.


- Matthias


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101272/#review3071
-----------------------------------------------------------


On May 2, 2011, 12:35 p.m., Matthias Fuchs wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101272/
> -----------------------------------------------------------
> 
> (Updated May 2, 2011, 12:35 p.m.)
> 
> 
> Review request for kdelibs and Aurélien Gâteau.
> 
> 
> Summary
> -------
> 
> When there is a long or wrapping text, the text is cut off. This patch fixes that.
> 
> 
> Diffs
> -----
> 
>   kdeui/widgets/kmessagewidget.cpp 212ea0d 
> 
> Diff: http://git.reviewboard.kde.org/r/101272/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> 
>   http://git.reviewboard.kde.org/r/101272/s/154/
> 
>   http://git.reviewboard.kde.org/r/101272/s/155/
> 
> 
> Thanks,
> 
> Matthias
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110508/8ea675a3/attachment.htm>