Review Request 109410: set the style to plastique if not running in KDE

[C. Boemann]

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

Ship it!


Fine with me, but I think it's wrong of you to hard code the style for krita - at least give the user a way out. It's not impossible that the user is running oxygen or some other style that will work, even if not on KDE

- C. Boemann


On March 11, 2013, 9:26 a.m., Boudewijn Rempt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109410/
> -----------------------------------------------------------
> 
> (Updated March 11, 2013, 9:26 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> For Krita, with its default dark theme, only two styles work: oxygen and plastique. Oxygen is default in KDE, if the user selects something else, it's their problem. On windows, osx and ubuntu, the default style is broken for krita, so hard-code plastique if the user is running krita outside KDE.
> 
> This means that on Windows, other apps than Krita will use the default windows style again. It might be worth it to reinstate the code that detects the windows version and sets the right style, but that is up for discussion.
> 
> 
> Diffs
> -----
> 
>   libs/main/KoApplication.cpp 87536f1 
> 
> Diff: http://git.reviewboard.kde.org/r/109410/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Boudewijn Rempt
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130311/ee06a915/attachment.htm>