Bug 452437 - Vertical scroll bar is activated in 'About this System" KCM despite there is nothing to scroll
Summary: Vertical scroll bar is activated in 'About this System" KCM despite there is ...
Status: RESOLVED FIXED
Alias: None
Product: kinfocenter
Classification: Applications
Component: About this System (other bugs)
Version First Reported In: master
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-09 15:09 UTC by Patrick Silva
Modified: 2022-04-11 17:59 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.25
Sentry Crash Report:


Attachments
screen recording (499.97 KB, video/webm)
2022-04-09 15:09 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2022-04-09 15:09:53 UTC
Created attachment 148072 [details]
screen recording

SUMMARY
My screen height is enough to make the whole content visible but the vertical scroll bar is activated anyway.
Please watch the attached screen recording.


EXPECTED RESULT
the scroll bar should not be unnecessarily activated

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.24.80
KDE Frameworks Version: 5.93.0
Qt Version: 5.15.3
Graphics Platform: Wayland
Comment 1 Nate Graham 2022-04-11 17:41:18 UTC
Can reproduce. This means that there's a whitespace element at the end that's taking up vertical space when it shouldn't be. Let me see if I can fix it.
Comment 2 Nate Graham 2022-04-11 17:58:47 UTC
Git commit 75aa549d6ac321a92b1e392ec648b14a681eac25 by Nate Graham.
Committed on 11/04/2022 at 17:54.
Pushed by ngraham into branch 'master'.

modules/about-distro: simplify spacing before "open in Info Center" button

We don't need an extra ColumnLayout here, which drags in extra unwanted
vertical spacing unless we remember to override it to be 0. It's simpler
to just not have it at all.

M  +15   -13   Modules/about-distro/src/package/contents/ui/main.qml

https://invent.kde.org/plasma/kinfocenter/commit/75aa549d6ac321a92b1e392ec648b14a681eac25
Comment 3 Nate Graham 2022-04-11 17:58:55 UTC
Git commit 0572deecaf1750969795c48cecefda6ae8616d41 by Nate Graham.
Committed on 11/04/2022 at 17:55.
Pushed by ngraham into branch 'master'.

modules/about-distro: use standard inner margins for page

We typically use Kirigami.Units.gridUnit for pages' inner margins. Let's
do the same thing here. This results in the inner margins being a bit
smaller than they were before, but it's still visually fine, and now
they're consistent with other pages.

M  +3    -3    Modules/about-distro/src/package/contents/ui/main.qml

https://invent.kde.org/plasma/kinfocenter/commit/0572deecaf1750969795c48cecefda6ae8616d41
Comment 4 Nate Graham 2022-04-11 17:59:03 UTC
Git commit a2984b0d79ddf613fa089659c41148fe43ce50f7 by Nate Graham.
Committed on 11/04/2022 at 17:44.
Pushed by ngraham into branch 'master'.

modules/about-distro: remove bottom padding from page

Is serves no visual purpose and causes the page to inappropriately
become scrollable when the content height is close to the total page
height.
FIXED-IN: 5.25

M  +0    -1    Modules/about-distro/src/package/contents/ui/main.qml

https://invent.kde.org/plasma/kinfocenter/commit/a2984b0d79ddf613fa089659c41148fe43ce50f7