<table><tr><td style="">hein 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/D3805" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Second review pass (now based on updated ivan/new-favourites-per-activity branch rather than this code):</p>

<p>‎[03:04] ‎<‎Sho_‎>‎ ivan|home: compiler warnings on gcc btw @ fav branch<br />
‎[03:19] ‎<‎Sho_‎>‎ ivan|home: I'm still seeing lots of bugs on the branch, e.g. something like this: Switch to my second activity, right-click a Kicker fav and set it from all activities to current activity, switch to first activity - the modified launcher has disappeard, but a launcher that was previously specific to the second activity is now on the first one too (but i can't open its context menu, nothing happens). then when i remove a different launcher and switch to the <br />
‎[03:19] ‎<‎Sho_‎>‎ second activity, the borked launcher (the one that shouldn't be on the first activity but now is) shows up twice there<br />
‎[03:19] ‎<‎Sho_‎>‎ ivan|home: i'm wondering if it would make the code easier if you used a stacking approach like i do in libtaskmanager - have all favorites in a first source model, then a filter model on top of it to filter out by activity<br />
‎[03:20] ‎<‎Sho_‎>‎ in my experience a good way to make models less hairy is to separate concerns by layering them, because then the individual 'passes' are simpler to reason about and individually testable<br />
‎[03:21] ‎<‎Sho_‎>‎ the onion layer approach has worked out really well in libtm (aside from some side-effects layer-punching bugs i introduced dumbly that had to be shaken out)</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3805" rel="noreferrer">https://phabricator.kde.org/D3805</a></div></div><br /><div><strong>To: </strong>ivan, mart, hein<br /><strong>Cc: </strong>plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>