<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>It aaaalmost looked good, but I got this bug back:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">add fav to current activity (activity 1)</li>
<li class="remarkup-list-item">add to activity 2 by checking in context menu while in activity 1</li>
<li class="remarkup-list-item">switch to activity 2, verify it was added</li>
<li class="remarkup-list-item">switch back to activity 1, in context menu uncheck activity 1</li>
<li class="remarkup-list-item">switch back to activity 2 to verify it's still there - but it's gone</li>
</ol>

<p>I tried to repeat those steps and then at step 3, the favorite that disappeared at step 5 actually did show up. So this might be a case where the model didn't refresh correctly on activity switch? However with the new favorite I was trying it with, it didn't disappear. Then when I tried it again after trying some other things, it happened again, and other favorites disappeared as well alongside it.</p>

<p>Another UX note: At some point during testing I added a new favorite directly to all activities. Then I switched from activity 1 to activity 2, and I saw on activity 2 the favorite wasn't appended to the list but appeared in front of the activity 2-only favorites. This isn't a huge deal breaker, although I feel like it might be simpler and more predictable for users to always just append - I thought the applets now have their own ordering conceptually so I was sort of surprised this didn't happen. But maybe I'm misunderstanding how per-applet ordering and global ordering (new applets need to get ordering from somewhere I guess ...) interact, or should interact. So don't get too scared by this feedback ...</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>Zren, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>