Bug 388866

Summary: Discover app list
Product: [Applications] Discover Reporter: andreas <kainz.a>
Component: discoverAssignee: Aleix Pol <aleixpol>
Status: RESOLVED FIXED    
Severity: normal CC: nate
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: app list with different wide

Description andreas 2018-01-12 14:30:44 UTC
Created attachment 109817 [details]
app list with different wide

jumping UI isn't best practice. When you move the section name to the right than you can shrink the UI and it didn't jump.

Be aware that text and button are align.

when the second line switch from full text to ... instead of ... the group name should hide until the window wide is that small that there isn't enough space for the app subtitle.

I would remove the blue line and use always the same layout.
Comment 1 Nate Graham 2018-01-12 15:32:28 UTC
In an earlier revision, I tried out putting the category under the install button like you're proposing, but I didn't think it looked as good or made as much sense as putting it under the app name. When it's under the app name, it feels connected with the app--which it is. But when it's under the install button, it feels somehow connected with installation--which it's not.

Also, making the category hide when it would elide the caption made for a weird visual experience, since you often got into a situation where there was a list of apps, and half of them didn't show the category because the caption was too long, but the other half did show them. It looked and felt odd.
Comment 2 Aleix Pol 2018-01-16 15:58:51 UTC
Git commit d5356147fea8f29525094ee85359e68205c9eaec by Aleix Pol.
Committed on 16/01/2018 at 12:14.
Pushed by apol into branch 'Plasma/5.12'.

Polish the ApplicationDelegate

Make sure the InstallApplicationButton never floats out of the delegate
when it's resized.
Reduce the delta between compact and non-compact

M  +21   -21   discover/qml/ApplicationDelegate.qml

https://commits.kde.org/discover/d5356147fea8f29525094ee85359e68205c9eaec