| ngraham |
| Plasma | |
| VDG |
When the applet is a page with a plasmoidHeading as header, merge it with systray one.
The only big problem is that it looks bad when you open said applet while in sidebar view, see:
I'd like to solve that by removing the sidebar and adding a back button, but that would be for another patch in the future.
To be tested together with D28467
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
I'd like to solve that by removing the sidebar and adding a back button, but that would be for another patch in the future.
-1 from me to remove the sidebar. I really liked how the sidebar looked in the final mockup in T10470 and it is consistent with the Kirigami collapsible sidebar drawer (only the color is wrong)
I like the idea of that back button in the sidebar.
@manueljlin how would the last mockup work? (back button and no sidebar)
The last one doesn't have a sidebar and goes back to the list after pressing the back button. It looks a bit cleaner but UX wise it's a bit worse too, having to click two times to do what currently only requires one. I prefer the first and second design but since that one was discussed in the VDG chat I didn't want to leave it there
We could still have a sidebar that goes all the way to the top of the pop-up and also a grid of items on the main page rather than a flat list. The grid items could even animate to their list positions in sidebar form! How exciting.
Anyway, material for another patch, for sure.
