Bug 497772

Summary: Screenshot blurry until clicked on
Product: [Applications] Discover Reporter: Justin Zobel <justin>
Component: discoverAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: aleixpol, nate
Priority: NOR    
Version: master   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 6.3.0
Sentry Crash Report:

Description Justin Zobel 2024-12-22 01:56:08 UTC
Open ISO Image Writer (the KDE one) page in Discover

Screenshot is blurry

If you click on the screenshot, it's clear.
Comment 1 Justin Zobel 2024-12-22 01:59:51 UTC
Maybe fixed by https://invent.kde.org/utilities/isoimagewriter/-/merge_requests/48
Comment 2 Aleix Pol 2025-01-25 12:09:40 UTC
The thumbnail we get is tiny:
https://dl.flathub.org/media/org/kde/isoimagewriter.desktop/3061d547797aef91a420ac6972407c57/screenshots/image-1_224x174@1.png

Potentially a flathub bug, needs investigating further.
Comment 3 Bug Janitor Service 2025-01-25 13:06:18 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/1016
Comment 4 Justin Zobel 2025-01-28 00:07:13 UTC
(In reply to Aleix Pol from comment #2)
> https://dl.flathub.org/media/org/kde/isoimagewriter.desktop/
> 3061d547797aef91a420ac6972407c57/screenshots/image-1_224x174@1.png

Interestingly that is not the size of the original image. The image in the appstream is https://cdn.kde.org/screenshots/isoimagewriter/isoimagewriter2.png
Comment 5 Aleix Pol 2025-01-29 22:41:48 UTC
Git commit 370588e4b6e1338a80b2cb70fde732bf617628b0 by Aleix Pol Gonzalez, on behalf of Aleix Pol.
Committed on 29/01/2025 at 22:32.
Pushed by apol into branch 'master'.

appstream: Disregard the thumbnail when it's too small

M  +2    -1    discover/qml/CarouselDelegate.qml
M  +3    -0    libdiscover/ScreenshotsModel.cpp
M  +1    -0    libdiscover/ScreenshotsModel.h
M  +11   -9    libdiscover/appstream/AppStreamUtils.cpp
M  +1    -1    libdiscover/appstream/AppStreamUtils.h
M  +1    -1    libdiscover/backends/KNSBackend/KNSResource.cpp
M  +4    -1    libdiscover/resources/AbstractResource.h

https://invent.kde.org/plasma/discover/-/commit/370588e4b6e1338a80b2cb70fde732bf617628b0
Comment 6 Aleix Pol 2025-01-29 22:58:34 UTC
Git commit 74b9d41f7b03af8e4ac364370515648ea22f7daf by Aleix Pol Gonzalez.
Committed on 29/01/2025 at 22:44.
Pushed by apol into branch 'Plasma/6.3'.

appstream: Disregard the thumbnail when it's too small


(cherry picked from commit 370588e4b6e1338a80b2cb70fde732bf617628b0)

Co-authored-by: Aleix Pol <aleixpol@kde.org>

M  +2    -1    discover/qml/CarouselDelegate.qml
M  +3    -0    libdiscover/ScreenshotsModel.cpp
M  +1    -0    libdiscover/ScreenshotsModel.h
M  +11   -9    libdiscover/appstream/AppStreamUtils.cpp
M  +1    -1    libdiscover/appstream/AppStreamUtils.h
M  +1    -1    libdiscover/backends/KNSBackend/KNSResource.cpp
M  +4    -1    libdiscover/resources/AbstractResource.h

https://invent.kde.org/plasma/discover/-/commit/74b9d41f7b03af8e4ac364370515648ea22f7daf