Review Request 116044: In kstyle: Use the iconSize from mainToolbar

[David Faure]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116044/#review50807
-----------------------------------------------------------


The part of the description that says "if accepted will modify kstyle as well" doesn't really make sense anymore (to fix if it's in your commit log too).

The bit I'm not sure about is: using MainToolbar icon style everywhere ... how does that take care of other toolbars then?
The idea (long ago) was to be able to have (large) icons and text in the main toolbar, and (small) icons only in other toolbars. Is that idea dropped, or handled elsewhere?

- David Faure


On Feb. 25, 2014, 11:28 a.m., Hugo Pereira Da Costa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116044/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2014, 11:28 a.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: frameworkintegration
> 
> 
> Description
> -------
> 
> This is a spinoff of https://git.reviewboard.kde.org/r/112335/ originally from afiestas
> Copying his own words: 
> 
> In the qplatformplugin  we use information from MainToolbar (which makes sense) but in the styles we use information from Toolbar.
> This unify both by using MainToolbar in styles (if accepted will modify kstyle as well).
> 
> Code has been removed from oxygen now that it derives from KStyle again
> 
> 
> Diffs
> -----
> 
>   src/kstyle/kstyle.cpp c0528b3 
> 
> Diff: https://git.reviewboard.kde.org/r/116044/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Hugo Pereira Da Costa
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140225/90f9c016/attachment.html>