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

Reply via email to