Commit 9dc1768a authored by Nate Graham's avatar Nate Graham 🔩
Browse files

Re-add category display to app delegates

Summary:
A year ago, I removed this in 0911fbcd 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 on the featured page: {F7641609}
Categories in browse lists, featuring suppression-when-equal-to-the-page-title: {F7641608}

Reviewers: #vdg, #discover_software_store, apol

Reviewed By: #discover_software_store, apol

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D24832
parent bbefb3de
......@@ -68,13 +68,28 @@ Kirigami.AbstractCard
columns: 2
rows: delegateArea.compact ? 4 : 3
Kirigami.Heading {
id: head
level: delegateArea.compact ? 3 : 2
RowLayout {
Layout.fillWidth: true
elide: Text.ElideRight
text: delegateArea.application.name
maximumLineCount: 1
Kirigami.Heading {
id: head
level: delegateArea.compact ? 3 : 2
Layout.fillWidth: !category.visible
elide: Text.ElideRight
text: delegateArea.application.name
maximumLineCount: 1
}
Kirigami.Heading {
id: category
level: 5
Layout.fillWidth: true
elide: Text.ElideRight
text: i18nc("Part of a string like this: '<app name> - <category>'", "- %1", delegateArea.application.categoryDisplay)
maximumLineCount: 1
opacity: 0.6
visible: delegateArea.application.categoryDisplay !== page.title
}
}
InstallApplicationButton {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment