D13085: [RadioButton] Rename "control" to "controlRoot"

2018-05-24 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes.
Closed by commit R858:9e864e8059c1: [RadioButton] Rename "control" to 
"controlRoot" (authored by broulik).

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D13085?vs=34785&id=34793

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

AFFECTED FILES
  org.kde.desktop/RadioButton.qml

To: broulik, #plasma, mart
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart


D13085: [RadioButton] Rename "control" to "controlRoot"

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
  This makes it consistent with `CheckBox` and also fixes the focus indicator.
  Also make the rest of the code identical to `CheckBox`

TEST PLAN
  - Compared the two files, were identical now except for the indicator 
component name and the onTriggered action
  - Focus indicator on RadioButton works again
  - Text color was still correct

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

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

AFFECTED FILES
  org.kde.desktop/RadioButton.qml

To: broulik, #plasma, mart
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart