Installer has two Step 1s, two Step 3s, and no Step 2

Bug #336740 reported by Matthew Paul Thomas
2
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: ubiquity

Ubuntu Jaunty alpha 5

1. Launch the installer.
2. At the introductory warning, choose "Forward".
3. At the "Welcome" step, choose "Forward".
4. At the "Where are you?" step, choose "Back".

What happens:
1. The introductory warning is labelled "Step 1".
2. The "Welcome" step is also labelled "Step 1".
3. The "Where are you?" step is labelled "Step 3".
4. The "Welcome" step is now also labelled "Step 3".

What should happen, if the step numbers are retained (see also bug 274164):
1. The introductory warning should not have its own step number.
2. The "Welcome" step should be "Step 1".
3. The "Where are you?" step should be "Step 2".
2. The "Welcome" step should be "Step 1".

Evan (ev)
Changed in ubiquity:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Evan (ev) wrote :

This appears to have been caused by the fix for bug 225876 (as attached). When the back button is pressed from the "Where are you?" page and on_steps_switch_page is called, the "step ? of ?" text is not updated because the code assumes it will be done when the UI is translated when the selection in the language list is changed, but since the user is pressing back, this doesn't immediately happen. This is evidenced by selecting a language from the list, which updates the text to the correct page number.

Revision history for this message
Evan (ev) wrote :

I have tested to make sure this does not break the fix for bug 225876.

Changed in ubiquity:
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubiquity - 1.11.15

---------------
ubiquity (1.11.15) jaunty; urgency=low

  [ Colin Watson ]
  * Expand ubiquity/text/password_extra_label to explain what makes a good
    password (text mostly borrowed from user-setup).
  * Cope with renaming of passwd/allow-password-empty to
    user-setup/allow-password-empty in user-setup 1.23ubuntu14.
  * Remove obsolete ubiquity/text/timezone_country_label and
    ubiquity/text/timezone_time_label templates.

  [ Roman Shtylman ]
  * moved common pixmaps to artwork package

  [ Evan Dandrea ]
  * Always return an integer from get_size as the calcuation could produce a
    float, and functions using the return value expect a number of bytes
    (LP: #334677).
  * Use better descriptions on the finished dialog and show an information
    icon to be consistent with other dialogs. Thanks Matthew Paul Thomas
    (LP: #258017).
  * Properly update the step number when pressing back to get to the language
    selection page (LP: #336740).
  * Automatic update of included source packages: apt-setup
    1:0.37ubuntu10, console-setup 1.28ubuntu6, debian-installer-utils
    1.66ubuntu1, hw-detect 1.71ubuntu5, kboot-installer 0.0.1ubuntu9,
    localechooser 2.09ubuntu2, partman-auto-loop 0ubuntu16, partman-base
    129ubuntu1, partman-basicfilesystems 62ubuntu2, partman-partitioning
    64ubuntu4, user-setup 1.23ubuntu14.

  [ Mario Limonciello ]
  * Remove dead AdditionalDrivers code.
  * Create a SQL, Mythweb & VNC password based on your initial user password.
  * Make the extra passwords page only show up if selecting !Master.
  * Change the method for mapping role to package removal to something
    more scalable.
  * bin/ubiquity-dm:
    - When checking for whether the X server failed to start, poll() the subprocess
      object to look for bad return codes.

 -- Evan Dandrea <email address hidden> Thu, 05 Mar 2009 09:04:28 +0000

Changed in ubiquity:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.