Bug 433123

Summary: screenshot selector is not working correctly
Product: [Applications] Discover Reporter: nttkde <watisthispoo>
Component: discoverAssignee: Dan Leinir Turthra Jensen <leinir>
Status: RESOLVED FIXED    
Severity: normal CC: aleixpol, carl, nate
Priority: VHI Keywords: regression, usability
Version: 5.21.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.21.1
Attachments: screenshot selector

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