This cleans up and simplifies the tabbox code, mainly by replacing manual anchoring by QtQuick Layouts.
Spacing and fonts are also streamlined.
Details
- Reviewers
- None
- Group Reviewers
Plasma: Design Plasma - Commits
- R120:49d0e94839f1: [Window Switcher] Cleanup
R871:49d0e94839f1: [Window Switcher] Cleanup
I was thinking about widening the tabbox to fit Plasma's sidebars (widget explorer / activity manager) but then the window thumbnails became so large I could only see three windows at a time.
Diff Detail
- Repository
- R120 Plasma Workspace
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Why is the font for the window titles so large? I don't really see much benefit from that...
It was to match the icon and task manager tooltips but I can also just use regular Plasma font size.
+1 code looks good,
using headings as window titles looks good to me, but ok on whatever vdg decides
I've just asked the other VDG members for their take on the font size.
Since this is not a big issue either way, the patch is fine to go in anyway.
If the VDG decides we'd like the font to be smaller, it would be nice if it could be changed in another patch.
LibreOffice change there background for the document preview from white to gray cause show screenshots from a white paper on a white background, ... the screenshots lose the focus. So is it possible to use the dark background from the dash as background for the window switcher. The screenshots would be more visible (I'm not sure how it would look in breeze-dark)
thanks for the patch kai
I'd rather not change the fundamental visuals of this thing now.
If we were to add a dark gray background to the window thumbnails (which I find a good idea, btw) we should do that consistently everywhere (eg. also in task manager tooltips).
As discussed with Thomas I'll push it now as is and once VDG makes up their mind on the fonts (preferably before Thursday noon...) we can still change it.