ngraham created this revision.
ngraham added reviewers: VDG, Discover Software Store, apol.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  A year ago, I removed this in 0911fbcd561c 
<https://phabricator.kde.org/R134:0911fbcd561c0bc4b6080d9c0f3a3c70db5638f4> 
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: F7641610: Categories in search lists.png 
<https://phabricator.kde.org/F7641610>
  Categories on the featured page: F7641609: Categories on featured page.png 
<https://phabricator.kde.org/F7641609>
  Categories in browse lists, featuring 
suppression-when-equal-to-the-page-title: F7641608: Categories on browse lists 
featuring deduplication.png <https://phabricator.kde.org/F7641608>

REPOSITORY
  R134 Discover Software Store

BRANCH
  re-add-categories-display-on-delegate (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D24832

AFFECTED FILES
  discover/qml/ApplicationDelegate.qml

To: ngraham, #vdg, #discover_software_store, apol
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to