Created attachment 135963 [details] screen recording Title says it all. Watch the attached screen recording please. Operating System: Arch Linux KDE Plasma Version: 5.21.0 KDE Frameworks Version: 5.79.0 Qt Version: 5.15.2 Graphics Platform: X11
What's going on here is that the arrow internally, invisibly selects the next screenshot rather than scrolling. If the next screenshot is capable of being selected without scrolling because it is already visible, the view will not scroll. The arrows should always scroll rather than changing the index of which screenshot is selected.
-Also, optimally, the arrows should take the screenshot gallery(/window) width into account and hide when it's wide enough to show all screenshots at once (as the arrows are then unnecessary and obviously won't do anything when user clicks on them). Currently, the logic seems to be the arrows are hidden when there is only one screenshot. On the other hand, the horizontal gallery scrollbar does hide correctly based on gallery width. -And the current way it works, as Nate said, causes also another bug; the arrows aren't affected by the horizontal gallery scrollbar. If I drag the scrollbar to the right side and then press the right arrow, it'll jump back to the second screenshot from the start.
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 465794, bug 465796, bug 455304, 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