D11820: Handle properties with multiple values

[Michael Heidelbach]

michaelh added a comment.


  In D11820#246525 <https://phabricator.kde.org/D11820#246525>, @astippich wrote:
  
  > Sorry for the late answer. Isn't this already handled in widgetfactory.cpp toString function (line 87)? During my testing with multiple values I've never had issues with baloo-widgets, only with baloo itself.
  
  
  Have you tried `autotests/samplefiles/multi/*.*` of  D12197 <https://phabricator.kde.org/D12197>? For me baloo-widgets mostly displays only one author or keyword.
  
  > If there are still changes required in baloo-widgets, I think they should happen inside the toString() function.
  
  That could be done in `toString()`. The concatenation is only an intermediate step. The real plan is to use `TagWidget` for e.g. keywords, genres and so on, in that case I would have to move all the decision making back to where it is now.

REPOSITORY
  R824 Baloo Widgets

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

To: michaelh, #baloo, #frameworks
Cc: astippich, mgallien, elvisangelaccio, ashaposhnikov, spoorun, alexeymin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180415/29e1c956/attachment.html>