Merge lp://staging/~brendan-donegan/checkbox/bug1052506 into lp://staging/checkbox

Proposed by Brendan Donegan
Status: Merged
Merged at revision: 1692
Proposed branch: lp://staging/~brendan-donegan/checkbox/bug1052506
Merge into: lp://staging/checkbox
Diff against target: 57 lines (+9/-2)
3 files modified
debian/changelog (+3/-0)
qt/frontend/qtfront.cpp (+2/-1)
qt/frontend/qtfront.ui (+4/-1)
To merge this branch: bzr merge lp://staging/~brendan-donegan/checkbox/bug1052506
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+125257@code.staging.launchpad.net

Description of the change

Make the Continue button the default and give it focus when Checkbox is loaded. Also I renamed the UI widget from 'friendlyTestsButton' to 'continueButton' as the previous name made little sense.

To post a comment you must log in.
1679. By Marc Tardif

Merged fix to bug #1052601 by bladernr.

1680. By Marc Tardif

Merged fix to bug #868643 by bladernr.

1681. By Marc Tardif

Merged fix to bug #990538 by bladernr.

1682. By Marc Tardif

Merged fix to bug #1053010 by bladernr.

1683. By Marc Tardif

Merged fix to bug #1053032 by bladernr.

1684. By Marc Tardif

Merged fix to bug #946184 by bladernr.

1685. By Marc Tardif

Merged fix to bug #1053033 by sylvain-pineau.

1686. By Brendan Donegan

qt/frontend/qtfront.ui, qt/frontend/qtfront.cpp - set focus to Continue
button and make it the default so that it can be 'clicked' using Enter,
as well as renaming it to 'continueButton' (LP: #1052506)

1687. By Brendan Donegan

Add UIFe to the commit message.

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Tested with the Spacebar and Enter keys, works perfectly.

Thanks

review: Approve

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