D11842: Remove categories from app delegates

2018-04-08 Thread Nathaniel Graham
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R134:0911fbcd561c: Remove categories from app delegates (authored by ngraham). REPOSITORY R134 Discover Software Store C

D11842: Remove categories from app delegates

2018-04-07 Thread Nathaniel Graham
ngraham added a comment. Patch to bump the version required: D12035: Bump KF5 version requirement to 5.45 REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D11842 To: ngraham, apol, #discover_software_store Cc: rkflx

D11842: Remove categories from app delegates

2018-04-07 Thread Nathaniel Graham
ngraham added a comment. In D11842#241862 , @rkflx wrote: > FWIW, have a +1 for the idea. Removing the categories is not ideal, but having the ratings can be much more enticing for the user to install the app, and there is only so much space… Th

D11842: Remove categories from app delegates

2018-04-07 Thread Henrik Fehlauer
rkflx added a comment. FWIW, have a +1 for the idea. Removing the categories is not ideal, but having the ratings can be much more enticing for the user to install the app, and there is only so much space… Thus I think it's the better compromise to go with the ratings. --- BTW, cou

D11842: Remove categories from app delegates

2018-04-01 Thread Nathaniel Graham
ngraham updated this revision to Diff 31109. ngraham added a comment. Rebase on master REPOSITORY R134 Discover Software Store CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11842?vs=31054&id=31109 BRANCH remove-categories-on-delegates (branched from master) REVISION DETAIL

D11842: Remove categories from app delegates

2018-03-31 Thread Nathaniel Graham
ngraham created this revision. ngraham added reviewers: apol, Discover Software Store. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. ngraham requested review of this revision. REVISION SUMMARY This patch removes category labels from the