Merge lp:~oem-qa/checkbox/patch_next_button_default into lp:checkbox
- patch_next_button_default
- Merge into trunk
Status: | Merged |
---|---|
Merged at revision: | 964 |
Proposed branch: | lp:~oem-qa/checkbox/patch_next_button_default |
Merge into: | lp:checkbox |
Diff against target: |
20 lines (+2/-1) 1 file modified
checkbox_gtk/gtk_interface.py (+2/-1) |
To merge this branch: | bzr merge lp:~oem-qa/checkbox/patch_next_button_default |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel Manrique (community) | Approve | ||
Review via email: mp+38871@code.launchpad.net |
Commit message
Description of the change
Set default response id to NEXT
Next button (or any other button that returns checkbox.
- 859. By Marc Tardif
-
Merged from cli_no_answer branch.
- 860. By Marc Tardif
-
Changed connection request back to use the full url instead rather than the path.
- 861. By Marc Tardif
-
Replaced external plugin with remote plugin in autotest and ltp scripts.
- 862. By Marc Tardif
-
Merged from testsprint-
checkbox- base-sru- changes.
Javier Collado (javier.collado) wrote : | # |
- 863. By Marc Tardif
-
Updated parsing of config parameters which fixes bug #689140.
- 864. By Marc Tardif
-
Merged from trunk.
- 865. By Marc Tardif
-
Merged from whitelist_
file_comments_ support. - 866. By Marc Tardif
-
Fixed jobs_info plugin to strip commented lines in whitelist and blacklist files which might start with a space.
- 867. By Marc Tardif
-
Added additional logging to the reactor when firing messages.
- 868. By Marc Tardif
-
Merged from checkbox-
add-pm- utils-requireme nt branch. - 869. By Marc Tardif
-
Added support for TOUCH devices.
- 870. By Jeff Lane
-
Merged cr3s fix for bug #561816
- 871. By Marc Tardif
-
Merged from bladernr to add _attachment suffix and lsmod attachment.
- 872. By Marc Tardif
-
Updated pot file.
- 873. By Marc Tardif
-
Fixed memory persistence to be rooted like file persistence.
- 874. By Marc Tardif
-
Fixed persist module to support not being given a filename.
- 875. By Marc Tardif
-
Extended persist_info plugin to answer to both begin and prompt-begin messages.
- 876. By Marc Tardif
-
Added support for Python 2.5 in checkbox.
lib.transport. - 877. By Marc Tardif
-
Merged from audio_test_failing branch.
- 878. By Marc Tardif
-
Merged from syslog branch.
- 879. By Marc Tardif
-
Added stop signal when executing messages.
- 880. By Marc Tardif
-
Migrated UI from libglade to gtkbuilder which fixes bug #403534.
- 881. By Marc Tardif
-
Defining default options in checkbox.
application rather than CHECKBOX_OPTIONS environment variable. - 882. By Marc Tardif
-
Added changelog entry for candidate revision.
- 883. By Marc Tardif
-
Merged from 719552.
- 884. By Marc Tardif
-
Changed description of nautilus_file_copy job which fixes bug #709688.
- 885. By Marc Tardif
-
Fixed title in progress dialog.
- 886. By Marc Tardif
-
Updated changelog with new upstream release.
- 887. By Marc Tardif
-
Updated pot file.
- 888. By Marc Tardif
-
Merged from 727411.
- 889. By Marc Tardif
-
Merged from 691241.
- 890. By Marc Tardif
-
Added support for trying to submit twice which fixes bug #531010.
- 891. By Marc Tardif
-
Merged from 642001.
- 892. By Jeff Lane
-
Merged cli-cleanup for bug #221400
- 893. By Marc Tardif
-
Merged from checkbox-bug-fixes.
- 894. By Jeff Lane
-
Land translation work by Mahyuddin Susanto via Michael Terry
- 895. By Jeff Lane
-
Merged cr3 changes to changelog and control
- 896. By Marc Tardif
-
Added changelog entry for candidate revision.
- 897. By Jeff Lane
-
Commited Gerhard Burgers punctuation fix for LP #744167
- 898. By Marc Tardif
-
Merged from 553777.
- 899. By Marc Tardif
-
Moved Gerhard Burger in changelog from 0.11.2 to 0.11.3.
- 900. By Marc Tardif
-
Fixed missing capital letter in sleep_state_test description.
- 901. By Marc Tardif
-
Merged from 744964.
- 902. By Jeff Lane
-
Merged Marc Tardif fix for lp:729431
- 903. By Jeff Lane
-
Merged Marc Tardif fixes to hibernate test for lp:630785
- 904. By Jeff Lane
-
Merged addition of rtc test to sleep.txt.in to meet dependencies
- 905. By Marc Tardif
-
Reintroduced pm-utils requirements for jobs calling the sleep_test command and update po files.
- 906. By Marc Tardif
-
Merged checkbox log to apport report.
- 907. By Marc Tardif
-
Added changelog entry for candidate revision.
- 908. By Marc Tardif
-
Merged branch to enable camera detect test.
- 909. By Jeff Lane
-
Merged Carl Milettes fix for lp:507943 (disk_bench_test hard coded drive)
- 910. By Marc Tardif
-
Fixed eval of resources with names like list item names.
- 911. By Marc Tardif
-
Merged from pygi-gtk3-port.
- 912. By Marc Tardif
-
Removed dead pixel test.
- 913. By Marc Tardif
-
Merged from 773667.
- 914. By Marc Tardif
-
Merged from 776734.
- 915. By Marc Tardif
-
Merged from 786924.
- 916. By Marc Tardif
-
Merged from 776712.
- 917. By Jeff Lane
-
Applied Marc Tardifs changes to allow for remote submission (send results from a system other than the system under test)
- 918. By Marc Tardif
-
Merged from 588539.
- 919. By Marc Tardif
-
Merged from 665299.
- 920. By Marc Tardif
-
Wrapped submission stream to check for illegal xml characters.
- 921. By Marc Tardif
-
Replaced dependency on pytz with dateutil.
- 922. By Marc Tardif
-
Merged from 621880.
- 923. By Marc Tardif
-
Merged from add_camera_tests.
- 924. By Marc Tardif
-
Merged from fix_get_
pointer_ error. - 925. By Marc Tardif
-
Incremented version.
- 926. By Marc Tardif
-
Merged from 798200.
- 927. By Marc Tardif
-
Merged from patch_resourceo
bject_contains. - 928. By Marc Tardif
-
Merged from checkbox-
fix-pygi- misbehaviors. - 929. By Marc Tardif
-
Merged from 744190.
- 930. By Marc Tardif
-
Merged from add_interface_
option_ for_multi_ nic_test. - 931. By Marc Tardif
-
Merged from test-name-
in-report- a-bug. - 932. By Marc Tardif
-
Incremented version.
- 933. By Marc Tardif
-
Only reading CHECKBOX_* environment variables in config which fixes bug #802458.
- 934. By Marc Tardif
-
Added preliminary default.whitelist.
- 935. By Marc Tardif
-
Imported scripts and jobs from Platform Services.
- 936. By Daniel Manrique
-
merge patch_log_
format_ object - 937. By Marc Tardif
-
Imported plugins from Platform Services.
- 938. By Daniel Manrique
-
merged patch_apport_
interface_ yesno - 939. By Marc Tardif
-
Merged switch to dh_python2 and debhelper7 which fixes bug #788514.
- 940. By Brendan Donegan
-
Removed some tests that shouldn't have appeared in the whitelist.
- 941. By Marc Tardif
-
Merged from improve_
failed_ test_message. - 942. By Daniel Manrique
-
merged Barry Warsaw's fix for problem with non-existing executables while building from clean tree
- 943. By Marc Tardif
-
Merged from camera_
test_grouping. - 944. By Marc Tardif
-
Merged from story325_
test_organisati on. - 945. By Marc Tardif
-
Updated pot file.
- 946. By Daniel Manrique
-
merged missing_suites branch
- 947. By Daniel Manrique
-
Fixed typo in local job definition which caused exception and test to not be run
- 948. By Daniel Manrique
-
merge story221_
move_30_ cycles_ to_stress - 949. By Daniel Manrique
-
merged fixup_powermana
gement_ suspends - 950. By Daniel Manrique
-
merged story221_
cpu_before_ after_suspend - 951. By Daniel Manrique
-
merge patch_exit_
on_whitelist_ file_not_ found - 952. By Daniel Manrique
-
merged patch_display_
text_in_ show_tree_ method - 953. By Daniel Manrique
-
merged from story221_
memory_ suspend - 954. By Marc Tardif
-
Merged from fix-backend-
protocol. - 955. By Marc Tardif
-
Merged from 804369-808423.
- 956. By Daniel Manrique
-
Change mention of OpenOffice to LibreOffice (thanks to David Baucum)
Daniel Manrique (roadmr) wrote : | # |
Hi Javier,
I tested this code and it works well with the latest Checkbox trunk (tested on Oneiric). I agree that it might save time, although I personally am used to just pressing "alt-x" which is only slightly more difficult than pressing enter.
My only, usability-related, concern with this is that if a trigger-happy user presses Enter without reading, he might skip or otherwise miss important information. This is mitigated by the fact that checkbox can now pretty reliably go back and rerun tests.
Still I'd like to get a bit of feedback from other testers on this, so I'll ask them to pitch in with comments, and once we know how other people feel, we can make a decision. If you know of some checkbox users who might want to comment, please also ask them to comment here.
- 957. By Launchpad Translations on behalf of checkbox-dev
-
Launchpad automatic translations update.
Daniel Manrique (roadmr) wrote : | # |
Approved and merged, per positive input from two of our testers (incidentally it's all the input I got, positive or negative).
Thanks!
Javier Collado (javier.collado) wrote : | # |
Sorry, I forgot about calling for feedback. I'm glad that the change was finally merged.
- 958. By Javier Collado
-
Set default response id to NEXT
Preview Diff
1 | === modified file 'checkbox_gtk/gtk_interface.py' |
2 | --- checkbox_gtk/gtk_interface.py 2011-05-17 13:47:10 +0000 |
3 | +++ checkbox_gtk/gtk_interface.py 2011-06-08 14:57:36 +0000 |
4 | @@ -202,6 +202,7 @@ |
5 | |
6 | dialog = dialog or self._dialog |
7 | dialog.connect("response", on_dialog_response, self) |
8 | + dialog.set_default_response(NEXT) |
9 | dialog.show() |
10 | Gtk.main() |
11 | if self.direction == Gtk.ResponseType.DELETE_EVENT: |
12 | @@ -518,7 +519,7 @@ |
13 | message_format=text) |
14 | message_dialog.set_modal(True) |
15 | message_dialog.set_title(_("Error")) |
16 | - message_dialog.set_default_response(1) |
17 | + message_dialog.set_default_response(NEXT) |
18 | message_dialog.add_buttons(Gtk.STOCK_CLOSE, NEXT) |
19 | self._run_dialog(message_dialog) |
20 | message_dialog.hide() |
Please let me know if there's any reason that makes this change a bad proposal. As explained in the description, defaulting to next button it's useful to save time.