Merge lp://qastaging/~ubuntu-sdk-team/ubuntu-ui-toolkit/focusPassesFromChildToParent into lp://qastaging/ubuntu-ui-toolkit/staging

Proposed by Cris Dywan
Status: Merged
Approved by: Zsombor Egri
Approved revision: 1725
Merged at revision: 1732
Proposed branch: lp://qastaging/~ubuntu-sdk-team/ubuntu-ui-toolkit/focusPassesFromChildToParent
Merge into: lp://qastaging/ubuntu-ui-toolkit/staging
Prerequisite: lp://qastaging/~ubuntu-sdk-team/ubuntu-ui-toolkit/enoFocusPriSecItem
Diff against target: 243 lines (+60/-11)
8 files modified
components.api (+1/-0)
src/Ubuntu/Components/1.3/TextArea.qml (+9/-1)
src/Ubuntu/Components/1.3/TextField.qml (+12/-1)
src/Ubuntu/Components/plugin/ucabstractbutton.cpp (+1/-0)
src/Ubuntu/Components/plugin/ucstyleditembase.cpp (+10/-1)
src/Ubuntu/Components/plugin/ucstyleditembase.h (+9/-0)
tests/unit_x11/tst_components/tst_focus.qml (+4/-3)
tests/unit_x11/tst_components/tst_textinput_common13.qml (+14/-5)
To merge this branch: bzr merge lp://qastaging/~ubuntu-sdk-team/ubuntu-ui-toolkit/focusPassesFromChildToParent
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Zsombor Egri Approve
Review via email: mp+277962@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2015-11-18.

Commit message

Don't set activeFocusOnPress on TextField but on child only

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

A small comment, otherwise it's good to go from my side!!!

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1720. By Cris Dywan

More solid tests and more robust fix

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1721. By Cris Dywan

Add qdoc comment for activeFocusOnPress

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1722. By Cris Dywan

Consistently apply activeFocusOnTab everywhere

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1724. By Cris Dywan

Make the must_not_grab_focus test clearer

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1725. By Cris Dywan

Make the test actually work

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

Still like it :)

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

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