Bug 388721 - App detail view header toolbar
Summary: App detail view header toolbar
Status: RESOLVED NOT A BUG
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-09 10:43 UTC by andreas
Modified: 2018-01-17 15:42 UTC (History)
1 user (show)

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


Attachments
Discover detail app view with header (128.67 KB, image/jpeg)
2018-01-09 10:43 UTC, andreas
Details
another header layout (129.27 KB, image/jpeg)
2018-01-09 15:04 UTC, andreas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description andreas 2018-01-09 10:43:17 UTC
Created attachment 109753 [details]
Discover detail app view with header

the toolbar with center align of application name and icon look a bit out of place cause everything else is left align

I'm not sure to have the app icon in the header toolbar in addition the application description look a bit out of place "Playes music and last.fm streams"

In the past Category "Multimedia" was blue cause it was everywhere else also blue
Comment 1 Nate Graham 2018-01-09 14:01:08 UTC
App icon and name in header:
I think it is nice to have the app icon in the header, but IMHO It was even nicer before when we had the app icon really huge on the main page. This is probably a better location for a huge icon than the browse/search lists, even. Maybe we could keep the icon in the header but re-add it to the left of the info?

App caption looks out of place:
Yeah, I kind of agree. I wasn't sure what else to do with it, though.

Blue text:
Making it blue was misleading; everywhere else, a blue color indicates that the text is a link. We could make it blue again only if we make the text a link to that category.
Comment 2 andreas 2018-01-09 14:54:31 UTC
Blue text:
+1 for make it a link to the category. Than you don't have to remove the blue and the color is usefull too.
Comment 3 andreas 2018-01-09 15:04:53 UTC
Created attachment 109759 [details]
another header layout

mix between old and new as the Description "Header" has the same size than the application name header it fit's quite good.
Comment 4 Nate Graham 2018-01-09 15:24:07 UTC
I don't like the left-aligned "Clementine" in the header because being right next to the back button, it seems like clicking on the back button will take you *back* to Clementine, which isn't accurate.
Comment 5 andreas 2018-01-09 15:33:54 UTC
ok than center align (best would be left align to the information blog on top.
Comment 6 Aleix Pol 2018-01-17 15:42:10 UTC
We already reworked this page for 5.12.