<table><tr><td style="">wsdfhjxc added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11352">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>That seems like you are getting some javascript error in <tt style="background: #ebebeb; font-size: 13px;">updateItemVisibility</tt>. Can look at what plasma shell is reporting (or paste it here)?</p></blockquote>

<p>There are no other errors except of <tt style="background: #ebebeb; font-size: 13px;">ConfigEntries.qml:228:34: Unable to assign [undefined] to QKeySequence</tt> spam which is unrelated. Not sure what I did wrong.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>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.</p></blockquote>

<p>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.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>If we conclude that sort based on categories + text is no sufficient I would strongly prefer using <a href="https://phabricator.kde.org/D11292" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11292</a> possibly with a larger default config rather hardcoding half random long list into <tt style="background: #ebebeb; font-size: 13px;">main.qml</tt>. (Long list in default config is fine by me because user can modify that to their wishes.)</p></blockquote>

<p>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 <tt style="background: #ebebeb; font-size: 13px;">UnknownCategory</tt> 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.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11352">https://phabricator.kde.org/D11352</a></div></div><br /><div><strong>To: </strong>Pitel<br /><strong>Cc: </strong>wsdfhjxc, mart, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>