Merge lp:~ochosi/ubiquity/partition_spinbutton into lp:ubiquity

Proposed by Simon Steinbeiß
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: 6483
Merged at revision: 6483
Proposed branch: lp:~ochosi/ubiquity/partition_spinbutton
Merge into: lp:ubiquity
Diff against target: 13 lines (+2/-1)
1 file modified
gui/gtk/stepPartAdvanced.ui (+2/-1)
To merge this branch: bzr merge lp:~ochosi/ubiquity/partition_spinbutton
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Approve
Review via email: mp+305534@code.launchpad.net

Description of the change

This (dead-simple) patch is aimed at resolving this bug: https://bugs.launchpad.net/ubuntu/+source/ubiquity/+bug/1622258

The gist is that the current width of the partition_size_spinbutton widget in Ubiquity's stepPartAdvanced.ui is set to 6 characters, which is barely enough for displaying GB values in MB. If people have larger drives they will always have to scroll, which may lead to errors and is generally tedious in such a small input box.

I consequently suggest bumping the width_chars value from 6 to 10 (Petabyte) and setting the "hexpand" option, which means the spinbutton will occupy all remaining horizontal space.

To post a comment you must log in.
Revision history for this message
Simon Steinbeiß (ochosi) wrote :

Note that this also restores what we saw in 16.04 (i.e. the spinbutton expanding horizontally).

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'gui/gtk/stepPartAdvanced.ui'
2--- gui/gtk/stepPartAdvanced.ui 2014-03-13 06:23:07 +0000
3+++ gui/gtk/stepPartAdvanced.ui 2016-09-12 21:45:37 +0000
4@@ -193,7 +193,8 @@
5 <property name="can_focus">True</property>
6 <property name="invisible_char">●</property>
7 <property name="activates_default">True</property>
8- <property name="width_chars">6</property>
9+ <property name="hexpand">True</property>
10+ <property name="width_chars">10</property>
11 <property name="xalign">1</property>
12 <property name="invisible_char_set">True</property>
13 <property name="primary_icon_activatable">False</property>

Subscribers

People subscribed via source and target branches

to status/vote changes: