> 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. > > Matthias Fuchs wrote: > 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. >
(Answering almost one year later, new record :/) I worked a bit on KMessageWidget two weeks ago, the current version in KDE/4.8 should correctly adjusts its height when wrapping and react to resizing. Can you test it? - Aurélien ----------------------------------------------------------- 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. > > > Description > ------- > > 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 Fuchs > >
