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

[Boudewijn Rempt]

> On March 11, 2013, 9:35 a.m., C. Boemann wrote:
> > 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

Well, I can check whether the current style is oxygen, and if not, enable plastique, or even enable plastique, then try to enable oxygen (which is better than plastique). Or make the style configurable in krita's settings dialog. But I don't want to risk anyone to start krita with another style than these two -- I haven't seen a single style other than these that is not broken.


- Boudewijn


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


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/cb64144d/attachment.htm>