STEPS TO REPRODUCE 1. open Discover 2. open the description page of an application that has only one screenshot. KolourPaint, for example. 3. hover over the screenshot 4. try to scroll the description page vertically OBSERVED RESULT nothing happens EXPECTED RESULT description page should be vertically scrolled SOFTWARE/OS VERSIONS Operating System: KDE neon Unstable Edition KDE Plasma Version: 5.23.80 KDE Frameworks Version: 5.88.0 Qt Version: 5.15.3 Graphics Platform: Wayland
Yeah we should probably disable the horizontal scrollview in the screenshot container when everything fits.
Argh I can't find a way to do this with ScrollView
This is fixed for Plasma 6 now.
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 455304 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