Bug 440437 - Button to scroll left shows when no scroll-able area exists
Summary: Button to scroll left shows when no scroll-able area exists
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: 5.22.4
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Dan Leinir Turthra Jensen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-31 02:58 UTC by kingofgamesyami
Modified: 2021-08-02 20:29 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.23


Attachments
Screenshot of IntelliJ IDEA Community package in Discover (691.99 KB, image/png)
2021-07-31 02:58 UTC, kingofgamesyami
Details

Note You need to log in before you can comment on or make changes to this bug.
Description kingofgamesyami 2021-07-31 02:58:59 UTC
Created attachment 140419 [details]
Screenshot of IntelliJ IDEA Community package in Discover

SUMMARY
When viewing a package in Discover (in my example, IntelliJ IDEA Community Flatpak) that has few pictures, there is no scroll area for the pictures.

However, the button to scroll to the next position within the scroll area still exists. Clicking the button removes it and adds a button to scroll to the previous position, which functions similarly.

STEPS TO REPRODUCE
1. Open discover on a large display
2. Open the description for IntelliJ IDEA Community

OBSERVED RESULT
Button exists

EXPECTED RESULT
No button

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: KDE Neon 5.22
(available in About System)
KDE Plasma Version: 5.22.4
KDE Frameworks Version: 5.84.0
Qt Version: 5.15.3

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2021-08-02 20:13:13 UTC
Can confirm.
Comment 2 Nate Graham 2021-08-02 20:29:16 UTC
Git commit 1c28a1ccc9cdcbbf77a1cab870e102503c4500d1 by Nate Graham.
Committed on 02/08/2021 at 20:28.
Pushed by ngraham into branch 'master'.

Only show screenshot arrow buttons when the view is visibly scrollable

This does not affect the popup, where we always want to show the arrow
buttons because only one screenshot is visible at a time.
FIXED-IN: 5.23

M  +2    -2    discover/qml/ApplicationScreenshots.qml

https://invent.kde.org/plasma/discover/commit/1c28a1ccc9cdcbbf77a1cab870e102503c4500d1