D13087: [FocusRect] Use manual placement instead of anchors

2018-05-24 Thread Kai Uwe Broulik 
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

D13087: [FocusRect] Use manual placement instead of anchors

2018-05-24 Thread Kai Uwe Broulik 
This revision was automatically updated to reflect the committed changes.
Closed by commit R858:ec06b3bc2f98: [FocusRect] Use manual placement instead of 
anchors (authored by broulik).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D13087?vs=34787&id=34791#toc

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D13087?vs=34787&id=34791

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