Bug 388866 - Discover app list
Summary: Discover app list
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Aleix Pol
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-01-12 14:30 UTC by andreas
Modified: 2018-01-16 15:58 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
app list with different wide (202.55 KB, image/jpeg)
2018-01-12 14:30 UTC, andreas
Details

Note You need to log in before you can comment on or make changes to this bug.
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