D19873: [image-wallpaper] Fix horizontal alignment of the "Positioning:" row

[Filip Fila]

filipf added a comment.


  In D19873#434571 <https://phabricator.kde.org/D19873#434571>, @davidre wrote:
  
  > It seems it was changed in this commit to explicitly use Kirigami units:  98d9f681a37e2ac2feb6bf5cb5e8a54f4c7e874e <https://phabricator.kde.org/R120:98d9f681a37e2ac2feb6bf5cb5e8a54f4c7e874e>
  >  Was the Color wallpaper forgotten or did this reason not apply to it? Should we revert it?
  
  
  I think the rationale was "we're already importing Kirigami, so let's not import PlasmaCore as well solely for its units when Kirigami can provide units". As for why it wasn't ported elsewhere, maybe those plugins weren't using Kirigami yet.
  
  Bottom line though is we can either tweak Kirigami's units or just go back to the old ones (although they need tweaking as well). It's more or less the same to me, the best solution would be to use FormLayout in specific plugins.

REPOSITORY
  R120 Plasma Workspace

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

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