Bug 488044 - Loading overlay doesn't cover full page when entering Edit Page mode
Summary: Loading overlay doesn't cover full page when entering Edit Page mode
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (show other bugs)
Version: master
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Nate Graham
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2024-06-05 01:41 UTC by tqd8
Modified: 2024-10-17 01:16 UTC (History)
4 users (show)

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


Attachments
Screencast of loading overlay issue, not covering full page (248.60 KB, video/webm)
2024-06-05 01:41 UTC, tqd8
Details
Elements overlapping scrollbar while loading overlay is present (37.51 KB, image/png)
2024-07-05 02:45 UTC, tqd8
Details

Note You need to log in before you can comment on or make changes to this bug.
Description tqd8 2024-06-05 01:41:19 UTC
Created attachment 170154 [details]
Screencast of loading overlay issue, not covering full page

When clicking "Edit Page", the full-page overlay (containing the spinning gear icon) appears to cover only part of the page. The System Monitor sections on the page seem to peek out over the top of the overlay. Video attached.

Happens at fractional and 100% scale, X11 and Wayland, at any window size.

OS: openSUSE Krypton (today's git master)
Comment 1 Nate Graham 2024-06-11 20:08:22 UTC
Can reproduce.
Comment 2 tqd8 2024-06-13 03:51:19 UTC
I think the culprit might be here? https://invent.kde.org/plasma/plasma-systemmonitor/-/blob/master/src/page/EditablePage.qml?ref_type=heads#L206

I don't have the ability to test changes to this right now, but I'm listing this here just in case it helps out.
Comment 3 Nate Graham 2024-06-13 15:19:34 UTC
Yes, that looks quite suspicious. Thanks!
Comment 4 Nate Graham 2024-06-13 15:30:45 UTC
Actually it was something else. Patch incoming.
Comment 5 Bug Janitor Service 2024-06-13 15:31:39 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/299
Comment 6 tqd8 2024-06-13 19:38:04 UTC
Ah nice thanks!
Comment 7 Nate Graham 2024-06-14 13:01:26 UTC
Git commit 3951c374c25fed5ef518e044273b47c6c5923e3d by Nate Graham.
Committed on 13/06/2024 at 15:28.
Pushed by ngraham into branch 'master'.

EditablePage: position loading placeholder overlay correctly

It was using negative margins to counteract the ones added to the page,
but was then re-parented to something that wasn't the page. If those two
things didn't have the exact same geometry, the overlay would be
mis-positioned.

Fix this by dropping the re-parenting, and instead manually set the Z to
something very high to ensure it always covers the content.
FIXED-IN: 6.1.1

M  +1    -1    src/page/EditablePage.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/3951c374c25fed5ef518e044273b47c6c5923e3d
Comment 8 Nate Graham 2024-06-14 13:02:18 UTC
Git commit eee81c58dc0169753ef17a68c663600685c701f6 by Nate Graham.
Committed on 14/06/2024 at 13:02.
Pushed by ngraham into branch 'Plasma/6.1'.

EditablePage: position loading placeholder overlay correctly

It was using negative margins to counteract the ones added to the page,
but was then re-parented to something that wasn't the page. If those two
things didn't have the exact same geometry, the overlay would be
mis-positioned.

Fix this by dropping the re-parenting, and instead manually set the Z to
something very high to ensure it always covers the content.
FIXED-IN: 6.1.1


(cherry picked from commit 3951c374c25fed5ef518e044273b47c6c5923e3d)

Co-authored-by: Nate Graham <nate@kde.org>

M  +1    -1    src/page/EditablePage.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/eee81c58dc0169753ef17a68c663600685c701f6
Comment 9 tqd8 2024-07-05 02:45:44 UTC
Created attachment 171390 [details]
Elements overlapping scrollbar while loading overlay is present

Looks like another similar issue is happening now (on today's git master). Not sure if it was there before.

When the loading overlay pops up, the underlying content can overlap the scrollbar (visible if you hover to highlight it).

Note this only happens when the page doesn't have a scrollbar to begin with and only gets a scrollbar when the overlay appears. (That's maybe another bug: it seems the overlay is taller than the page, so when it appears, it gets a scrollbar no matter what.) So I'm guessing the margins/anchors of the elements underneath somehow don't take this temporary scrollbar width into account? 

Not sure if I should reopen or file a new bug report.
Comment 10 Nate Graham 2024-07-23 18:22:16 UTC
I can't reproduce that with current git master. Can you describe how you're accomplishing it?
Comment 11 tqd8 2024-07-24 02:42:59 UTC
1. Ensure the System Monitor page is scaled large enough so that it has no scrollbar
2. Click "Edit Page"
3. A scrollbar appears -- move your mouse all around it.

You should see the blue squares peeking into the scrollbar where your mouse hovers (like in my screenshot).

Note: this doesn't happen if, in step 1, the page is small enough that it *does* have a scroll bar to begin with.
Comment 12 tqd8 2024-10-05 05:05:44 UTC
I can still reproduce this new issue (in Comments 9, 10, & 11) on current git master.
Comment 13 Nate Graham 2024-10-16 16:06:10 UTC
Can you open a new bug report for it?
Comment 14 tqd8 2024-10-17 01:16:43 UTC
Sure! Bug 494902