Merge ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:report-keyboard-navigation into ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/master

Proposed by Sebastien Bacher
Status: Merged
Merged at revision: aaf7f6120ef8d0003bf4595d1416a54dbf343172
Proposed branch: ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:report-keyboard-navigation
Merge into: ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/master
Diff against target: 22 lines (+2/-2)
1 file modified
debian/patches/0001-Add-Ubuntu-mode-with-special-pages.patch (+2/-2)
Reviewer Review Type Date Requested Status
Andrea Azzarone (community) Approve
Review via email: mp+361873@code.launchpad.net

Commit message

Set the ubuntu-report yes/no controls to accept focus

Those options can't be changed by keyboard navigation otherwise

Description of the change

Set the ubuntu-report yes/no controls to accept focus

Those options can't be changed by keyboard navigation otherwise

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/patches/0001-Add-Ubuntu-mode-with-special-pages.patch b/debian/patches/0001-Add-Ubuntu-mode-with-special-pages.patch
index ddef978..f5f66d5 100644
--- a/debian/patches/0001-Add-Ubuntu-mode-with-special-pages.patch
+++ b/debian/patches/0001-Add-Ubuntu-mode-with-special-pages.patch
@@ -3295,7 +3295,7 @@ index 0000000..f8474a2
3295+ <child>3295+ <child>
3296+ <object class="GtkRadioButton" id="opt_in_radio">3296+ <object class="GtkRadioButton" id="opt_in_radio">
3297+ <property name="visible">True</property>3297+ <property name="visible">True</property>
3298+ <property name="can_focus">False</property>3298+ <property name="can_focus">True</property>
3299+ <property name="margin_top">18</property>3299+ <property name="margin_top">18</property>
3300+ <property name="xalign">0</property>3300+ <property name="xalign">0</property>
3301+ <property name="label" translatable="yes">Yes, send system info to Canonical</property>3301+ <property name="label" translatable="yes">Yes, send system info to Canonical</property>
@@ -3304,7 +3304,7 @@ index 0000000..f8474a2
3304+ <child>3304+ <child>
3305+ <object class="GtkRadioButton" id="opt_out_radio">3305+ <object class="GtkRadioButton" id="opt_out_radio">
3306+ <property name="visible">True</property>3306+ <property name="visible">True</property>
3307+ <property name="can_focus">False</property>3307+ <property name="can_focus">True</property>
3308+ <property name="margin_top">6</property>3308+ <property name="margin_top">6</property>
3309+ <property name="xalign">0</property>3309+ <property name="xalign">0</property>
3310+ <property name="label" translatable="yes">No, don't send system info</property>3310+ <property name="label" translatable="yes">No, don't send system info</property>

Subscribers

People subscribed via source and target branches

to all changes: