D13087: [FocusRect] Use manual placement instead of anchors

Kai Uwe Broulik noreply at phabricator.kde.org
Thu May 24 08:57:32 UTC 2018


broulik created this revision.
broulik added reviewers: Plasma, mart.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  I thought I was smart by using `anchors` that get auto-mirroring but `leftPadding` of the `control` already took that into account leading to double mirroring undoing the effect.

TEST PLAN
  Ran in `qmlscene -reverse` and focus rect is properly aligned in Breeze and Windows style. Left-to-right is still correct

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

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

AFFECTED FILES
  org.kde.desktop/private/FocusRect.qml

To: broulik, #plasma, mart
Cc: plasma-devel, 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/20180524/fa844c6d/attachment.html>


More information about the Plasma-devel mailing list