Re: Review Request: Make tools use all the available width

2009-02-15 Thread Marco Martin
On Saturday 14 February 2009, Artur Souza (MoRpHeUz) wrote: > On Sat, Feb 14, 2009 at 2:17 PM, Marco Martin wrote: > >maybe KIconLoader::SizeSmallMedium instead of 22? > > Well, it uses "22" all over the place, including in other > classes...maybe I can provide another patch replacing this "22

Re: Review Request: Make tools use all the available width

2009-02-14 Thread Artur Souza (MoRpHeUz)
On Sat, Feb 14, 2009 at 2:17 PM, Marco Martin wrote: >maybe KIconLoader::SizeSmallMedium instead of 22? Well, it uses "22" all over the place, including in other classes...maybe I can provide another patch replacing this "22"s with KIconLoader::SizeSmallMedium ? Thanks!! -- --

Re: Review Request: Make tools use all the available width

2009-02-14 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/77/#review105 --- Ship it! i think it looks better, like it :) /trunk/KDE/kdelibs/plasma

Review Request: Make tools use all the available width

2009-02-14 Thread Artur de Souza
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/77/ --- Review request for Plasma. Summary --- This patch makes each tool (action) in