KDE CI: Plasma plasma-desktop kf5-qt5 FreeBSDQt5.9 - Build # 80 - Still unstable!

2018-01-22 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Plasma%20plasma-desktop%20kf5-qt5%20FreeBSDQt5.9/80/ Project: Plasma plasma-desktop kf5-qt5 FreeBSDQt5.9 Date of build: Tue, 23 Jan 2018 04:14:23 + Build duration: 1 hr 16 min and counting JUnit Tests

D10032: Swap play/pause icons in mediacontroller plasmoid

2018-01-22 Thread Nicolas Fella
nicolasfella created this revision. nicolasfella added reviewers: Plasma, VDG. nicolasfella added a project: Plasma. nicolasfella requested review of this revision. REVISION SUMMARY The play/pause status of the system tray icon appears inverse to the play/pause button in the plasmoid. This

D10031: Fix potential crash

2018-01-22 Thread Aleix Pol Gonzalez
apol created this revision. apol added reviewers: Kirigami, mart. Restricted Application added projects: Plasma, Kirigami. Restricted Application added a subscriber: plasma-devel. apol requested review of this revision. REVISION SUMMARY Turns out in some weird (wrong) cases, ListView sets a

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Andres Betts
abetts accepted this revision. REPOSITORY R134 Discover Software Store BRANCH master REVISION DETAIL https://phabricator.kde.org/D10027 To: anemeth, apol, #discover_software_store, davidedmundson, abetts Cc: ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Aleix Pol Gonzalez
apol added a comment. Please land on the 5.12 branch. REPOSITORY R134 Discover Software Store BRANCH master REVISION DETAIL https://phabricator.kde.org/D10027 To: anemeth, apol, #discover_software_store, davidedmundson Cc: ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai,

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Aleix Pol Gonzalez
apol added a comment. Can you push? REPOSITORY R134 Discover Software Store BRANCH master REVISION DETAIL https://phabricator.kde.org/D10027 To: anemeth, apol, #discover_software_store, davidedmundson Cc: ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Aleix Pol Gonzalez
apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R134 Discover Software Store BRANCH master REVISION DETAIL https://phabricator.kde.org/D10027 To: anemeth, apol, #discover_software_store, davidedmundson Cc: ngraham, plasma-devel, ZrenBot,

D10027: Removed duplicate preview screenshots

2018-01-22 Thread David Edmundson
davidedmundson accepted this revision. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D10027 To: anemeth, apol, #discover_software_store, davidedmundson Cc: ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts,

D9848: Updated the blur method to use the more efficient dual kawase blur algorithm.

2018-01-22 Thread Fredrik Höglund
fredrik accepted this revision. This revision is now accepted and ready to land. Restricted Application edited projects, added Plasma; removed KWin. REPOSITORY R108 KWin BRANCH master REVISION DETAIL https://phabricator.kde.org/D9848 To: anemeth, #plasma, #kwin, fredrik Cc: luebking,

D9848: Updated the blur method to use the more efficient dual kawase blur algorithm.

2018-01-22 Thread Alex Nemeth
anemeth marked an inline comment as done. Restricted Application edited projects, added KWin; removed Plasma. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D9848 To: anemeth, #plasma, #kwin Cc: luebking, broulik, romangg, zzag, anthonyfieroni, mart, davidedmundson,

D9848: Updated the blur method to use the more efficient dual kawase blur algorithm.

2018-01-22 Thread Alex Nemeth
anemeth updated this revision to Diff 25782. anemeth added a comment. Restricted Application edited projects, added Plasma; removed KWin. Implemented the change suggested by @fredrik REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9848?vs=25720=25782 BRANCH

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread René J . V . Bertin
David Edmundson wrote: My previous reply to this seems to have gone missing. > That plugin has it's own system tray handling that uses KNotification, > instead of the platform DBus code you looked at in Qt. > Combined with a getenv check in knotification will be the source of your > issue. I

D9848: Updated the blur method to use the more efficient dual kawase blur algorithm.

2018-01-22 Thread Fredrik Höglund
fredrik added inline comments. Restricted Application edited projects, added KWin; removed Plasma. INLINE COMMENTS > kwinglutils.cpp:1104 > + > +s_renderTargets.reserve(s_renderTargets.size() + targets.size()); > + The reserve() call should be in the else {} clause. REPOSITORY R108 KWin

D9944: [WIP]expose if the tablet mode switch is available

2018-01-22 Thread Martin Flöser
graesslin added inline comments. Restricted Application edited projects, added KWin; removed Plasma. INLINE COMMENTS > input.cpp:1897 > +#endif > +return true; > +} false REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D9944 To: mart, #kwin, #plasma Cc: graesslin,

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Alex Nemeth
anemeth updated this revision to Diff 25776. anemeth added a comment. Checking for duplicate now added to appendIfValid() instead REPOSITORY R134 Discover Software Store CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10027?vs=25774=25776 BRANCH master REVISION DETAIL

D10028: Checking for duplicate now added to appendIfValid() instead

2018-01-22 Thread Alex Nemeth
anemeth created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. anemeth requested review of this revision. REPOSITORY R134 Discover Software Store BRANCH master REVISION DETAIL https://phabricator.kde.org/D10028

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Alex Nemeth
anemeth updated this revision to Diff 25774. anemeth added a comment. Now I see what you mean. :) That makes it a lot smaller. REPOSITORY R134 Discover Software Store CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10027?vs=25773=25774 BRANCH master REVISION DETAIL

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Nathaniel Graham
ngraham added a comment. In https://phabricator.kde.org/D10027#194446, @anemeth wrote: > In https://phabricator.kde.org/D10027#194443, @apol wrote: > > > Please modify appendIfValid to not add duplicates. > > > What do you mean? > In what situation would it add duplicates?

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Alex Nemeth
anemeth added a comment. In https://phabricator.kde.org/D10027#194443, @apol wrote: > Please modify appendIfValid to not add duplicates. What do you mean? In what situation would it add duplicates? This change is to make sure it does not add duplicates. REPOSITORY R134

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Aleix Pol Gonzalez
apol requested changes to this revision. apol added a comment. This revision now requires changes to proceed. Please modify appendIfValid to not add duplicates. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D10027 To: anemeth, apol,

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Alex Nemeth
anemeth edited the summary of this revision. anemeth edited the test plan for this revision. anemeth added reviewers: apol, Discover Software Store. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D10027 To: anemeth, apol, #discover_software_store Cc:

D10027: Removed duplicate preview screenshots

2018-01-22 Thread Alex Nemeth
anemeth created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. anemeth requested review of this revision. REPOSITORY R134 Discover Software Store BRANCH master REVISION DETAIL https://phabricator.kde.org/D10027

D10024: Add supportedSchemes feature

2018-01-22 Thread Aleix Pol Gonzalez
apol added a comment. Makes sense I guess. Sounds like it would be useful to set this through kiosk, would that work? INLINE COMMENTS > kdiroperator.cpp:2650 > +{ > +d->supportedSchemes = schemes; > +rereadDir(); only if d->supportedSchemes != schemes? REPOSITORY R241 KIO

D9965: Provide a tooltip for the main action

2018-01-22 Thread Aleix Pol Gonzalez
This revision was automatically updated to reflect the committed changes. Closed by commit R169:3db732db861b: Provide a tooltip for the main action (authored by apol). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D9965?vs=25655=25771#toc REPOSITORY R169 Kirigami CHANGES SINCE LAST

Re: KDE and Google Summer of Code 2018

2018-01-22 Thread Thomas Pfeiffer
> On 22. Jan 2018, at 00:35, David Edmundson wrote: > > > > On Sun, Jan 21, 2018 at 7:25 PM, Albert Astals Cid > wrote: > El divendres, 19 de gener de 2018, a les 10:42:44 CET, Marco Martin va > escriure: > > On Mon, Jan 15,

D10024: Add supportedSchemes feature

2018-01-22 Thread Milian Wolff
mwolff added a dependent revision: D10025: Use KFileWidget::setSupportedSchemes when available. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D10024 To: mwolff, #plasma, dfaure Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

D10025: Use KFileWidget::setSupportedSchemes when available

2018-01-22 Thread Milian Wolff
mwolff created this revision. mwolff added a reviewer: dfaure. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. mwolff requested review of this revision. REVISION SUMMARY This makes QFileDialog::setSupportedSchemes work for the KDE file

D10024: Add supportedSchemes feature

2018-01-22 Thread Milian Wolff
mwolff created this revision. mwolff added reviewers: Plasma, dfaure. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. mwolff requested review of this revision. REVISION SUMMARY This addds a whitelist of supported URL

D10023: Improve display of long app captions and name/caption padding

2018-01-22 Thread Nathaniel Graham
ngraham closed this revision. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D10023 To: ngraham, apol, #discover_software_store, #vdg Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10023: Improve display of long app captions and name/caption padding

2018-01-22 Thread Aleix Pol Gonzalez
apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R134 Discover Software Store BRANCH Plasma/5.12 REVISION DETAIL https://phabricator.kde.org/D10023 To: ngraham, apol, #discover_software_store, #vdg Cc: plasma-devel, ZrenBot, progwolff, lesliezhai,

D9944: [WIP]expose if the tablet mode switch is available

2018-01-22 Thread Marco Martin
mart updated this revision to Diff 25763. mart added a comment. Restricted Application edited projects, added Plasma; removed KWin. - wire up tabletModeAvailable REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9944?vs=25537=25763 BRANCH

D10023: Improve display of long app captions and name/caption padding

2018-01-22 Thread Aleix Pol Gonzalez
apol requested changes to this revision. apol added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > ApplicationPage.qml:157 > text: appInfo.application.name > +lineHeight: 1.0 > maximumLineCount: 1

D10023: Improve display of long app captions and name/caption padding

2018-01-22 Thread Nathaniel Graham
ngraham edited the test plan for this revision. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D10023 To: ngraham, apol, #discover_software_store, #vdg Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol,

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread David Edmundson
davidedmundson planned changes to this revision. davidedmundson added a comment. Good point. Will revisit. REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D10019 To: davidedmundson, #plasma Cc: anthonyfieroni, sebas, broulik, plasma-devel, ZrenBot,

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread Anthony Fieroni
anthonyfieroni added a comment. Thats what doubt me: https://phabricator.kde.org/source/kdeplasma-addons/browse/master/applets/minimizeall/plugin/minimizeall.cpp;65fc80441388c5f3e70832c40e1d7afea6778e83$63-68 REPOSITORY R114 Plasma Addons REVISION DETAIL

D9965: Provide a tooltip for the main action

2018-01-22 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R169 Kirigami BRANCH master REVISION DETAIL https://phabricator.kde.org/D9965 To: apol, #kirigami, mart Cc: broulik, plasma-devel, apol, davidedmundson, mart, hein

D10021: Improve app icon padding on Application page

2018-01-22 Thread Nathaniel Graham
ngraham closed this revision. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D10021 To: ngraham, apol, andreaska, #discover_software_store, #vdg, andreask Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas,

D10021: Improve app icon padding on Application page

2018-01-22 Thread Andreas Kainz
andreask accepted this revision. REPOSITORY R134 Discover Software Store BRANCH Plasma/5.12 REVISION DETAIL https://phabricator.kde.org/D10021 To: ngraham, apol, andreaska, #discover_software_store, #vdg, andreask Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,

D10021: Improve app icon padding on Application page

2018-01-22 Thread Aleix Pol Gonzalez
apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R134 Discover Software Store BRANCH Plasma/5.12 REVISION DETAIL https://phabricator.kde.org/D10021 To: ngraham, apol, andreaska, #discover_software_store, #vdg Cc: plasma-devel, ZrenBot, progwolff,

D10021: Improve app icon padding on Application page

2018-01-22 Thread Nathaniel Graham
ngraham created this revision. ngraham added reviewers: apol, andreaska, Discover Software Store, VDG. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. ngraham requested review of this revision. REVISION SUMMARY Make the padding to the

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread René J . V . Bertin
David Edmundson wrote: > You missed reading the code that calls this in > src/widgets/util/qsystemtrayicon_x11.cpp which will make an X system tray > if the platform doesn't. I understand now why this doesn't happen in my case. Out of scope here. > That plugin has it's own system tray handling

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread David Edmundson
On Mon, Jan 22, 2018 at 1:14 PM, René J. V. Bertin wrote: > >>It's not plasma-specific > > > > It is. Hence the name... > > Just have a look at what it actually does; > nothing requires a plasma environment > to be present (with the possible exception of the look-and-feel

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread René J . V . Bertin
>>It's not plasma-specific > > It is. Hence the name... Just have a look at what it actually does; nothing requires a plasma environment to be present (with the possible exception of the look-and-feel feature but that's far from crucial). Its goal is to provide theming (visual and

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread Sebastian Kügler
sebas added a comment. I like, getting rid of C++ code is always welcome. Once the patch to libtaskmanager is in and kbroulik's comments addressed, +2 from my side. Thanks! REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D10019 To: davidedmundson,

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread Kai Uwe Broulik
broulik added a comment. Don't you also need to remove the cpp stuff from CMakeLists and what not? REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D10019 To: davidedmundson, #plasma Cc: broulik, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread Kai Uwe Broulik
broulik added a comment. BUG: 377213 REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D10019 To: davidedmundson, #plasma Cc: broulik, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread Kai Uwe Broulik
broulik added a comment. Cool! INLINE COMMENTS > main.qml:57 > +tasksModel.requestToggleMinimized(idx); > +clients.push(tasksModel.makePeristentModelndex(i)); > +} typo REPOSITORY R114 Plasma Addons REVISION DETAIL

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread David Edmundson
davidedmundson updated this revision to Diff 25753. davidedmundson added a comment. Debug-- REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10019?vs=25752=25753 BRANCH master REVISION DETAIL https://phabricator.kde.org/D10019 AFFECTED FILES

D10019: Port minimize all applet from it's own plugin to org.kde.TaskManager

2018-01-22 Thread David Edmundson
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. davidedmundson requested review of this revision. REVISION SUMMARY This reduces the code we have and implicitly

D9506: milti-screen sleep fix

2018-01-22 Thread Sebastian Kügler
sebas added a comment. Yeah, so could you abandon this phabricator review? It makes it harder to sift through reviews that actually are interesting... REPOSITORY R110 KScreen Library REVISION DETAIL https://phabricator.kde.org/D9506 To: McPain, sebas, drosca Cc: zerg, anthonyfieroni,

D9414: Implement OSD to select action when unknown monitors is connected

2018-01-22 Thread Sebastian Kügler
sebas added a comment. In https://phabricator.kde.org/D9414#193341, @dvratil wrote: > Please disregard the noise in this patch, sebas merged master into the dvratil/osd branch so I can't generate a proper diff against the base revision without the noise from master. Sorry for

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread David Edmundson
>It's not plasma-specific It is. Hence the name...and not surprisingly, you having issues when it integrates with Plasma. Use outside is 100% not supported. >I'll be looking into that too today. Please do not waste your time or my time in trying to "fix" issues in an already broken setup.

Minutes Monday Plasma Hangout

2018-01-22 Thread Sebastian Kügler
Meeting minutes and log... -- sebas http://www.kde.org | http://vizZzion.orgPlasma Team Meeting, 22 january 2018 Present: Sho, notmart, d_ed, kbroulik, bshah, romangg, sebas Sho: * [Kicker] Fixed the search result that Enter/Return will run while still in the search field not being

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread René J . V . Bertin
David Edmundson wrote: > You missed reading the code that calls this in > src/widgets/util/qsystemtrayicon_x11.cpp which will make an X system tray > if the platform doesn't. Indeed, but that's curious. I see what you mean, but I followed the flow in a debugger and AFAICT the qpa_sys==NULL

D10016: [KFileTreeView] Manually expand tree when model tells us to

2018-01-22 Thread Kai Uwe Broulik
broulik created this revision. broulik added reviewers: Plasma, dfaure. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. broulik requested review of this revision. REVISION SUMMARY It seems just setting the current index isn't enough to

D10016: [KFileTreeView] Manually expand tree when model tells us to

2018-01-22 Thread Kai Uwe Broulik
broulik edited the summary of this revision. REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D10016 To: broulik, #plasma, dfaure Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

KDE CI: Plasma plasma-desktop kf5-qt5 FreeBSDQt5.9 - Build # 79 - Failure!

2018-01-22 Thread CI System
BUILD FAILURE Build URL https://build.kde.org/job/Plasma%20plasma-desktop%20kf5-qt5%20FreeBSDQt5.9/79/ Project: Plasma plasma-desktop kf5-qt5 FreeBSDQt5.9 Date of build: Mon, 22 Jan 2018 10:36:25 + Build duration: 7 min 57 sec and counting CONSOLE

KDE CI: Plasma libksysguard kf5-qt5 FreeBSDQt5.9 - Build # 20 - Still unstable!

2018-01-22 Thread CI System
BUILD UNSTABLE Build URL https://build.kde.org/job/Plasma%20libksysguard%20kf5-qt5%20FreeBSDQt5.9/20/ Project: Plasma libksysguard kf5-qt5 FreeBSDQt5.9 Date of build: Mon, 22 Jan 2018 10:20:20 + Build duration: 20 min and counting JUnit Tests

D9962: Make sure the search result that Return will run is highlighted

2018-01-22 Thread Eike Hein
This revision was automatically updated to reflect the committed changes. Closed by commit R119:19796bc57165: Make sure the search result that Return will run is highlighted (authored by hein). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE

D9945: Don't close plasmoid when closing a subdialog

2018-01-22 Thread Eike Hein
This revision was automatically updated to reflect the committed changes. Closed by commit R119:fc155e94457d: Dont close plasmoid when closing a subdialog (authored by hein). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D9945?vs=25639=25743#toc REPOSITORY R119 Plasma Desktop CHANGES

D9962: Make sure the search result that Return will run is highlighted

2018-01-22 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.12 REVISION DETAIL https://phabricator.kde.org/D9962 To: hein, #plasma, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread David Edmundson
On Sun, Jan 21, 2018 at 6:06 PM, René J. V. Bertin wrote: > Hi, > > > I think you've only half analysed this. If nothing is really listening on > > DBus it will fall back to legacy X which will work on XFCE. > > Where did I miss this in the code? The

KDE CI: Plasma libksysguard kf5-qt5 SUSEQt5.9 - Build # 32 - Fixed!

2018-01-22 Thread CI System
BUILD SUCCESS Build URL https://build.kde.org/job/Plasma%20libksysguard%20kf5-qt5%20SUSEQt5.9/32/ Project: Plasma libksysguard kf5-qt5 SUSEQt5.9 Date of build: Mon, 22 Jan 2018 10:20:20 + Build duration: 2 min 39 sec and counting JUnit Tests

Re: DBus statusnotifier (aka systray) daemon/bridge for GTk/XFCE environments?

2018-01-22 Thread René J . V . Bertin
René J. V. Bertin wrote: > PS: Ah, when I move the platform integration plugin aside I indeed get a > systray icon. Hmmm... Hmmm indeed. Pushing the plugin aside doesn't take down an already running host of course. When I do that too, the error message comes back that tells me that no systray

D9961: [Plasma Theme] Guard against invalid KPluginInfo object

2018-01-22 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:9870027494b5: [Plasma Theme] Guard against invalid KPluginInfo object (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE

D9961: [Plasma Theme] Guard against invalid KPluginInfo object

2018-01-22 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D9961 To: broulik, #plasma, mart Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

D9945: Don't close plasmoid when closing a subdialog

2018-01-22 Thread Marco Martin
mart accepted this revision. mart added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > kcm_keyboard.desktop:126 > Comment[hu]=Billentyűzetkiosztás > -Comment[id]=Tata Letak dan Peranti Keras Keyboard > +Comment[id]=Tata Letak dan Perangkat Keras Keyboard >

D9974: Improve header and metadata display

2018-01-22 Thread Nathaniel Graham
ngraham edited the summary of this revision. ngraham edited the test plan for this revision. REPOSITORY R134 Discover Software Store BRANCH combobox-to-choose-source REVISION DETAIL https://phabricator.kde.org/D9974 To: ngraham, apol, abetts, colomar, #discover_software_store, #vdg,

Re: Fwd: KDE CI: Plasma ksshaskpass kf5-qt5 SUSEQt5.9 - Build # 11 - Still Failing!

2018-01-22 Thread Ben Cooksley
On Mon, Jan 22, 2018 at 7:30 PM, Martin Koller wrote: > On Sonntag, 21. Jänner 2018 22:40:31 CET Ben Cooksley wrote: >> On Mon, Jan 22, 2018 at 10:15 AM, Martin Koller wrote: >> > Hi, >> >> Hi Martin, >> >> > >> > On Sonntag, 21. Jänner 2018 20:43:42 CET Ben

D9506: milti-screen sleep fix

2018-01-22 Thread Oleg Solovyov
McPain added a comment. UPD: I found monitor combination that makes KScreen works: AOC(hdmi)-iiyama(D-SUB) If I swap monitors, KSCeen doesn't work (noticed it past friday evening) REPOSITORY R110 KScreen Library REVISION DETAIL https://phabricator.kde.org/D9506 To: McPain, sebas,

D9506: milti-screen sleep fix

2018-01-22 Thread Sebastian Kügler
sebas added a comment. Alright, please abandon this patch then, it's definitely wrong. REPOSITORY R110 KScreen Library REVISION DETAIL https://phabricator.kde.org/D9506 To: McPain, sebas, drosca Cc: zerg, anthonyfieroni, ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai,