> On July 14, 2016, 11:14 a.m., Hannah von Reth wrote:
> > sublime/kwidgetstyleselector.cpp, line 43
> > <https://git.reviewboard.kde.org/r/128056/diff/3/?file=468753#file468753line43>
> >
> >     Dont fall back to Windows.
> >     Windows is the win3.11 like style.
> >     I guess the current one is Windows Vista?
> 
> René J.V. Bertin wrote:
>     I have no idea and no way to check (except through trying to understand 
> the MSWin-specific Qt sources). It has to be an appropriate style that's 
> always available because part of Qt.

>From the Windows qpa.
```
static inline QStringList styleNames()
{
    QStringList result;
    if (QSysInfo::WindowsVersion >= QSysInfo::WV_VISTA)
        result.append(QStringLiteral("WindowsVista"));
    if (QSysInfo::WindowsVersion >= QSysInfo::WV_XP)
        result.append(QStringLiteral("WindowsXP"));
    result.append(QStringLiteral("Windows"));
    return result;
}
```


- Hannah


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


On June 14, 2016, 5:20 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128056/
> -----------------------------------------------------------
> 
> (Updated June 14, 2016, 5:20 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X, KDE Frameworks and KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> I filed a bug report recently that raises an issue about the QTabBar widget 
> for the tabbed document interface  
> (https://bugs.kde.org/show_bug.cgi?id=363473). On OS X, that widget is 
> rendered like the native tab bar widget that should only be used in dialogs 
> and comparable views where the number of tabs is preferably fixed and 
> limited. There are also other rendering issues which probably stem from 
> presumptions KDevelop makes about the tab layout in the extensions it 
> implements.
> Qt does provide a `documentMode` which changes the look to suit use for 
> document tabs better, but this mode doesn't work well with KDevelop's 
> extensions either.
> 
> For lack of a better solution or workaround I would thus like to explore the 
> idea to provide a widget style picker, like KDenlive does (presumably not 
> without reason either). The underlying idea is that it allows users to find 
> an style that works better for them if they feel a reason to do so. This 
> option works regardless of whether a platform theme plugin is available.
> 
> For now the patch is a proof-of-concept and work in progress. It is still 
> lacking a mechanism to make the style choice persistent across restarts; I 
> think I'll need a hand in determining how to do that correctly (it should be 
> a global setting, not a session-specific setting I think).
> 
> 
> Diffs
> -----
> 
>   sublime/CMakeLists.txt 2144087 
>   sublime/kwidgetstyleselector.h PRE-CREATION 
>   sublime/kwidgetstyleselector.cpp PRE-CREATION 
>   sublime/mainwindow_p.cpp 74ef494 
> 
> Diff: https://git.reviewboard.kde.org/r/128056/diff/
> 
> 
> Testing
> -------
> 
> On OS X 10.9.5 and Linux, both with fw. (5.20.0 and) 5.22.0 and Qt 5.6.0
> 
> 
> File Attachments
> ----------------
> 
> diff for kdevelopui.rc
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/05/30/cb07a061-aef5-42f5-bc83-68ca7ce2ce3b__patch-kdevplatform-add-style-menu-uirc.diff
> This screenshot shows where the menu item appears with the kdevelopui.rc 
> patch in another attachment. This KDevelop instance was running with my 
> platform theme plugin and my OS X palette and config for the QtCurve style. 
> Only the UI fonts change when the p
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2016/05/30/543bffc8-26fd-44f8-8bd8-372a24c9b01f__Screen_Shot_2016-05-30_at_15.47.14.png
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to