Bug 480514 - Popup to configure preview plugins for desktop is cut off
Summary: Popup to configure preview plugins for desktop is cut off
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Containment (other bugs)
Version First Reported In: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2024-01-30 01:06 UTC by Nicolas Fella
Modified: 2024-02-01 22:19 UTC (History)
5 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/kirigami/-/commit/b18c1e6194238689faa258e02f0e765e795f6ea7
Version Fixed In:
Sentry Crash Report:

Attachments
Screenshot (297.36 KB, image/png)
2024-01-30 01:06 UTC, Nicolas Fella 		Details
moving the preview plugins window (266.04 KB, video/webm)
2024-01-30 05:56 UTC, Doug 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Fella 2024-01-30 01:06:25 UTC 
Created attachment 165331 [details]
Screenshot

STEPS TO REPRODUCE
1. Right-click on desktop
2. Configure Desktop and Wallpaper
3. Click "Icons"
4. Click "Configure Preview Plugins" 

OBSERVED RESULT
An overlaysheet opens. The Apply/Cancel button on the bottom are cut off and can't be used. See screenshot

EXPECTED RESULT
Not that

SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 6.7

ADDITIONAL INFORMATION
Comment 1 Doug 2024-01-30 05:56:18 UTC 
Created attachment 165337 [details]
moving the preview plugins window
Comment 2 Doug 2024-01-30 05:57:16 UTC 
It was visible for me initially, but that "Preview Plugins" popup window can be moved up and down by grabbing it's title bar, but it only draws inside its parent window.
Comment 3 Nate Graham 2024-01-30 21:45:19 UTC 
Works as expected for me with Qt 6.6.0. Maybe a Qt 6.7 issue?
Comment 4 Doug 2024-01-31 03:40:13 UTC 
(In reply to Nicolas Fella from comment #0)
> Created attachment 165331 [details]
> Screenshot
> 
> STEPS TO REPRODUCE
> 1. Right-click on desktop
> 2. Configure Desktop and Wallpaper
> 3. Click "Icons"
> 4. Click "Configure Preview Plugins" 
> 
> OBSERVED RESULT
> An overlaysheet opens. The Apply/Cancel button on the bottom are cut off and
> can't be used. See screenshot
> 
> EXPECTED RESULT
> Not that
> 
> SOFTWARE/OS VERSIONS
> KDE Plasma Version: master
> KDE Frameworks Version: master
> Qt Version: 6.7
> 
> ADDITIONAL INFORMATION

Can you drag the overlaysheet up and down by it's titlebar?
Comment 5 Doug 2024-01-31 03:40:46 UTC 
(In reply to Nate Graham from comment #3)
> Works as expected for me with Qt 6.6.0. Maybe a Qt 6.7 issue?

I'm on Qt 6.6.1.  Can you drag the overlaysheet up and down by it's titlebar?
Comment 6 Nate Graham 2024-01-31 17:41:47 UTC 
You cannot.
Comment 7 Patrick Silva 2024-02-01 14:41:32 UTC 
Can reproduce on Arch Linux running Plasma 6 RC2 and Qt 6.7 beta2.
Comment 8 Bug Janitor Service 2024-02-01 16:21:18 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kirigami/-/merge_requests/1454
Comment 9 Nicolas Fella 2024-02-01 22:19:36 UTC 
Git commit b18c1e6194238689faa258e02f0e765e795f6ea7 by Nicolas Fella, on behalf of Marco Martin.
Committed on 01/02/2024 at 22:19.
Pushed by nicolasfella into branch 'master'.

Fix Overlay sheet size hints

Make sure the actual root element of the Overlay sheet doesn't go out of
the app window by limiting the hints, then in the case of a manual
flickable inserted in the sheet, use the flickable content height as
implicit height to try to show it without a scrollbar
Related: bug 480673, bug 480508, bug 480147

M  +10   -6    src/controls/templates/OverlaySheet.qml

https://invent.kde.org/frameworks/kirigami/-/commit/b18c1e6194238689faa258e02f0e765e795f6ea7