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=18132#toc

REPOSITORY
  R297 KDED

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D6899?vs=17177=18132

REVISION DETAIL
  https://phabricator.kde.org/D6899

AFFECTED FILES
  src/kded.cpp
  src/kded.h

To: mart, #frameworks, #plasma, davidedmundson
Cc: dfaure, plasma-devel, broulik, #frameworks, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

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, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

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-devel, broulik, #frameworks, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

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 be a new key, would you prefer that version?

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.


  "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, #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-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=17176

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D6899

AFFECTED FILES
  src/kded.cpp
  src/kded.h
  src/kdedmodule.desktop

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 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=17177

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D6899

AFFECTED FILES
  src/kded.cpp
  src/kded.h

To: mart, #frameworks, #plasma
Cc: plasma-devel, broulik, #frameworks, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas