Created attachment 149725 [details] Application screenshot example We could make screenshot bigger in the applications page, trying to check what an application looks like without opening the screenshot overlay thing is very hard, maybe we could make the screenshot preview really good, like very legible and easy to see and then remove the screenshot overlay thing. STEPS TO REPRODUCE 1. Open discover and open an application page 2. Check application's screenshots OBSERVED RESULT It's very hard to understand anything, very hard to see EXPECTED RESULT We should be able to see and understand a screenshot without needing the overlay screenshot thing SOFTWARE/OS VERSIONS Linux/KDE Plasma: Fedora KDE 36 (available in About System) KDE Plasma Version: 5.25 KDE Frameworks Version: 5.95 Qt Version: 5.15.3
The issue with making the screenshots bigger is that eventually the header section that they live in will become enormous and push down the rest of the content really far. We can do that of course, but when I tried it I didn't really like the results. Maybe you can play around with the layout and see if you can come up with something good?
On KNS resources (rather than AppStream generally) we generally get tiny pictures. I guess we could be skipping their thumbnails altogether? See any app (e.g. digiKam), they look much better over there.
Screenshot thumbnails are now a bit larger in Plasma 6, and further improvements to the screenshot UX are going to be merged shortly!
Git commit ef687b397b95ef3530f780c236fc3fe7a64d12f6 by ivan tkachenko. Committed on 26/10/2023 at 17:49. Pushed by ratijas into branch 'master'. carousel: Implement FullScreen and Overlay views Current index is synchronized between views, with "other" view updating without animations. Both overlays (native window and in-scene popup) are self-destructible on close. Animated gifs are auto-playing when maximizing, and stop playing when current index changes away from them. Play/Pause button reacts on mouse move, and auto-hides after a brief timeout. Related: bug 433321, bug 465794, bug 465796, bug 444103 M +60 -33 discover/qml/CarouselDelegate.qml M +1 -0 discover/qml/CarouselFullScreenMaximizedView.qml M +8 -1 discover/qml/CarouselInlineView.qml M +101 -6 discover/qml/CarouselMaximizedViewContent.qml M +1 -0 discover/qml/CarouselOverlayMaximizedView.qml M +1 -0 discover/qml/CarouselPageIndicator.qml https://invent.kde.org/plasma/discover/-/commit/ef687b397b95ef3530f780c236fc3fe7a64d12f6