Created attachment 173608 [details] Large icon on panel instead of menu SUMMARY When adding a audio volume widget onto a thick plasma panel it shows a large icon instead of the audio volume widget menu STEPS TO REPRODUCE 1. Have a vertical thick panel (e.g 300px panel width) 2. Add a audio volume widget directly to the panel or 1. Have a thick panel (e.g 300px panel width) 2. Drag a audio volume widget from the desktop onto the thick panel 3. Log out/Log in OBSERVED RESULT large icon instead of the widget menu EXPECTED RESULT showing the widget menu on the panel SOFTWARE/OS VERSIONS Linux/KDE Plasma: CachyOS Linux KDE Plasma Version: 6.1.5 KDE Frameworks Version: 6.5.0 Qt Version: 6.7.2
(In reply to isaacmcnatty from comment #0) > Created attachment 173608 [details] > Large icon on panel instead of menu > > SUMMARY > > When adding a audio volume widget onto a vertical thick plasma panel it shows a large > icon instead of the audio volume widget menu > > > STEPS TO REPRODUCE > 1. Have a vertical thick panel (e.g 300px panel width) > 2. Add a audio volume widget directly to the panel > > or > > 1. Have a thick panel (e.g 300px panel width) > 2. Drag a audio volume widget from the desktop onto the thick panel > 3. Log out/Log in > > OBSERVED RESULT > large icon instead of the widget menu > > EXPECTED RESULT > showing the widget menu on the panel > > > SOFTWARE/OS VERSIONS > Linux/KDE Plasma: CachyOS Linux > KDE Plasma Version: 6.1.5 > KDE Frameworks Version: 6.5.0 > Qt Version: 6.7.2
I cannot replicate this on the latest from git master. When you click on the volume icon in the new large panel does it show the volume menu?
Created attachment 173609 [details] how the widget was on plasma 5
Created attachment 173610 [details] how the widget was on plasma 6
i didnt mean when you click on the panel icon back in plasma 5 when the vertical panel was thick enough e.g 300px wide it would change the icon into the menu ive added 2 pictures to show it better
Ah yes now I see. Can confirm on git master, only icon is shown, not the applet with functionality shown. Screenshots always help so much, thank you.
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-pa/-/merge_requests/283
Git commit 7878c4566048f14783e1f4d62a7cc218386f7de2 by Christoph Wolk. Committed on 16/09/2024 at 07:38. Pushed by cwo into branch 'master'. applet: directly set switching sizes The volume applet currently reads the minimum size for switching to full representation from the Layout. This does not work in a panel though, because the applet is not in a layout, and the fullRepresentation is not loaded so the Layout properties set there can't be used for this either. This change sets the switching values explicitly (in multiples of gridUnit) and has the fullRepresentation set the Layout properties based on those, rather than the other way around. This is how similar applets do it (e.g. webbrowser or mediacontroller), and restores Plasma 5 functionality. M +4 -4 applet/contents/ui/main.qml https://invent.kde.org/plasma/plasma-pa/-/commit/7878c4566048f14783e1f4d62a7cc218386f7de2
Git commit 1138cefa411f18eee0c433b4c8a13833aa1423dd by Christoph Wolk. Committed on 16/09/2024 at 07:43. Pushed by cwo into branch 'Plasma/6.2'. applet: directly set switching sizes The volume applet currently reads the minimum size for switching to full representation from the Layout. This does not work in a panel though, because the applet is not in a layout, and the fullRepresentation is not loaded so the Layout properties set there can't be used for this either. This change sets the switching values explicitly (in multiples of gridUnit) and has the fullRepresentation set the Layout properties based on those, rather than the other way around. This is how similar applets do it (e.g. webbrowser or mediacontroller), and restores Plasma 5 functionality. (cherry picked from commit 7878c4566048f14783e1f4d62a7cc218386f7de2) Co-authored-by: Christoph Wolk <christoph.wolk@gmx.de> M +4 -4 applet/contents/ui/main.qml https://invent.kde.org/plasma/plasma-pa/-/commit/1138cefa411f18eee0c433b4c8a13833aa1423dd