D20896: [Weather] Port settings window to QQC2+Kirigami FormLayout and modernize UI

[Friedrich W. H. Kossebau]

kossebau added inline comments.

INLINE COMMENTS

> ngraham wrote in ConfigWeatherStation.qml:85
> The `-symbolic` suffix means "this icon is always supposed to be monochrome". However in this case I used the -symbolic version simply because there was no non-symbolic version in Breeze icons. However it's not semantically incorrect to use that here anyway since buttons are generally only supposed to have monochrome icons in the first place.

Where is that documented? Asking because as developer writing new code one day I am now very confused what to use.
Besides why would there be action icons where this is not the case (at least with the Breeze style)? After all HIG says "Action and status icons [...] always use the monochrome style." (https://hig.kde.org/style/icon.html)
If other icon themes have different style ideas, that should be fine, after all that is the idea of themes to also have their own style guide, no?

So if possible, I would prefer that on changing this icon this is backed by something proper documented (and fixed where needed, e.g. ensuring there is an action icon with a normal name if needed) :)

REPOSITORY
  R114 Plasma Addons

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

To: ngraham, #vdg, #plasma, kossebau
Cc: filipf, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190429/488c0d16/attachment.html>