AbstractButton doesn't have a minimum size for the touch area
Bug #1508114 reported by
Andrea Bernabei
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
Zoltan Balogh | ||
| ubuntu-ui-toolkit (Ubuntu RTM) |
Fix Released
|
High
|
Zsombor Egri | ||
Bug Description
AbstractButton should have a minimum touch area of 4x4GU, as reported in the UI Toolkit spec (currently only in the ListItem layouts section, it has to be moved to a generic section)
That means
AbstractButton {
width: units.gu(6)
height: units.gu(6)
}
has a touch region of 6x6GUs, while
AbstractButton {
width: units.gu(2)
height: units.gu(2)
}
must have a touch region of 4x4GU.
UI Toolkit spec:
https:/
Related branches
lp:~zsombi/ubuntu-ui-toolkit/increasedSensingArea
- ubuntu-sdk-build-bot: Approve (continuous-integration)
- Tim Peeters: Approve
- PS Jenkins bot: Needs Fixing (continuous-integration)
-
Diff: 565 lines (+375/-0)8 files modifiedcomponents.api (+7/-0)
src/Ubuntu/Components/plugin/plugin.cpp (+1/-0)
src/Ubuntu/Components/plugin/plugin.pri (+1/-0)
src/Ubuntu/Components/plugin/ucabstractbutton.cpp (+135/-0)
src/Ubuntu/Components/plugin/ucabstractbutton.h (+9/-0)
src/Ubuntu/Components/plugin/ucabstractbutton_p.h (+2/-0)
src/Ubuntu/Components/plugin/ucmargins.h (+76/-0)
tests/unit_x11/tst_components/tst_abstractbutton13.qml (+144/-0)
Revision history for this message
| Zsombor Egri (zsombi) wrote | #1 |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | New → Confirmed |
| importance: | Undecided → High |
| assignee: | nobody → Zsombor Egri (zsombi) |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | Confirmed → In Progress |
| affects: | ubuntu-ui-toolkit (Ubuntu) → ubuntu-ui-toolkit (Ubuntu RTM) |
| Changed in ubuntu-ui-toolkit (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | New → Fix Committed |
| assignee: | nobody → Zoltan Balogh (bzoltan) |
| milestone: | none → ww08-2016 |
| Changed in canonical-devices-system-image: | |
| importance: | Undecided → High |
| Changed in canonical-devices-system-image: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This can be achieved only if the AbstractButton is not having clip set, as otherwise the clipping will restrict the sensing area too.