Merge lp://staging/~fboucault/ubuntu-ui-toolkit/empty_text_field_long_press_handling into lp://staging/ubuntu-ui-toolkit/staging

Proposed by Florian Boucault
Status: Superseded
Proposed branch: lp://staging/~fboucault/ubuntu-ui-toolkit/empty_text_field_long_press_handling
Merge into: lp://staging/ubuntu-ui-toolkit/staging
Diff against target: 73 lines (+33/-1)
3 files modified
modules/Ubuntu/Components/TextField.qml (+3/-0)
tests/unit_x11/tst_components/tst_textfield.qml (+1/-1)
tests/unit_x11/tst_components/tst_textinput_touch.qml (+29/-0)
To merge this branch: bzr merge lp://staging/~fboucault/ubuntu-ui-toolkit/empty_text_field_long_press_handling
Reviewer Review Type Date Requested Status
Zsombor Egri Needs Fixing
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+240119@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2014-11-04.

Commit message

TextField: make sure that a long press always works even when the field is empty by ensuring the TextInput's size is set properly.

To post a comment you must log in.
1312. By Florian Boucault

Fix alignment

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

You must adjust the unit tests to the changes.

review: Needs Fixing
1313. By Florian Boucault

Added unit test and fixed broken one.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

Seems the text area doesn't obey to the long tap.

review: Needs Fixing

Unmerged revisions

1313. By Florian Boucault

Added unit test and fixed broken one.

1312. By Florian Boucault

Fix alignment

1311. By Florian Boucault

TextField: make sure that a long press always works even when the field is empty by ensuring the TextInput's size is set properly.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches