Checkbox

Merge lp:~sylvain-pineau/checkbox/fix-1503677 into lp:checkbox

  1. fix-1503677
  2. Merge into trunk
Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 4218
Merged at revision: 4223
Proposed branch: lp:~sylvain-pineau/checkbox/fix-1503677
Merge into: lp:checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
providers/plainbox-provider-checkbox/bin/key_test (+1/-1)
To merge this branch: bzr merge lp:~sylvain-pineau/checkbox/fix-1503677
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+285857@code.launchpad.net

Description of the change

Fixes the linked bug

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-checkbox/bin/key_test'
2--- providers/plainbox-provider-checkbox/bin/key_test 2015-10-13 04:17:53 +0000
3+++ providers/plainbox-provider-checkbox/bin/key_test 2016-02-12 11:27:29 +0000
4@@ -163,7 +163,7 @@
5 # Check for ESC key pressed
6 self.show_text(_("Test cancelled"))
7 self.quit()
8- elif code > 1 and code < 10:
9+ elif 1 < code < 10 and type(self) == CLIReporter:
10 # Check for number to skip
11 self.toggle_key(self.keys[code - 2])
12 else:

Subscribers

People subscribed via source and target branches