Review Request: KHBox cleanup in frameworks

Albert Astals Cid tsdgeos at terra.es
Thu Jul 12 23:26:03 BST 2012



> On July 12, 2012, 10:20 p.m., Christoph Feck wrote:
> > kde3support/kdeui/k3listviewsearchline.cpp, line 537
> > <http://git.reviewboard.kde.org/r/105531/diff/1/?file=72349#file72349line537>
> >
> >     Do not set fixed spacing, let the style figure it out.

Sorry but no, i'm not doing that, i'm removing KHBox, if you want me to do other random stuff, i'm out


- Albert


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


On July 12, 2012, 10:18 p.m., Albert Astals Cid wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105531/
> -----------------------------------------------------------
> 
> (Updated July 12, 2012, 10:18 p.m.)
> 
> 
> Review request for kdelibs and David Faure.
> 
> 
> Description
> -------
> 
> Kill the uses of KHBox
> 
> 
> Diffs
> -----
> 
>   kde3support/kdeui/k3listviewsearchline.h 7073c02 
>   kde3support/kdeui/k3listviewsearchline.cpp 55b7d27 
>   kde3support/kdeui/k3passworddialog.cpp 9ef8ce5 
>   kdeui/dialogs/kpassworddialog.cpp 6ce9eb2 
>   kdeui/tests/kcomboboxtest.cpp 883a0f6 
>   kdeui/tests/ktreewidgetsearchlinetest.cpp 955bc15 
>   kdeui/widgets/kmenu.cpp 95a37c6 
>   kioslave/http/kcookiejar/kcookiewin.cpp 56f6926 
>   kutils/kcmultidialog.cpp 7ff62c9 
> 
> Diff: http://git.reviewboard.kde.org/r/105531/diff/
> 
> 
> Testing
> -------
> 
> Compiles
> 
> 
> Thanks,
> 
> Albert Astals Cid
> 
>

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


More information about the kde-core-devel mailing list