D11352: [RFC] Auto ordered systray

Wojciech Stachurski noreply at phabricator.kde.org
Fri Mar 16 09:59:31 UTC 2018


wsdfhjxc added a comment.


  > That seems like you are getting some javascript error in `updateItemVisibility`. Can look at what plasma shell is reporting (or paste it here)?
  
  There are no other errors except of `ConfigEntries.qml:228:34: Unable to assign [undefined] to QKeySequence` spam which is unrelated. Not sure what I did wrong.
  
  > I do not like the idea of large per applet order list because it will always be incomplete and give ugly results if user chooses to replace e.g. usual battery applet with some custom one.
  
  Yes, that would have worked only if the user didn't replace embedded items with some custom ones for the same indication purpose. So, you are right, it's probably a no-go.
  
  > If we conclude that sort based on categories + text is no sufficient I would strongly prefer using D11292 <https://phabricator.kde.org/D11292> possibly with a larger default config rather hardcoding half random long list into `main.qml`. (Long list in default config is fine by me because user can modify that to their wishes.)
  
  Judging by previous responses, manually arrangeable system tray is not going to be a thing. Although, having it consistent is a nice improvement. What bothers me is those groups being vague and also that some default items (such as Networks, Touchpad, Updates) are assigned to `UnknownCategory` instead of one of specialized ones. Yes, this makes the system tray consistent, but the ordering gives a messy result where perceptually unrelated items might be placed between related ones. Still, better than random first-in-first-shown behavior.

REPOSITORY
  R120 Plasma Workspace

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

To: Pitel
Cc: wsdfhjxc, mart, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180316/6e7f875a/attachment-0001.html>


More information about the Plasma-devel mailing list