Re-add category display to app delegates

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



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

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

Let's try and see what it feels like


just Layout.fillWidth: !category.visible.

This revision is now accepted and ready to land.Oct 21 2019, 2:20 PM
ngraham updated this revision to Diff 68443.Oct 21 2019, 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:



GTK 2/3.x Themes:

Splash Screens:

This could to be discussed in T10891.