Review Request: Bypassing Qt bug in the desktop toolbox

[Marco Martin]

On Monday 08 June 2009, Alexis Ménard wrote:
> http://qt.gitorious.org/qt/qt/merge_requests/407
>
> Is this fix resolve your problem? Can you try?

/me thinks so, i did that patch for this kind of problem... (it fixes the look 
of opensocial applet that had that problem)
for qt-copy uuh there weren't talks to scrap it? is it still active?
if so yeah, probably that patch should be put on it, what is the policy to put 
stuff in here?

Cheers,
Marco Martin


> If yes, then i would prefer a qt-copy patch than a hack in the code. People
> will forget it and it pollute the code base. Qt-copy patches are
> updated/removed properly when new release of Qt are available.
>
> I will check tomorrow if this commit is in 4.5.2 packages...if yes, then no
> point to do this, 4.5.2 arrive very soon.
>
> On Mon, Jun 8, 2009 at 8:16 PM, Ivan Cukic
> <ivan.cukic+kde at gmail.com<ivan.cukic%2Bkde at gmail.com>
>
> > wrote:
> >
> >
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://reviewboard.kde.org/r/804/
> > -----------------------------------------------------------
> >
> > (Updated 2009-06-08 11:16:37.652296)
> >
> >
> > Review request for Plasma.
> >
> >
> > Summary (updated)
> > -------
> >
> > Qt has a b-u-g causing it to show a child element after calling show()
> > even though the parent is hidden.
> >
> > (spelled b-u-g since revboard automatically makes links for bugs...)
> >
> >
> > Diffs
> > -----
> >
> >  /trunk/KDE/kdelibs/plasma/private/desktoptoolbox.cpp 978915
> >
> > Diff: http://reviewboard.kde.org/r/804/diff
> >
> >
> > Testing
> > -------
> >
> > Yes :)
> >
> >
> > Thanks,
> >
> > Ivan
> >
> > _______________________________________________
> > Plasma-devel mailing list
> > Plasma-devel at kde.org
> > https://mail.kde.org/mailman/listinfo/plasma-devel