System tray popups are currently resizeable through means of an obscure method (ALT + right click + drag) and the resized values are not remembered on plasmashell restart.
This patch attempts to implement a sidebar mode, which is just the popup expanded to maximum available height.
WIP because it doesn't work well for when the panel is on the bottom or right (see vid).
When it works (left and top panel orientation):
When it doesn't work (right and bottom panel orientation):
| No Linters Available |
| No Unit Test Coverage |
| Buildable 17589 | |
| Build 17607: arc lint + arc unit |
| applets/systemtray/package/contents/ui/ExpandedRepresentation.qml | ||
|---|---|---|
| 69 | Now that you added a RowLayout, this could go inside it rather than being positioned with anchors | |
| applets/systemtray/package/contents/ui/ExpandedRepresentation.qml | ||
|---|---|---|
| 69 | It seems to me like this whole file can be rewritten with layouts (as opposed to using anchors). The problem I'm having is that even a minimal touch (putting the heading in the row layout) breaks stuff. So I've set out to just rewrite the whole thing. So far it's not great, but not awful either. | |
This is more a comment on the technical side.
It seems quite a brittle implementation to me, and quite prone to breakage (anything that relies on PlasmaCore.Dialog trying to make with it things that weren't in its architecture is bound to break, and that class in general is not much touchable.
also, assumes that the systray will be positioned in a panelon a screen edge, and since this is not enforced, it should not be assumed.
Anything that wants to do such a sidebar.. that's really a panel (different window flags, different many things, but still is a containment of Plasma::Types::Vertical form factor and Plasma::Types::Right location, withthe plasmoids inside that would react to that concept.
That would be quite an huge things touching pretty much every level down to plasma-framework, but i feel this approach wouldn't really work.
On this systray plasmoid only, I would already prefer trying to give it a resize handle and allow generic vertical resizing (so also allowing to resize it to full height) is still not very robust, but at least kinda makes sense when the systray is not exactly where is expected (if necessary, there is quite some popup resizing code that was deleted still buried somewhere in years old git history)
I agree that it would probably make more sense to allow the system tray popup to be arbitrarily resized, and use a panel instead when you wand a vertical sidebar type of UI. The issues with using a panel to accomplish that are discussed in https://bugs.kde.org/show_bug.cgi?id=412483.