diff options
| author | Frederik Gladhorn <frederik.gladhorn@digia.com> | 2014-10-15 13:41:17 +0200 |
|---|---|---|
| committer | Jan Arve Sæther <jan-arve.saether@theqtcompany.com> | 2014-10-15 13:47:31 +0200 |
| commit | cde75ab705b43c3512a594562e3742fa5f66da44 (patch) | |
| tree | e887fd3c900198a6f32007adc62653f16a8d7e60 | |
| parent | 7d669145f9182ecc4604f2abc5e67157f3b3368b (diff) | |
Move the city mouse area in the delegate up
For accessibility purposes the mouse area represents the city-country line.
In the current order the delete button is found before the city, thus a
screen reader will first offer the delete button and then the city which is
confusing.
Change-Id: Id05a92d2f9526915bb04320436d489a8330a2c0a
Reviewed-by: Jan Arve Sæther <jan-arve.saether@theqtcompany.com>
| -rw-r--r-- | qml/touch/ListViewDelegate.qml | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/qml/touch/ListViewDelegate.qml b/qml/touch/ListViewDelegate.qml index f040ad8..8f4c616 100644 --- a/qml/touch/ListViewDelegate.qml +++ b/qml/touch/ListViewDelegate.qml @@ -54,6 +54,18 @@ Rectangle { color: mouseNext.pressed ? ApplicationInfo.colors.smokeGray : ApplicationInfo.colors.white Accessible.role: Accessible.ListItem + MouseArea { + id: mouseNext + anchors.left: parent.left + width: parent.width - 80 * ApplicationInfo.ratio - ApplicationInfo.hMargin + height: parent.height + onClicked: rect.clicked() + Accessible.role: Accessible.Button + Accessible.name: loader.item.accessibleName + function accessiblePressAction() { + rect.clicked() + } + } GridLayout { id: _grid anchors.fill: parent @@ -86,18 +98,6 @@ Rectangle { width: parent.width color: ApplicationInfo.colors.paleGray } - MouseArea { - id: mouseNext - anchors.left: parent.left - width: parent.width - 80 * ApplicationInfo.ratio - ApplicationInfo.hMargin - height: parent.height - onClicked: rect.clicked() - Accessible.role: Accessible.Button - Accessible.name: loader.item.accessibleName - function accessiblePressAction() { - rect.clicked() - } - } property Component searchViewRow: RowLayout { spacing: 0 |