Bug 389321

Summary:	ApplicationPage is overwhelming with more than 3 screenshots at a time
Product:	[Applications] Discover	Reporter:	Nate Graham <nate>
Component:	discover	Assignee:	Aleix Pol <aleixpol>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	anditosan1000
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Neon
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Awesome horizontal scrollview for screenshots

Description Nate Graham 2018-01-22 18:26:18 UTC

[Discover from git Plasma/5.12 branch]

Right now, when there are more than about 2 or 3 screenshots on Discover's app page, they tile and create more rows. This looks fine as long as the window is large and the screenshots are all the same size (https://i.imgur.com/Rrr6WoC.png), but any deviation from this makes the window look ugly or crowded: https://i.imgur.com/NnRYjFo.jpg2)

VDG would like to see the screenshot thumbnails put into a horizontal scrollview instead of creating more rows.

As a part of this, once there will only ever be one row, it might be reasonable to increase the height of the thumbnails to `Kirigami.Units.gridUnit * 8` or maybe even 9 (https://cgit.kde.org/discover.git/tree/discover/qml/ApplicationScreenshots.qml?h=Plasma/5.12#n124)

Comment 1 Andres Betts 2018-01-22 20:12:46 UTC

I support this idea! It goes in line with what most app stores will do and it also helps the user make a quicker decision to install or not install.

Comment 2 Aleix Pol 2018-01-23 00:20:54 UTC

To be honest, I'm not very fond of how everything intertwines:
https://i.imgur.com/7hUZbZ4.png

I'd say it would be better to just have a + that opened the popup or so.

Comment 3 Aleix Pol 2018-01-23 01:16:57 UTC

Nate contacted in private asking for a frame. Here's with a frame. :)

1 screenshot:
https://i.imgur.com/e4C68qk.png

several:
https://i.imgur.com/1FDMuNp.png

several, no frame:
https://i.imgur.com/fAuLEXR.png

Comment 4 Nate Graham 2018-01-24 00:06:56 UTC

Created attachment 110080 [details]
Awesome horizontal scrollview for screenshots

I think we can call this fixed as of today's commits to the Plasma/5.12 branch. It's fantastic!