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, could not run the patch, because Discover master fails to start for me:
  
    QQmlApplicationEngine failed to load component
    qrc:/qml/DiscoverWindow.qml:159 Cannot assign to non-existent property 
"showCloseButton"
  
  Any tips? I ran this on a non-master KF5/Plasma desktop, but CMake did not 
complain so it should be fine, right?
  
  One more thing: Lately I'm missing the "before" screenshots in your patches, 
although they would be really helpful! (If you installed to `~/usr` instead of 
`/usr`, for non-unique apps you could even run both versions in parallel for 
all your screenshotting needs, or in sequence for unique apps…)

REPOSITORY
  R134 Discover Software Store

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

To: ngraham, apol, #discover_software_store
Cc: rkflx, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to