<table><tr><td style="">hein requested changes to this revision.<br />hein added a comment.<br />This revision now requires changes to proceed.
</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/D8816" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I will push it for you when it's done (you need a developer account to push).</p>

<p>But actually I thought about it some more and I realized I need to ask you for some changes in the patch. I think there may be some existing users (i.e. UIs) of the model which rely on all top-level items in RootModel being categories with child models. That means we need to implement this in a backwards-compatible way. Can you implement a 'showTopLevelApps' (or any better name you come up with) property in AppsModel , default it to false and only show the items when it's set to true? Then we can patch the Kicker and Kickoff UIs to set it to true, but leave e.g. Application Dashboard unchanged.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8816" rel="noreferrer">https://phabricator.kde.org/D8816</a></div></div><br /><div><strong>To: </strong>lopatin, Plasma, hein<br /><strong>Cc: </strong>ngraham, hein, mart, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>