broulik added inline comments.

INLINE COMMENTS

> lesf0 wrote in main.qml:61
> That would make much more sence, but (at least for now) `QStringList` is 
> array-like object and not an actual `Array` instance, and I don't really feel 
> like I want to mess with all those "is-arrayish 
> <https://www.npmjs.com/package/is-arrayish>" javascript stuff. Strings are 
> way easier to check.

That's odd, I thought a `QStringList` would just turn into a plain JS `Array` 
on QML side. Fair enough. Will land this for you tomorrow, thanks.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  mediacontroller-applet-multiartist-support (branched from master)

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

To: lesf0, broulik
Cc: broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to