- User Since
- Oct 4 2018, 6:27 PM (29 w, 3 h)
And also make it blue like the other search button.
I personally think this could be quite useful.
Just one tip, I'd replace the "Search" button with a search icon, since it looks better and the search placeholder already says "Search...".
Can you please update this patch? It doesn't apply anymore. It would also be nice if you used Arcanist/arc in the future: https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist_to_post_patches
Looks good UI-wise.
Should we really wait until those are merged and not have the new layout in Plasma 5.16? Merging these will take quite some time and I don't see how it would make a difference layout-wise.
Tue, Apr 23
You can now use edit-remove, which got added in D20700.
Fix one 16px path point
(Hopefully) fix icons and add 22px version of edit-none
Mon, Apr 22
Add 22px version of paint-none and add symlink for it
Sun, Apr 21
Use symlinks: edit-remove -> paint-none
Sat, Apr 20
Oops, forgot that paint-none already exists, which looks like this. I will just make a symlink then.
Here you go: https://phabricator.kde.org/D20700
Thu, Apr 18
Wed, Apr 17
@broulik Do you want to commandeer this revision and do the rest?
Address some comments
Move currentApplet.applet.prepareContextualActions(); to onVisualParentChanged
BTW, we should also unify the styles for the lists itself. The list headers and items currently don't all look consistent.
Hmm, it seems like the Plasma theme list-remove is different from the normal Breeze icon theme list-remove. I think we should have the icon from the Plasma theme in the Breeze icon theme too, but with a different name, so we don't overwrite the existing one.
Mon, Apr 15
Now it's great.
Fix "No Preview" icon
Fix initial selection bug and add load indication
Looks ok to me.
Try to fix diff