D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-08-14 Thread Marco Martin
This revision was automatically updated to reflect the committed changes. Closed by commit R297:937ade5d7357: support X-KDE-OnlyShowOnQtPlatforms (authored by mart). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D6899?vs=17177&id=18132#toc REPOSITORY R297 KDED CHANGES SINCE LAST UPDAT

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-08-14 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R297 KDED BRANCH master REVISION DETAIL https://phabricator.kde.org/D6899 To: mart, #frameworks, #plasma, davidedmundson Cc: dfaure, plasma-devel, broulik, #frameworks, ZrenBot, progwolff, l

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-08-14 Thread Marco Martin
mart added a comment. ping? REPOSITORY R297 KDED REVISION DETAIL https://phabricator.kde.org/D6899 To: mart, #frameworks, #plasma Cc: dfaure, plasma-devel, broulik, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-08-08 Thread David Faure
dfaure added a comment. Ah, this key is already used, for applications? I wasn't aware, if that's the case. Hmm, then maybe better keep the same key indeed. REPOSITORY R297 KDED REVISION DETAIL https://phabricator.kde.org/D6899 To: mart, #frameworks, #plasma Cc: dfaure, plasma-deve

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-08-08 Thread Marco Martin
mart added a comment. In https://phabricator.kde.org/D6899#133625, @dfaure wrote: > "Show" is a bit of a historical name here, this isn't about showing apps in a menu... > > X-KDE-OnlyLoadOnQtPlatforms would be closer to the truth, no? > > Looks good otherwise. that would

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-08-08 Thread David Faure
dfaure added a comment. "Show" is a bit of a historical name here, this isn't about showing apps in a menu... X-KDE-OnlyLoadOnQtPlatforms would be closer to the truth, no? Looks good otherwise. REPOSITORY R297 KDED REVISION DETAIL https://phabricator.kde.org/D6899 To: mart, #f

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-07-25 Thread Marco Martin
mart updated this revision to Diff 17176. mart added a comment. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. readstringlist REPOSITORY R297 KDED CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6899?vs=17175&id=17176 BRANC

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-07-25 Thread Marco Martin
mart marked 2 inline comments as done. REPOSITORY R297 KDED REVISION DETAIL https://phabricator.kde.org/D6899 To: mart, #frameworks, #plasma Cc: plasma-devel, broulik, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D6899: support X-KDE-OnlyShowOnQtPlatforms

2017-07-25 Thread Marco Martin
mart updated this revision to Diff 17177. mart added a comment. less blanks REPOSITORY R297 KDED CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6899?vs=17176&id=17177 BRANCH master REVISION DETAIL https://phabricator.kde.org/D6899 AFFECTED FILES src/kded.cpp src/kded.h