Re: Online sprint(?!)

On Thu, May 14, 2020 at 6:32 AM Bhushan Shah wrote: > So I am seeing this idea floating around in multiple occasion in Plasma > IRC channel and as well on other places, but nothing concrete came up > out of this discussion so I thought I should start formal discussion, > > Due to COVID-19 we had

D29709: Give all OverlaySheets consistent headers

apol added a comment. +1 for the change btw, thanks! REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D29709 To: ngraham, apol, #discover_software_store, #vdg Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus,

D29709: Give all OverlaySheets consistent headers

apol added a comment. We are in string freeze right now, aren't we? INLINE COMMENTS > ApplicationPage.qml:282 > enabled: appInfo.application.isInstalled > -visible: reviewsModel.backend && > reviewsModel.backend.isResourceSupported(appInfo.application) > +//

D29632: [kcms/keys] Use better collator settings

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. It could make sense to leverage QCollatorSortKey in this class if it's sorting often. REPOSITORY R119 Plasma Desktop BRANCH collator (branched from master) REVISION DETAIL

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 82302. apol added a comment. Adapt to changes in the protocol, implement windows REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=81926=82302 BRANCH arcpatch-D28884 REVISION

D29476: WIP: Port klipper to use wayland clipboard

apol added a comment. +1 INLINE COMMENTS > systemclipboard.cpp:63 > + > +class DataControlDeviceManager : public > QWaylandClientExtensionTemplate > +, public QtWayland::zwlr_data_control_manager_v1 Maybe split the DataControl classes into a separate file? > systemclipboard.h:45 >

D29476: WIP: Port klipper to use wayland clipboard

apol added a comment. In D29476#664751 , @nicolasfella wrote: > Isn't this something that should be in QClipboard in an ideal world? Or is that currently not feasible since the protocol is non-standard? It's more that only few processes

D29437: Use font: instead of font.pointSize: where possible

apol added a comment. You can do that now but this behaviour is deprecated and disallowed in Qt 6 AFAIK. I don't know a better way of doing this, so +0 from me. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D29437 To: ngraham, cblack, #plasma Cc:

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 81926. apol added a comment. Rebase on top of kwayland-server changes REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=81343=81926 BRANCH arcpatch-D28884 REVISION DETAIL

Re: repo freeze tomorrow

yland-protocols > > Jonathan > > > On Fri, 1 May 2020 at 02:13, Aleix Pol wrote: >> >> On Thu, Apr 30, 2020 at 7:57 PM Rik Mills wrote: >> > >> > On 30/04/2020 18:33, Aleix Pol wrote: >> > > Yes, in practice. >> > >> > In

D29344: [applets/systemtray] Fix invisible plasmoid icons when Media Player is added

apol added inline comments. INLINE COMMENTS > main.qml:92 > +function onExtraItemsChanged() { > +plasmoid.nativeInterface.allowedPlasmoids = > plasmoid.configuration.extraItems > } Why's this needed? REPOSITORY R120 Plasma Workspace REVISION DETAIL

Re: Information regarding upcoming Gitlab Migration: clarifications

On Fri, May 1, 2020 at 7:18 AM Ben Cooksley wrote: > > On Fri, May 1, 2020 at 4:38 PM Nate Graham wrote: > > > > > > > > On 4/30/20 5:59 PM, Aleix Pol wrote: > > > El jue., 30 de abr. de 2020 a la(s) 18:15, Albert Astals Cid > > >> Am I the

D29150: Address Connections syntax warnings on the systray plasmoid

This revision was automatically updated to reflect the committed changes. Closed by commit R120:4c767ab70005: Address Connections syntax warnings on the systray plasmoid (authored by apol). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE

Re: repo freeze tomorrow

On Thu, Apr 30, 2020 at 7:57 PM Rik Mills wrote: > > On 30/04/2020 18:33, Aleix Pol wrote: > > Yes, in practice. > > In practice? What other way is there? Now I check some more they seem > very non-optional. I mean that plasma-wayland-protocols is supposed to be relea

Re: Information regarding upcoming Gitlab Migration: clarifications

On Fri, May 1, 2020 at 1:08 AM Nicolás Alvarez wrote: > > El jue., 30 de abr. de 2020 a la(s) 18:15, Albert Astals Cid > (aa...@kde.org) escribió: > > > > El dijous, 30 d’abril de 2020, a les 21:31:02 CEST, Ben Cooksley va > > escriure: > > > On Fri, May 1, 2020 at 6:04 AM Ivan Čukić wrote: > >

2 new repositories: plasma-wayland-protocols and kwayland-server

Hi everyone, After long discussions, we have broken kwayland into 3 pieces so we can work and improve kwin_wayland without having to extend our stable framework constantly. This of course comes together with an updated policy with regards to where things should be developed with the goal of only

Re: Information regarding upcoming Gitlab Migration: clarifications

On Wed, Apr 29, 2020 at 12:25 PM Bhushan Shah wrote: > > Good afternoon, > > [Please keep sysad...@kde.org list or bs...@kde.org in the CC for > replies] > > I want to clarify some bits for which we have gotten a questions about, > > - Non unique naming: There's some teams which prefer if we

Re: repo freeze tomorrow

Yes, in practice. Aleix On Thu, Apr 30, 2020 at 6:19 PM Rik Mills wrote: > > On 29/04/2020 22:04, David Edmundson wrote: > > We have something announced that I'm not sure is on the list. > > > > Plasma will gain a new repo called kwayland-server. > > This is a split of half the code currently

Re: repo freeze tomorrow

Are there any specific steps needed? I tried to make it so the cmake variables on the /CMakeLists.txt are named like in Plasma. We will also need plasma-wayland-protocols released as kdesupport soon. Right now I'm trying to make sure all is well on the CI side. Aleix On Wed, Apr 29, 2020 at

D27960: Make it possible to scroll through the padding

apol added a comment. @ngraham can you test again? it works well for me (admittedly better than when I first submitted it). Maybe something else got fixed too in the meantime. REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D27960 To: apol, #kirigami, mart,

D27960: Make it possible to scroll through the padding

apol updated this revision to Diff 81481. apol added a comment. Rebase and improve test REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27960?vs=77323=81481 BRANCH arcpatch-D27960 REVISION DETAIL https://phabricator.kde.org/D27960 AFFECTED FILES

D29150: Address Connections syntax warnings on the systray plasmoid

apol added a comment. Ping? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D29150 To: apol, #plasma Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed,

D29071: FileChooser portal: add support for opening directories

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Looks good to me. REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop BRANCH filechooser-directory REVISION DETAIL https://phabricator.kde.org/D29071 To: jgrulich, apol,

D29224: [Task Manager] Avoid crash with QStringBuilder

apol added a comment. Good catch :) REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D29224 To: broulik, #plasma, fvogt, anthonyfieroni, davidedmundson Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen,

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 81343. apol added a comment. Explicitly request zkde_screencast_unstable_v1 REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=81115=81343 BRANCH master REVISION DETAIL

Re: Information regarding upcoming Gitlab Migration

On Mon, Apr 27, 2020 at 12:55 PM Ben Cooksley wrote: > > On Mon, Apr 27, 2020 at 10:39 PM Aleix Pol wrote: > > > > On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wrote: > > > > > > [Please keep sysad...@kde.org list or bs...@kde.org in the CC for > > >

Re: Information regarding upcoming Gitlab Migration

On Mon, Apr 27, 2020 at 3:41 AM Bhushan Shah wrote: > > [Please keep sysad...@kde.org list or bs...@kde.org in the CC for > replies] > > Hello Community members, > > In view of upcoming Gitlab migration, we sysadmin team wants to share > the recommended structuring for the repositories on Gitlab.

Re: Type=FSDevice desktop files

On Sun, Apr 26, 2020 at 11:45 PM David Faure wrote: > > Would it be OK if we dropped support for desktop files that represent devices? > This was a KDE1/KDE2/KDE3? feature, you'd write/ship a desktop file with > Type=FSDevice and Dev=/dev/sdc > and this would give you Mount and Unmount in the

D29202: Remove unused QtConcurrent from colorutils

apol added a comment. Then remove it from CMakeLists.txt as well? REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D29202 To: patrickelectric, apol, mart Cc: plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, ahiemstra, davidedmundson, mart

D29154: Use QWindow overload of QIcon::pixmap

apol added a comment. Yeah, I guess that phabricator scaled the pictures to take less space. My impression was that the patch improved things. REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D29154 To: davidre, apol, broulik, ndavis, #breeze, davidedmundson,

D29154: Use QWindow overload of QIcon::pixmap

apol accepted this revision. apol added a comment. This is what it looks like for me with the patch applied. scale 1x: F8258193: Screenshot_20200424_203628.png scale 2x: F8258201: Screenshot_20200424_203729.png

D29154: Use QWindow overload of QIcon::pixmap

apol added a comment. In D29154#656516 , @cblack wrote: > Icon handling looks somewhat funky with this patch applied when a window changes DPI. > F8257898: ksnip_20200424-111720.png Right, should

D29154: Use QWindow overload of QIcon::pixmap

apol added a comment. +1 I did the same thing for kirigami @ D29100 and plasma-framework @ D29102 yesterday after looking at Qt's code and assessing it was the right thing to do. I am not familiar with Breeze's

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 81115. apol added a comment. Adapt to the changes in hte kwayland patch REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80985=81115 BRANCH master REVISION DETAIL

D29150: Address Connections syntax warnings on the systray plasmoid

apol created this revision. apol added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. TEST PLAN Using it REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80985. apol added a comment. Adopt kwayland changes, provide the right buffer size. REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80797=80985 BRANCH master REVISION DETAIL

D29100: Icon: improve icon rendering on multi-screen multi-dpi setups

This revision was automatically updated to reflect the committed changes. Closed by commit R169:6b76ce30811c: Icon: improve icon rendering on multi-screen multi-dpi setups (authored by apol). REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDATE

D28711: WIP: OpenStore Integration

apol added a comment. Hey, I went through the review, the patch looks good. There's some fairly important bits missing: - un/installing - running - state of resources. Currently it searches only on the server but doesn't check if it's installed locally. - detecting if the

D29100: Icon: improve icon rendering on multi-screen multi-dpi setups

apol created this revision. Herald added a project: Kirigami. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Prefer the usage of QIcon::pixmap(QWindow*...) overload that will take into account which window it's rendering to. Otherwise it was

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80797. apol added a comment. Remember to run the event loop REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80795=80797 BRANCH master REVISION DETAIL

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80795. apol added a comment. Remove conditional compilation to pipewire It's not in this module, we don't need to depend on it. Also removes code regarding EGL initialization that I think now is not necessary. REPOSITORY R838 Flatpak Support: KDE

D29033: Remove duplicated code

apol abandoned this revision. apol added a comment. My bad, thanks! REPOSITORY R130 Frameworks integration plugin using KWayland REVISION DETAIL https://phabricator.kde.org/D29033 To: apol, #frameworks, zzag Cc: zzag, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh,

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80719. apol added a comment. Proper initialisation REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80663=80719 BRANCH master REVISION DETAIL

D29033: Remove duplicated code

apol created this revision. apol added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Went through it while reviewing some code. TEST PLAN Been running it for a while, including direct

D29028: feat(wayland): add Wrapland plugin

apol requested changes to this revision. apol added a comment. I don't really see why we'd want to support something that is not offering ABI stability and doesn't push Plasma in any direction. REPOSITORY R110 KScreen Library REVISION DETAIL https://phabricator.kde.org/D29028 To:

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80663. apol added a comment. Adadpt to changes in kwayland D28882 REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80408=80663 BRANCH

D28978: [PluginLoader] Replace one usage of QRegExp with QString::startsWith()

apol added a comment. I wonder, while this will work with all our uses of DropUrlPatterns, this would have been able to work with something like `scheme:/*potato*` which after this patch won't work. In fact, it's called Patterns, not DropUrlWhenStartsWith. I'd appreciate feedback from

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80408. apol added a comment. Renamed classes REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80340=80408 BRANCH master REVISION DETAIL https://phabricator.kde.org/D28884

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80340. apol added a comment. Include the notification REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80285=80340 BRANCH master REVISION DETAIL

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol updated this revision to Diff 80285. apol added a comment. Fix build REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28884?vs=80280=80285 BRANCH master REVISION DETAIL https://phabricator.kde.org/D28884

D28884: screencasting: Adoption of the org_kde_plasma_video_streaming protocol

apol created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Instead of creating the pipewire feeds in the portal, they will be created in KWin. Then the portal task becomes to orchestrate the

D28853: krunner: Prevent regression

apol created this revision. apol added a reviewer: Plasma. apol added a project: Plasma. apol requested review of this revision. REVISION SUMMARY It was pointed out to me that using NoDisplay=true breaks starting krunner. Alternatively D24956 could be

D28852: Yep

apol created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D28852 AFFECTED FILES krunner/CMakeLists.txt

D28821: [Notifier] Port from KRun::runCommand

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Thanks! REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D28821 To: broulik, #plasma, apol Cc: apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev,

D28783: krunner: polish placement

This revision was automatically updated to reflect the committed changes. Closed by commit R120:c963fe873f7c: krunner: polish placement (authored by apol). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28783?vs=80036=80091 REVISION DETAIL

Re: Update on Status of Gitlab Migration

On Mon, Apr 13, 2020 at 8:30 PM Ben Cooksley wrote: > > On Tue, Apr 14, 2020 at 3:35 AM Nate Graham wrote: > > > > On 4/13/20 4:44 AM, Albert Vaca Cintora wrote: > > > Regarding this: is the subdomain going to stay invent.kde.org once we > > > have officially moved? I find it's a bit confusing

D28783: krunner: polish placement

apol updated this revision to Diff 80036. apol added a comment. DOn't set it to Dock type, all sort of bad things happen REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28783?vs=79966=80036 BRANCH arcpatch-D28783 REVISION DETAIL

D28724: Fix build

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. I don't really see why this would be necessary, but it's okay to add it. REPOSITORY R135 Integration for Qt applications in Plasma BRANCH fix-build (branched from master) REVISION DETAIL

D28627: Adaptive panel opacity

apol added a comment. In D28627#646898 , @ngraham wrote: > +1 for making this a user-controllable setting, but how does it interact with plasma themes where there isn't any transparency? Can we only show this UI for plasma themes with

D28783: krunner: polish placement

apol added a subscriber: davidedmundson. apol added a comment. This looks like a regression from e2d372d3dfd14ea3210a9e1f5755883bd2578c18 , so I'll wait for @davidedmundson's comment. REPOSITORY R120 Plasma

D28783: krunner: polish placement

apol created this revision. apol added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Tell the parent Dialog class what the location is so it can configure the dialog accordingly.

D28677: Make PipeWire, GBM and Epoxy libs as required dependencies

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. CMake bits look good. It's a bit of a shame that this whole complexity needs to be added, but oh well. REPOSITORY R838 Flatpak Support: KDE Portal for XDG Desktop BRANCH

D28113: Fix initial loading of autostart program names

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Hi, sorry for the delay, this review skipped my attention. The patch looks good to me. REPOSITORY R119 Plasma Desktop BRANCH names (branched from master) REVISION DETAIL

Re: Making PipeWire required for xdp-kde

On Tue, Apr 7, 2020 at 1:27 PM Jan Grulich wrote: > > Hi, > > I recently pushed a background portal implementation, which also needs some > KWayland stuff, while previously it was needed just for screen sharing purpose > and compiled only when PipeWire and some other libraries are found. Now I

D28214: Add background portal

apol accepted this revision. apol added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > jgrulich wrote in background.cpp:194 > Doesn't need to? I think it just specifies that the desktop file should > ignore the exec line and start it through DBus (assuming

D28214: Add background portal

apol added inline comments. INLINE COMMENTS > background.cpp:69 > +qCDebug(XdgDesktopPortalKdeBackground) << "GetAppState called: no > parameters"; > +return m_appStates; > +} I don't really understand why we're exposing this. Is it for contained apps? > background.cpp:133 > + > +

D28066: Remove the STATIC_LIBRARY option to fix static builds

This revision was automatically updated to reflect the committed changes. Closed by commit R169:989cd3098ced: Remove the STATIC_LIBRARY option to fix static builds (authored by bartoc, committed by apol). REPOSITORY R169 Kirigami CHANGES SINCE LAST UPDATE

D28539: [Emoji panel] search search when the user begins typing

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Fonts are slow... REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D28539 To: ngraham, apol, #plasma, #vdg Cc: plasma-devel, Orage, LeGast00n,

D27980: make opengl module support multiple dri devices

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. LGTM REPOSITORY R102 KInfoCenter BRANCH multidri REVISION DETAIL https://phabricator.kde.org/D27980 To: sitter, apol Cc: pino, zzag, apol, adridg, plasma-devel, Orage, LeGast00n,

D28519: [KMenuEdit] Port QRegExp to QRegularExpression

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. It could make sense for you to test that the names are properly parsed besides just making sure it starts. REPOSITORY R103 KMenu Editor BRANCH l-QRE (branched from master) REVISION

D28517: Send notification when screencasting fails

apol added inline comments. INLINE COMMENTS > ngraham wrote in screencaststream.cpp:385 > Are there more user-friendly and actionable strings we could for these error > messages? I have no idea what this means specifically and would read it as > "there was some gobbeldygook error." While I

D27980: make opengl module support multiple dri devices

apol added inline comments. INLINE COMMENTS > opengl.cpp:281 > +if (ReadPipe("sysctl -n hw.dri.0.name", pci_info)) { > +DriInfo info{}; > +info.module = pci_info[0].mid(0, pci_info[0].indexOf(0x20)); the {} isn't necessary here. > opengl.cpp:290 > > -static bool

D27980: make opengl module support multiple dri devices

apol added inline comments. INLINE COMMENTS > sitter wrote in opengl.cpp:281 > As it turns out the /proc/dri stuff was supposedly removed here > https://github.com/torvalds/linux/commit/cb6458f97b53d7f73043206c18014b3ca63ac345 > making this code entirely irrelevant since kernel 3.12. > Any

D28491: Properly store the powerdown action

This revision was automatically updated to reflect the committed changes. Closed by commit R122:a9abb0cb7924: Properly store the powerdown action (authored by apol). REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28491?vs=79077=79121 REVISION DETAIL

D28492: Improve powerdevil's main

This revision was automatically updated to reflect the committed changes. Closed by commit R122:41ef07d6d3c1: Improve powerdevils main (authored by apol). REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28492?vs=79078=79079 REVISION DETAIL

D28492: Improve powerdevil's main

apol created this revision. apol added reviewers: Plasma, broulik. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Make it possible to restart the daemon for easier debugging. Use Qt::AA_DisableSessionManager

D28491: Properly store the powerdown action

apol created this revision. apol added reviewers: Plasma: Mobile, bshah, broulik. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY The event needs to propagate correctly. TEST PLAN Played with it on my laptop

D28481: Revert "Fix build with Qt 5.15"

This revision was automatically updated to reflect the committed changes. Closed by commit R122:9dbae3f1c3d6: Revert Fix build with Qt 5.15 (authored by apol). REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28481?vs=79043=79050 REVISION DETAIL

D28482: Revert "Fix build with Qt 5.15"

apol abandoned this revision. apol added a comment. Once is enough REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D28482 To: apol Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham,

D28481: Revert "Fix build with Qt 5.15"

apol added a comment. commit 46ebd11e66f6f3180043a7258b46a1aff79a69f8 Author: Lars Knoll Date: Wed Mar 25 11:20:28 2020 +0100 Fix deprecation of QComboBox:::currentIndexChanged(const QString&) Don't introduce another overload with two parameters. Users want a simple

D28481: Revert "Fix build with Qt 5.15"

apol updated this revision to Diff 79043. apol added a comment. remove unrelated changes REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28481?vs=79041=79043 BRANCH arcpatch-D28481 REVISION DETAIL https://phabricator.kde.org/D28481 AFFECTED FILES

D28482: Revert "Fix build with Qt 5.15"

apol created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY This reverts commit 3a5aca82527ec45d22d09e220ff3ff00a42971d2

D28481: Revert "Fix build with Qt 5.15"

apol created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY This reverts commit 3a5aca82527ec45d22d09e220ff3ff00a42971d2

D28398: Use Qt::SplitBehavior enum instead of QString::SplitBehavior

apol added a comment. +1 If plasma pass is part of plasma, which I'm not sure of, you can consider just bumping the required Qt version to Qt 5.14, as it will be the minimum Qt for Plasma 5.19. REPOSITORY R1028 Plasma Pass REVISION DETAIL https://phabricator.kde.org/D28398 To:

D28389: Add support for logging categories

apol added a comment. Other than that, the patch looks good. +1 INLINE COMMENTS > romangg wrote in CMakeLists.txt:23 > Any idea how to reuse the declaration from the top-directory CMake file? Make sure the include() call is before the add_subdirectory(). That's why we usually put them

D28385: Remove unused file

This revision was automatically updated to reflect the committed changes. Closed by commit R120:184cee85e9f5: Remove unused file (authored by apol). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28385?vs=78753=78802 REVISION DETAIL

D28385: Remove unused file

apol updated this revision to Diff 78753. apol added a comment. was only meant to include this REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28385?vs=78752=78753 BRANCH master REVISION DETAIL https://phabricator.kde.org/D28385 AFFECTED

D28385: Remove unused file

apol created this revision. apol added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Still included dcop, hadn't been compiled in years I presume. TEST PLAN All good and normal REPOSITORY

D28368: [krunner] Avoid (a third!) pointless config parse

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Sounds like you can merge this with the other similar one, right? REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D28368 To: davidedmundson,

D28365: [krunner] Avoid (another) pointless config load on start

apol accepted this revision. apol added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > view.cpp:110 > // Catch both, direct changes to the config file ... > -connect(KDirWatch::self(), ::dirty, this, ::reloadConfig); > -connect(KDirWatch::self(),

D27914: [Kilpper] Port QRegExp to QRegularExpression

apol accepted this revision. apol added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > ahmadsamir wrote in urlgrabber.h:185 > The code stores the QRegularExpression::capturedTexts(), > m_regexCapturedTexts, which is eventually what ClipCommandProcess uses.

D27914: [Kilpper] Port QRegExp to QRegularExpression

apol added inline comments. INLINE COMMENTS > ahmadsamir wrote in urlgrabber.h:185 > Another point (which I forgot since it has been some time since I last looked > at this patch), QRegExp stores the match info in itself, whereas > QRegularExpression stores the match info in a

D27914: [Kilpper] Port QRegExp to QRegularExpression

apol added inline comments. INLINE COMMENTS > ahmadsamir wrote in urlgrabber.h:185 > I find it confusing to call the pattern "regExp" and the QRegExp object > "m_myRegExp". Then call it m_regularExpression? REPOSITORY R120 Plasma Workspace REVISION DETAIL

D27914: [Kilpper] Port QRegExp to QRegularExpression

apol added inline comments. INLINE COMMENTS > urlgrabber.h:185 > private: > - QRegExp m_myRegExp; > + QString m_regexPattern; > + QStringList m_regexCapturedTexts; Why did you change how it works right here? REPOSITORY R120 Plasma Workspace REVISION DETAIL

D28339: [kapplymousetheme] Avoid loading QPT

apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Good catch! REPOSITORY R119 Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D28339 To: davidedmundson, #plasma, apol Cc: apol, plasma-devel, Orage, LeGast00n,

D27203: Don't try to open files we can't figure out where they are

apol added a comment. Ping? REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D27203 To: apol, #frameworks Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham,

D28129: Read the new message string after rather than before

This revision was automatically updated to reflect the committed changes. Closed by commit R871:cfa43ed278c8: Read the new message string after rather than before (authored by apol). REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28129?vs=78627=78628

D28129: Read the new message string after rather than before

apol updated this revision to Diff 78627. apol added a comment. address weird +2 in the code REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28129?vs=78605=78627 BRANCH arcpatch-D28129 REVISION DETAIL https://phabricator.kde.org/D28129 AFFECTED FILES

D28129: Read the new message string after rather than before

apol updated this revision to Diff 78605. apol added a comment. Don't leak REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28129?vs=78604=78605 BRANCH arcpatch-D28129 REVISION DETAIL https://phabricator.kde.org/D28129 AFFECTED FILES

D28129: Read the new message string after rather than before

apol updated this revision to Diff 78604. apol marked 3 inline comments as done. apol added a comment. Rebase on top of ksyntaxhighlighting, works great REPOSITORY R871 DrKonqi CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28129?vs=78117=78604 BRANCH arcpatch-D28129 REVISION

<    1   2   3   4   5   6   7   8   9   10   >