Re-add category display to app delegates
ClosedPublic

Authored by ngraham on Mon, Oct 21, 2:12 PM.

Details

Summary

A year ago, I removed this in 0911fbcd561c with no review. :/ Sorry about that. In the
past year, Discover's UI has come a long way, and I think we can re-add the category
display now without causing the app delegates to become super cluttered.

The category name is suppressed if it would be identical to the page's title.

Test Plan

Categories in search lists:


Categories on the featured page:

Categories in browse lists, featuring suppression-when-equal-to-the-page-title:

Diff Detail

Repository
R134 Discover Software Store
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Mon, Oct 21, 2:12 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMon, Oct 21, 2:12 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Mon, Oct 21, 2:12 PM
apol accepted this revision.Mon, Oct 21, 2:20 PM

Let's try and see what it feels like

discover/qml/ApplicationDelegate.qml
77

just Layout.fillWidth: !category.visible.

This revision is now accepted and ready to land.Mon, Oct 21, 2:20 PM
ngraham updated this revision to Diff 68443.Mon, Oct 21, 2:44 PM
ngraham marked an inline comment as done.

Simplify layout conditional

This revision was automatically updated to reflect the committed changes.

Could add the more appropriate icons for categories?

I found some Breeze icons that should looks good with category name.

Color Schemes:


Emoticons:

Fonts:

GTK 2/3.x Themes:

Splash Screens:

This could to be discussed in T10891.