D26032: feat: add Control API for auto rotation

volkov added inline comments. INLINE COMMENTS > control.cpp:266 > +const auto val = info[QStringLiteral("autorotate")]; > +return !val.canConvert() || val.toBool(); > +} Why treat an inconvertible value as true and not false? REPOSITORY R104 KScreen REVISION

D28683: Get rid of workaround that prevents file dialog from being blocked

volkov abandoned this revision. volkov added a comment. Moved to https://invent.kde.org/plasma/plasma-integration/-/merge_requests/1 REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D28683 To: volkov, #plasma Cc: plasma-devel, Orage,

D11573: Use QPlatformWindow::startSystemMove() with Qt 5.11

volkov abandoned this revision. volkov added a comment. Moved to https://invent.kde.org/plasma/breeze/-/merge_requests/7 REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D11573 To: volkov, #breeze Cc: graesslin, davidedmundson, broulik, plasma-devel, Orage, LeGast00n,

D28683: Get rid of workaround that prevents file dialog from being blocked

volkov added a comment. DialogIsQtWindow was introduced in https://codereview.qt-project.org/c/qt/qtbase/+/144710 REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D28683 To: volkov, #plasma Cc: plasma-devel, Orage, LeGast00n,

D28683: Get rid of workaround that prevents file dialog from being blocked

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Invisible proxy QDialog blocks input when the native dialog is shown, and the workaround is to show the native file dialog with a

D14562: Introduce Theme.smallestFont

volkov updated this revision to Diff 76554. volkov added a comment. registered for 2.2 REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14562?vs=38964=76554 BRANCH master REVISION DETAIL https://phabricator.kde.org/D14562 AFFECTED FILES

D26244: Install all kcm plugins into kcms subdir

volkov updated this revision to Diff 72390. volkov added a comment. build metadata from desktop files into the plugins REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D26244?vs=72249=72390 BRANCH master REVISION DETAIL

D26244: Install all kcm plugins into kcms subdir

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY To reduce clutter in the plugins dir. REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL

D6313: Support Icon Scale from Icon naming specification 0.13

volkov added inline comments. INLINE COMMENTS > broulik wrote in kiconloader.cpp:1368 > Dunno, probably oversight or forgotten to rebase.. Looks like a rebase: https://phabricator.kde.org/D6313?vs=31197=34779=ignore-most#toc REPOSITORY R302 KIconThemes REVISION DETAIL

D6313: Support Icon Scale from Icon naming specification 0.13

volkov added inline comments. INLINE COMMENTS > kiconloader.cpp:1368 > > -if (group >= 0 && group < KIconLoader::LastGroup) { > +if (group >= 0) { > img = d->mpEffect.apply(img, group, state); Why this line reverts D12002 ? REPOSITORY

D22766: Fix positioning of output identifiers on screens with different dprs

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Find QScreen corresponding to KScreen::Output and use its geometry to position the output identifier. REPOSITORY R104 KScreen

D22482: WIP: Show file dialog in directory mode when only directory is requested

volkov added a comment. This is for non-sandboxed applications that want to use KDE file dialog. REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop REVISION DETAIL https://phabricator.kde.org/D22482 To: volkov, apol, jgrulich Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas,

D22482: WIP: Show file dialog in directory mode when only directory is requested

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop BRANCH Plasma/5.15 REVISION DETAIL https://phabricator.kde.org/D22482 AFFECTED

D20445: signalplotter: Add highdpi support

volkov added a comment. Beams look ugly when their width is >= 4 pixels. The idea is to draw them with the same width regardless of DPI. Another option is to not set Qt::FlatCap for thick pens, but I prefer this one. REPOSITORY R111 KSysguard Library REVISION DETAIL

D20445: signalplotter: Add highdpi support

volkov added inline comments. INLINE COMMENTS > zzag wrote in ksignalplotter.cpp:897-899 > Could you please explain why we have to do this? To avoid artifacts when painting beams. REPOSITORY R111 KSysguard Library REVISION DETAIL https://phabricator.kde.org/D20445 To: volkov Cc: zzag,

D20445: signalplotter: Add highdpi support

volkov updated this revision to Diff 56102. volkov added a comment. use QPen::setWidthF() instead of QPen::setWidth() REPOSITORY R111 KSysguard Library CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20445?vs=56079=56102 BRANCH master REVISION DETAIL

D20445: signalplotter: Add highdpi support

volkov updated this revision to Diff 56079. volkov added a comment. reduced beam width REPOSITORY R111 KSysguard Library CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20445?vs=55922=56079 BRANCH master REVISION DETAIL https://phabricator.kde.org/D20445 AFFECTED FILES

D20445: signalplotter: Add highdpi support

volkov added a comment. Hi, thanks for the review. I know about the issues, I've uploaded the patch to avoid losing it :) REPOSITORY R111 KSysguard Library REVISION DETAIL https://phabricator.kde.org/D20445 To: volkov Cc: zzag, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot,

D20445: signalplotter: Add highdpi support

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R111 KSysguard Library BRANCH master REVISION DETAIL https://phabricator.kde.org/D20445 AFFECTED FILES

D19496: [kcm/componentchooser] Remove unused ktimerdialog.{h,cpp}

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R119:14d6402abc34: [kcm/componentchooser] Remove unused ktimerdialog.{h,cpp} (authored by volkov). REPOSITORY R119

D19496: [kcm/componentchooser] Remove unused ktimerdialog.{h,cpp}

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Amend 2f83a4434a888cd17b03af1f9925cbb054256ade .

D18454: Fonts KCM: Port from kde_file.h to QFileInfo

volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D18454 To: volkov, #plasma, davidedmundson Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D18454: Fonts KCM: Port from kde_file.h to QFileInfo

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D18454 AFFECTED FILES kcms/fonts/kxftconfig.cpp

D18449: Remove dependency on KDELibs4Support for some KCM modules

volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D18449 To: volkov, broulik Cc: broulik, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D18449: Remove dependency on KDELibs4Support for some KCM modules

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.14 REVISION DETAIL https://phabricator.kde.org/D18449 AFFECTED FILES

D18300: [Mouse KCM] Port away from KDELibs4Support

volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D18300 To: volkov, broulik Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D18300: [Mouse KCM] Port away from KDELibs4Support

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D18300 AFFECTED FILES kcms/mouse/CMakeLists.txt

D17659: Set parent window for dialogs

volkov added a comment. Oh, I see. REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop REVISION DETAIL https://phabricator.kde.org/D17659 To: volkov, jgrulich, apol Cc: graesslin, broulik, apol, plasma-devel, kvanton, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai,

D17659: Set parent window for dialogs

volkov added a comment. I mean this plugin: https://cgit.kde.org/kwin.git/tree/plugins/qpa REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop REVISION DETAIL https://phabricator.kde.org/D17659 To: volkov, jgrulich, apol Cc: graesslin, broulik, apol, plasma-devel, kvanton,

D17659: Set parent window for dialogs

volkov closed this revision. REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop REVISION DETAIL https://phabricator.kde.org/D17659 To: volkov, jgrulich, apol Cc: graesslin, broulik, apol, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts,

D17659: Set parent window for dialogs

volkov added a comment. Is it possible to implement support for foreign windows in KWin QPA plugin? REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop BRANCH Plasma/5.14 REVISION DETAIL https://phabricator.kde.org/D17659 To: volkov, jgrulich, apol Cc: graesslin, broulik,

D17659: Set parent window for dialogs

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY This requires fix in Qt to work for standard dialogs: https://codereview.qt-project.org/#/c/248559/ REPOSITORY R838 Flatpak

D16269: Show correct current download and upload speed

volkov added a comment. The problem with this change is that speed is shown in bytes, not in bytes/s. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D16269 To: jgrulich, #plasma, broulik Cc: volkov, plasma-devel, ragreen, Pitel, ZrenBot,

D16269: Show correct current download and upload speed

volkov added a comment. I mentioned in D14835 that the total number of received/transmitted bytes is more useful than the current speed. One can see the current speed on the graph. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL

D16400: Prefer to set current mode in resolution combobox

volkov closed this revision. REPOSITORY R104 KScreen REVISION DETAIL https://phabricator.kde.org/D16400 To: volkov, #plasma, davidedmundson Cc: davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D16547: Connect slots explicitly instead of using auto-connections

volkov closed this revision. REPOSITORY R121 Policykit (Polkit) KDE Agent REVISION DETAIL https://phabricator.kde.org/D16547 To: volkov, #plasma, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D16547: Connect slots explicitly instead of using auto-connections

volkov updated this revision to Diff 44551. volkov added a comment. rename slot REPOSITORY R121 Policykit (Polkit) KDE Agent CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16547?vs=44550=44551 BRANCH master REVISION DETAIL https://phabricator.kde.org/D16547 AFFECTED FILES

D16547: Connect slots explicitly instead of using auto-connections

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Auto-connections is bad practice cause object renames break code and it can't be detected at compile time. REPOSITORY R121

D16400: Prefer to set current mode in resolution combobox

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Preferred resolution may be lower than current resolution (e.g. on KVM), in this case it will be selected in the combobox instead

D15969: Add support for systemd's suspend-then-hibernate

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R122 Powerdevil BRANCH master REVISION DETAIL https://phabricator.kde.org/D15969 AFFECTED FILES

D15946: Avoid creation of needless temporary containers

volkov closed this revision. REPOSITORY R110 KScreen Library REVISION DETAIL https://phabricator.kde.org/D15946 To: volkov, #plasma, broulik Cc: broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15946: Avoid creation of needless temporary containers

volkov added inline comments. INLINE COMMENTS > broulik wrote in output.cpp:101 > The previous code also compared the contents of both, or does your new loop > below do that? Yes, the check is in the loop below: `after` and `before` are of the same sizes, and if `after` contains each key from

D15946: Avoid creation of needless temporary containers

volkov updated this revision to Diff 42860. volkov added a comment. fixed REPOSITORY R110 KScreen Library CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15946?vs=42859=42860 BRANCH master REVISION DETAIL https://phabricator.kde.org/D15946 AFFECTED FILES

D15946: Avoid creation of needless temporary containers

volkov added inline comments. INLINE COMMENTS > broulik wrote in waylandoutput.cpp:64 > I think we typically use `const...()` instead of `c...()` but since this > method is `const`, shouldn't be neccessary to begin with Isn't it better to follow STL style? cbegin is also popular in KDE

D15946: Avoid creation of needless temporary containers

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Found by clazy. REPOSITORY R110 KScreen Library BRANCH master REVISION DETAIL https://phabricator.kde.org/D15946 AFFECTED

D15297: Use QOverload to select overloaded functions

volkov closed this revision. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D15297 To: volkov, jgrulich Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15297: Use QOverload to select overloaded functions

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R116 Plasma Network Management Applet BRANCH master REVISION DETAIL https://phabricator.kde.org/D15297 AFFECTED FILES

D15289: Use lambda instead of QSignalMapper

volkov closed this revision. REPOSITORY R111 KSysguard Library REVISION DETAIL https://phabricator.kde.org/D15289 To: volkov, #plasma, broulik Cc: broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D15289: Use lambda instead of QSignalMapper

volkov updated this revision to Diff 41045. volkov added a comment. revert change of actionTriggered's signature REPOSITORY R111 KSysguard Library CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15289?vs=41043=41045 BRANCH master REVISION DETAIL

D15289: Use lambda instead of QSignalMapper

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY QSignalMapper is obsolete and the code with lambda is shorter and clearer. REPOSITORY R111 KSysguard Library BRANCH master

D14835: Use NetworkManager::DeviceStatistics instead of Plasma data engine

volkov closed this revision. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D14835 To: volkov, #plasma, jgrulich, sitter, jriddell Cc: broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol,

D14835: Use NetworkManager::DeviceStatistics instead of Plasma data engine

volkov updated this revision to Diff 40632. volkov added a comment. minimize the number of setDeviceStatisticsRefreshRateMs() callings REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14835?vs=39790=40632 BRANCH master REVISION

D14835: Use NetworkManager::DeviceStatistics instead of Plasma data engine

volkov added a comment. Unfortunately KDE Neon is based on Ubuntu 16.04 (Xenial) which is using NM 1.2: https://packages.ubuntu.com/source/xenial/network-manager REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D14835 To: volkov, #plasma,

D14835: Use NetworkManager::DeviceStatistics instead of Plasma data engine

volkov updated this revision to Diff 39790. volkov added a comment. update device statistics only when it's shown REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14835?vs=39698=39790 BRANCH master REVISION DETAIL

D14835: Use NetworkManager::DeviceStatistics instead of Plasma data engine

volkov added inline comments. INLINE COMMENTS > broulik wrote in networkmodel.cpp:220 > Would it be better to only request the data when the statistics are open to > avoid wakeups and processing? Yes, good point. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL

D14835: Use NetworkManager::DeviceStatistics instead of Plasma data engine

volkov created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY It's more correct to get information about devices from a single source, i.e. from NetworkManager. Besides it allows to show the

D14794: Port away from KLocale and KDELibs4Support

volkov closed this revision. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D14794 To: volkov, #plasma, jgrulich Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D14794: Port away from KLocale and KDELibs4Support

volkov updated this revision to Diff 39618. volkov added a comment. QLocale::nativeCountryName() may return an empty string, fallback to using QLocale::countryToString() in this case. REPOSITORY R116 Plasma Network Management Applet CHANGES SINCE LAST UPDATE

D14793: Update connection details only when necessary

volkov closed this revision. REPOSITORY R116 Plasma Network Management Applet REVISION DETAIL https://phabricator.kde.org/D14793 To: volkov, #plasma, jgrulich Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D14794: Port away from KLocale and KDELibs4Support

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REPOSITORY R116 Plasma Network Management Applet BRANCH master REVISION DETAIL

D14793: Update connection details only when necessary

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY They are needed only when the Details tab is visible. In other cases, their update is just eating

D14562: Introduce Theme.smallestFont

volkov added a comment. Why add it to qml-base theme implementations? Aren't they obsolete? REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D14562 To: volkov, #kirigami, mart Cc: mart, plasma-devel, apol, davidedmundson, hein

D14562: Introduce Theme.smallestFont

volkov added inline comments. INLINE COMMENTS > volkov wrote in kirigamiplugin.cpp:173 > So, should it be registered for both versions 2.2 and 2.6? > I tried to use revisions, but it looks like they don't work for properties of > attached property objects: >

D14562: Introduce Theme.smallestFont

volkov added inline comments. INLINE COMMENTS > mart wrote in kirigamiplugin.cpp:173 > this will break completely every import < 2.6 So, should it be registered for both versions 2.2 and 2.6? I tried to use revisions, but it looks like they don't work for properties of attached property

D14562: Introduce Theme.smallestFont

volkov created this revision. Restricted Application added a project: Kirigami. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY It can be useful in itself and when porting code from Plasma. REPOSITORY R169 Kirigami BRANCH

D13193: Rename kcm_input to kcm_mouse

volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D13193 To: volkov, #plasma, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D13193: Rename kcm_input to kcm_mouse

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY This KCM provides settings for mouse only. REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.13

D12760: Improve single-instance application behavior

volkov closed this revision. REPOSITORY R102 KInfoCenter REVISION DETAIL https://phabricator.kde.org/D12760 To: volkov, #plasma, mart Cc: mart, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D12760: Improve single-instance application behavior

volkov added inline comments. INLINE COMMENTS > mart wrote in main.cpp:71 > KWindowSystem::forceActiveWindow should never be used by normal applications, > it's reserved for workspaces and won't work on Wayland. > QWidget::activateWindow should be enough, even if that's not guaranteed to >

D12760: Improve single-instance application behavior

volkov added a comment. The documentation of KDBusService recommends to use it: https://api.kde.org/frameworks/kdbusaddons/html/classKDBusService.html#af0ad38f597aedaf22a30045d0423ed0e And many applications do it: https://lxr.kde.org/search?_filestring=&_string=forceActiveWindow

D12760: Improve single-instance application behavior

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY 1. Avoid showing the window of the second instance by creating KDBusService object before creating

D11744: Revert "Revert "Get rid of KDELibs4Support""

volkov closed this revision. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D11744 To: volkov, davidedmundson, broulik, #plasma Cc: plasma-devel, ragreen, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D11744: Revert "Revert "Get rid of KDELibs4Support""

volkov added inline comments. INLINE COMMENTS > davidedmundson wrote in CMakeLists.txt:26 > I'm confused, the commit description doesn't match the diff What do you mean? It's revert revert... REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D11744 To: volkov,

D4939: Get rid of KDELibs4Support

volkov added a comment. D11744 REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D4939 To: denisshienkov, #plasma, davidedmundson, volkov, afiestas, broulik, graesslin Cc: graesslin, davidedmundson, broulik, plasma-devel,

D11744: Revert "Revert "Get rid of KDELibs4Support""

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY This reverts commit 55603160e9b894270a34e7d01e0aef5bbbfe8025

D11573: Use QPlatformWindow::startSystemMove() with Qt 5.11

volkov added a comment. IIRC, the X server floors coordinates, while Qt rounds them. REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D11573 To: volkov, #breeze Cc: davidedmundson, broulik, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas,

D11573: Use QPlatformWindow::startSystemMove() with Qt 5.11

volkov added a comment. https://codereview.qt-project.org/224251 REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D11573 To: volkov, #breeze Cc: davidedmundson, broulik, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D11573: Use QPlatformWindow::startSystemMove() with Qt 5.11

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY ... instead of X11 API to initiate window movement by a window manager. It works correctly when

D10038: Use QTemporaryDir::filePath() instead of path concatenation

volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10038 To: volkov, #plasma, hein Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10038: Use QTemporaryDir::filePath() instead of path concatenation

volkov added a comment. ping REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10038 To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10038: Use QTemporaryDir::filePath() instead of path concatenation

volkov created this revision. volkov added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY This is the preferred way to get a file path in a temporary

D10037: Remove QT_VERSION checks

volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10037 To: volkov, #plasma, davidedmundson Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10037: Remove QT_VERSION checks

volkov added a reviewer: Plasma. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10037 To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10037: Remove QT_VERSION checks

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY We require Qt 5.9, so these checks are redundant. REPOSITORY R119 Plasma Desktop BRANCH

D10036: Bump Qt version to 5.9

volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10036 To: volkov, #plasma, davidedmundson Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10036: Bump Qt version to 5.9

volkov added a reviewer: Plasma. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10036 To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10036: Bump Qt version to 5.9

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY For Plasma/5.12. REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.12 REVISION DETAIL

D9733: Display exclusive actions correctly in systray menus

volkov closed this revision. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9733 To: volkov, #plasma, davidedmundson Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol,

D9733: Display exclusive actions correctly in systray menus

volkov added a comment. Testing example: F5629785: tray-exclusive-actions.tar.xz REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9733 To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff,

D9733: Display exclusive actions correctly in systray menus

volkov added a reviewer: Plasma. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9733 To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D9733: Display exclusive actions correctly in systray menus

volkov created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY Add exclusive actions to their own action groups so that they could be diplayed as radio buttons by

D9643: Support submenus in context menu for systray icon

volkov closed this revision. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9643 To: volkov, #plasma, graesslin, davidedmundson Cc: broulik, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

D9643: Support submenus in context menu for systray icon

volkov added a comment. Thanks for the real-world example. REPOSITORY R135 Integration for Qt applications in Plasma BRANCH master REVISION DETAIL https://phabricator.kde.org/D9643 To: volkov, #plasma, graesslin, davidedmundson Cc: broulik, davidedmundson, plasma-devel, ZrenBot,

D9625: Require Qt 5.9

volkov closed this revision. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9625 To: volkov, #plasma, graesslin Cc: graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D9643: Support submenus in context menu for systray icon

volkov added a comment. BTW, why do we need to support KDEPlatformSystemTrayIcon? Why don't rely on QDBusTrayIcon? REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9643 To: volkov, #plasma, graesslin Cc: plasma-devel, ZrenBot,

D9643: Support submenus in context menu for systray icon

volkov added reviewers: Plasma, graesslin. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9643 To: volkov, #plasma, graesslin Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D9625: Require Qt 5.9

volkov retitled this revision from "Require Qt 5.7" to "Require Qt 5.9". volkov edited the summary of this revision. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9625 To: volkov, #plasma Cc: graesslin, plasma-devel, ZrenBot,

D9625: Require Qt 5.7

volkov updated this revision to Diff 24659. volkov added a comment. move to master Require Qt 5.7 -> Require Qt 5.9 REPOSITORY R135 Integration for Qt applications in Plasma CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9625?vs=24637=24659 BRANCH master REVISION DETAIL

D9625: Require Qt 5.7

volkov added a reviewer: Plasma. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D9625 To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D7013: Middle-click on QSystemTrayIcon сauses context menu

volkov added a comment. Restricted Application added a subscriber: plasma-devel. Is there a reason why KDEPlatformSystemTrayIcon is supported? Why don't drop it and rely on Qt's dbus tray implementation? REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL

  1   2   >