Bug 499284 - Some previews don't show the panel in its actual orientation, while others do
Summary: Some previews don't show the panel in its actual orientation, while others do
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Panel (show other bugs)
Version: 6.2.5
Platform: Other Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-01-29 17:48 UTC by Nate Graham
Modified: 2025-03-14 05:08 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.4.0
Sentry Crash Report:


Attachments
Only some previews match the actual panel position (198.06 KB, image/png)
2025-01-29 17:48 UTC, Nate Graham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2025-01-29 17:48:40 UTC
Created attachment 177794 [details]
Only some previews match the actual panel position

When the panel is not on the bottom of the screen, the previews for Position, Alignment, and Width will update accordingly, and show the panel in its actual position on screen.

However, the previews for Visibility, Opacity, and Style do not.

See attached screenshot.
Comment 1 Niccolò Venerandi 2025-02-03 15:50:27 UTC
This is sort-of intentional from my side. The position/size indicators do preview the position, whereas the "style" indicators only preview the style changes. Previewing the position for all of them IMO looks a bit silly, and complicates quite a bit the code (especially now that there's animation and such). Maybe I should *only* preview the position on the position indicator, and drop it from alignment/width?
Comment 2 Niccolò Venerandi 2025-02-07 13:17:47 UTC
Putting in NEEDSINFO as I'd like some discussion about this
Comment 3 Nate Graham 2025-02-10 20:01:26 UTC
If that's too hard, then it should be only the orientation graphic that reflected the actual panel orientation. But I think it would be more polished if everything reflected it.
Comment 4 Bug Janitor Service 2025-03-04 13:57:19 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2865
Comment 5 Niccolò Venerandi 2025-03-09 15:41:32 UTC
Git commit 706530f59cc81482c7a0eb3fc24d3088f10041d5 by Niccolò Venerandi.
Committed on 09/03/2025 at 15:41.
Pushed by niccolove into branch 'master'.

Make panel representations adapt to all panel orientations

Now, all panel previews will follow the actuar orientation
that the panel has.

M  +9    -0    containments/panel/contents/ui/main.qml
M  +6    -0    desktoppackage/contents/configuration/PanelConfiguration.qml
M  +26   -8    desktoppackage/contents/configuration/panelconfiguration/PanelRepresentation.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/706530f59cc81482c7a0eb3fc24d3088f10041d5