D22176: Rework system tray settings

[Nicolas Fella]

nicolasfella added a comment.


  In D22176#519573 <https://phabricator.kde.org/D22176#519573>, @kmaterka wrote:
  
  > @nicolasfella I updated D23413 <https://phabricator.kde.org/D23413>, now it can be used in configuration entries. Can you check if you can use this?
  
  
  That makes sense, thanks
  
  > I like the idea of removing "Categories", these are confusing and probably no-one uses them.
  >  **Important thing**, if we remove categories, then we need to remove:
  > 
  > - check in AbstractItem.qml
  > - "shownCategories" in main.qml
  > - probably small cleanup in few other places. If not, user won't see items, as they won't be able to re-enable categories again.
  
  ack
  
  > In D22176#489327 <https://phabricator.kde.org/D22176#489327>, @ngraham wrote:
  > 
  >> Honestly I would support staying with the QQC1 TableView for now. Trying to move to QQC2 without a replacement for it just means we're rolling our own table view, which seems even hackier.
  > 
  > 
  > In Qt 5.12 TableView is added to QtQuick module <https://blog.qt.io/blog/2018/08/29/tableview/>, in the similar way as ListView. There is no plan to add it to QQC2. I don't know how difficult migration is or how complete the implementation in Qt 5.12 is (missing header support?).
  
  I think QQC1 TableView and the new TableView are quite different things. I had some issues with the QQC1 TableView and my new Model, so I decided to use something else, but I can revisit this

REPOSITORY
  R120 Plasma Workspace

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

To: nicolasfella, #plasma, #vdg
Cc: kmaterka, filipf, broulik, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190826/ec4318c8/attachment.html>