Bug 433123 - screenshot selector is not working correctly
Summary: screenshot selector is not working correctly
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: 5.21.0
Platform: Other Linux
: VHI normal
Target Milestone: ---
Assignee: Dan Leinir Turthra Jensen
URL:
Keywords: regression, usability
Depends on:
Blocks:
 
Reported: 2021-02-17 22:17 UTC by nttkde
Modified: 2021-02-19 18:36 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.21.1


Attachments
screenshot selector (3.73 MB, video/mp4)
2021-02-17 22:17 UTC, nttkde
Details

Note You need to log in before you can comment on or make changes to this bug.
Description nttkde 2021-02-17 22:17:27 UTC
Created attachment 135798 [details]
screenshot selector

Operating System: KDE neon 5.21
KDE Plasma Version: 5.21.0
KDE Frameworks Version: 5.79.0
Qt Version: 5.15.2


See attached video.
The screenshot thumbnail view behaves weirdly and clicking on top of any other screenshot opens the 'active' screenshot (selected with the arrow buttons) anyways.
It behaves the same even if Discover window is maximized etc.
(In the second example on the video, the thumbnails seem quite low resolution too (only the first actual screenshot seems to be low-res). Might be some other bug.)
Comment 1 Nate Graham 2021-02-17 22:19:45 UTC
Confirmed.

Looks like clicking on a screenshot always opens the invisibly selected one, and the only thing that moves the selection is the arrow buttons.
Comment 2 Aleix Pol 2021-02-19 16:13:31 UTC
Git commit 4c77b238419aa437f41a2f6a0c6acd65c79aef88 by Aleix Pol.
Committed on 19/02/2021 at 16:13.
Pushed by apol into branch 'master'.

Show the correct screenshot when clicking it

M  +4    -1    discover/qml/ApplicationScreenshots.qml

https://invent.kde.org/plasma/discover/commit/4c77b238419aa437f41a2f6a0c6acd65c79aef88
Comment 3 Aleix Pol 2021-02-19 16:13:45 UTC
Git commit 357d51ff9902c3019da9cc16ccb59a63d1e304be by Aleix Pol Gonzalez, on behalf of Aleix Pol.
Committed on 19/02/2021 at 16:13.
Pushed by apol into branch 'Plasma/5.21'.

Show the correct screenshot when clicking it


(cherry picked from commit 4c77b238419aa437f41a2f6a0c6acd65c79aef88)

M  +4    -1    discover/qml/ApplicationScreenshots.qml

https://invent.kde.org/plasma/discover/commit/357d51ff9902c3019da9cc16ccb59a63d1e304be