> On May 31, 2016, 1:38 a.m., Aleix Pol Gonzalez wrote: > > Would it be possible to get this as a KWidgetsAddons patch? It seems like > > it could easily be turned into a `KStyleSelectorMenu`. > > René J.V. Bertin wrote: > True, and the same would go for the "Theme" (colour palette) menu > Kdenlive and digiKam provide. Why not adding it to KStandardActions btw? I'm > exploring a QProxyStyle solution (from the Qt ML) to adjusting the > `layoutSpacing` issue with the Macintosh style (which could be presented as a > virtual style not requiring any additional plugins) and the easier it is for > applications to add a Style menu the wider such a fix should become available. > > What about storing the user preference, would that go through the > framework too or would that be purely up to the application?
Maybe it should go in KConfigWidgets then, so you can leverage KConfig. For now, try to encapsulate it properly within KDevelop so we know what the class will look like. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128056/#review96062 ----------------------------------------------------------- On May 31, 2016, 10:48 a.m., René J.V. Bertin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128056/ > ----------------------------------------------------------- > > (Updated May 31, 2016, 10:48 a.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/mainwindow_p.h abef1a7 > 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 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
