Bug 401972 - Scrollbar partially overlaps Application page contents
Summary: Scrollbar partially overlaps Application page contents
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: unspecified
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Aleix Pol
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-12-10 18:16 UTC by Alexander Mentyu
Modified: 2018-12-13 15:36 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Screenshot of Transmission app info page in mobile mode (141.86 KB, image/png)
2018-12-10 18:16 UTC, Alexander Mentyu
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Mentyu 2018-12-10 18:16:44 UTC
Created attachment 116835 [details]
Screenshot of Transmission app info page in mobile mode

SOFTWARE/OS VERSIONS
Operating System: KDE neon Developer Edition
KDE Plasma Version: 5.14.80
KDE Frameworks Version: 5.54.0
Qt Version: 5.11.2
Kernel Version: 4.15.0-42-generic
Comment 1 Nate Graham 2018-12-11 18:15:31 UTC
Cannot confirm with 5.14.4. Can confirm with git master.

This issue does not only affect mobile mode; in desktop mode too, the vertical scrollbar overlaps the page contents. I suspect a Kirigami regression since the code for this page has not changed recently.
Comment 2 Aleix Pol 2018-12-13 03:43:16 UTC
https://phabricator.kde.org/D17551
Comment 3 Aleix Pol 2018-12-13 15:36:35 UTC
Git commit 582721206a03f5704df3855a1acc5d24a873bba2 by Aleix Pol.
Committed on 13/12/2018 at 15:34.
Pushed by apol into branch 'master'.

Take the scrollbar into account for the ScrollablePage margin

Summary: BUG: 401972

Test Plan: Now I never get that overlapping, it could have other ramifications though.

Reviewers: #kirigami, mart, ngraham

Reviewed By: #kirigami, mart, ngraham

Subscribers: ngraham, plasma-devel

Tags: #kirigami

Differential Revision: https://phabricator.kde.org/D17551

M  +2    -1    src/controls/ScrollablePage.qml

https://commits.kde.org/kirigami/582721206a03f5704df3855a1acc5d24a873bba2