Bug 494902

Summary: Elements overlap scrollbar while loading overlay is present
Product: [Applications] plasma-systemmonitor Reporter: tqd8 <tempqd888>
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: minor CC: ahiemstra, kdedev, nate, plasma-bugs
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=488044
Latest Commit: Version Fixed In: 6.2.4
Sentry Crash Report:
Attachments: Elements overlapping scrollbar while loading overlay is present

Description tqd8 2024-10-17 01:13:57 UTC
Created attachment 174927 [details]
Elements overlapping scrollbar while loading overlay is present

SUMMARY
When clicking "Edit Page", and the loading overlay shows up, the underlying content can overlap the scrollbar (visible if you hover to highlight it).

STEPS TO REPRODUCE
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.

OBSERVED RESULT
Blue highlights are seen overlapping the scrollbar.
Note: this doesn't happen if, in step 1, the page is small enough that it *does* have a scroll bar to begin with.
Maybe the scrollbar appearing is itself a bug?

EXPECTED RESULT
Nothing is seen


OS: openSUSE Krypton (git master, Qt 6.8.0)
Comment 1 Nate Graham 2024-10-22 19:11:33 UTC
Can reproduce. To notice it, you have the move the pointer over the loading overlay.
Comment 2 Bug Janitor Service 2024-11-21 13:40:47 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/316
Comment 3 Akseli Lahtinen 2024-11-22 10:05:16 UTC
Git commit e4fe1b39c5bff5d4a99a137090ebb0c43dbf8d7d by Akseli Lahtinen.
Committed on 22/11/2024 at 10:05.
Pushed by akselmo into branch 'master'.

EditablePage: Set loadOverlay margins explicitly, set loader invisible during loading

When items are being loaded, they should be invisible so they do not
flicker or otherwise show up during creation.

Set the loadOverlay margins to fit their parent. Previously if
parent had margins, loadOverlay would also use those same margins,
not hiding the page fully. This makes sure we ignore those margins
for loadOverlay.

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

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/e4fe1b39c5bff5d4a99a137090ebb0c43dbf8d7d
Comment 4 Akseli Lahtinen 2024-11-22 10:18:25 UTC
Git commit 86bb7928461d10553ba09179895b15abf25f6aea by Akseli Lahtinen.
Committed on 22/11/2024 at 10:15.
Pushed by akselmo into branch 'Plasma/6.2'.

EditablePage: Set loadOverlay margins explicitly, set loader invisible during loading

When items are being loaded, they should be invisible so they do not
flicker or otherwise show up during creation.

Set the loadOverlay margins to fit their parent. Previously if
parent had margins, loadOverlay would also use those same margins,
not hiding the page fully. This makes sure we ignore those margins
for loadOverlay.


(cherry picked from commit e4fe1b39c5bff5d4a99a137090ebb0c43dbf8d7d)

163fa492 EditablePage: Set loadOverlay margins explicitly, set loader invisible during loading
cc7ce951 Set visible false/true inside the if statements
2f8c1aed Remove centerIn parent

Co-authored-by: Akseli Lahtinen <akselmo@akselmo.dev>

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

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/86bb7928461d10553ba09179895b15abf25f6aea