[UIFe] Update ubiquity's Minimal Install page to match the spec

Bug #1758082 reported by Jeremy Bicha on 2018-03-22
20
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
Undecided
Unassigned

Bug Description

User Interface Freeze Exception Justification
---------------------------------------------
This is landing just a bit late because of the need for it to be reviewed before landing.

I was only aware of the spec on Tuesday and it took some time to figure out how to work on ubiquity.

This should improve the usability of the new minimal install option feature.

Change Description
------------------
This updates one page of ubiquity to more closely match the specification:

https://goo.gl/DqtvZL

Screenshots attached.

Testing Done
------------
1. I successfully built ubiquity.
2. I ran Ubuntu in Try Ubuntu mode.
3. I installed the new ubiquity* .debs.
4. I completed an install with the Normal install option.
5. I repeated steps 2-3 and completed a Minimal install.

6. I then did steps 2-3 with Ubuntu Kylin to test that the install worked correctly for a flavor that doesn't provide a Minimal install option yet. (Probably not a great choice since I don't speak Chinese…)

Testing Not Done
----------------
I did not test Kubuntu

Notifications to Lists
----------------------
https://lists.ubuntu.com/archives/ubuntu-doc/2018-March/020581.html
https://lists.ubuntu.com/archives/ubuntu-translators/2018-March/007442.html

Related branches

Jeremy Bicha (jbicha) on 2018-03-22
description: updated
Jeremy Bicha (jbicha) wrote :
Jeremy Bicha (jbicha) wrote :
Jeremy Bicha (jbicha) wrote :
description: updated
Jeremy Bicha (jbicha) on 2018-03-22
description: updated

On Thu, Mar 22, 2018 at 06:10:46PM -0000, Launchpad Bug Tracker wrote:
> You have been subscribed to a public bug by Jeremy Bicha (jbicha):
>
> User Interface Freeze Exception Justification
> ---------------------------------------------
> This is landing just a bit late because of the need for it to be reviewed before landing.
>
> I was only aware of the spec on Tuesday and it took some time to figure
> out how to figure out how to work on ubiquity.
>
> This should improve the usability of the new minimal install option
> feature.
>
> Change Description
> ------------------
> This updates one page of ubiquity to more closely match the specification:
>
> https://goo.gl/DqtvZL
>
> Screenshots attached.
>
> Testing Done
> ------------
> 1. I successfully built ubiquity.
> 2. I ran Ubuntu in Try Ubuntu mode.
> 3. I installed the new ubiquity* .debs.
> 4. I completed an install with the Normal install option.
> 5. I repeated steps 2-3 and completed a Minimal install.
>
> 6. I then did steps 2-3 with Ubuntu Kylin to test that the install
> worked correctly for a flavor that doesn't provide a Minimal install
> option yet. (Probably not a great choice since I don't speak Chinese…)

This looks much better, thanks. I'm OK from the release team's side as
long as docs/translations are too.

 status confirmed
 unsubscribe ubuntu-release

>
> Testing Not Done
> ----------------
> I did not test Kubuntu

It might be worthwhile pinging them to at least give the chance to
check. You could upload to a PPA, or we could block in proposed.

Cheers,

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

Changed in ubiquity (Ubuntu):
status: New → Confirmed
Gunnar Hjalmarsson (gunnarhj) wrote :

This sounds manageable from a translators POV, and the translators were notified on ubuntu-translators. (It doesn't affect the desktop guide.)

UIFe ok.

Rik Mills (rikmills) wrote :

KDE front end is unchanged.

Rik Mills (rikmills) wrote :

Apologies, not unchanged. Lacking the new radio boxes, and now with a much terser message.

Rik Mills (rikmills) wrote :

And from before

Rik Mills (rikmills) wrote :

With respect to Kubuntu, and testing the packages from Jeremy's PPA, this represents a regression in the KDE front end, without any on the benefits seen in the GTK one.

Simon Quigley (tsimonq2) wrote :

To echo what Rik said, with my Kubuntu hat on, NACK. Please fix the KDE frontend to reflect the same changes as in the GTK frontend.

Jeremy Bicha (jbicha) wrote :

Respectfully, I don't see an actual regression here. And I don't consider it my responsibility to tweak the Kubuntu installer.

If you want to use a radio button and extra headings like Ubuntu does, do it!
If you need a different normal install label since you keep a media player installed, add it!

For reference, my PPA is at
https://launchpad.net/~jbicha/+archive/ubuntu/temp-ubiquity/+packages

Simon Quigley (tsimonq2) wrote :

> Respectfully, I don't see an actual regression here. And I don't consider it my responsibility to tweak the Kubuntu installer.

Respectfully, if you're going to make a change and you're not willing to follow through with it in the whole codebase rather than just one part of it, I consider that a regression, regardless of which hat I wear. It's certainly not something that I would consider a priority for a release if you're only going to fix one frontend. Please, if you're introducing this change, all or nothing here.

Kubuntu still stands at a NACK.

I think this is way too late in the process to throw a regression at an LTS.

Why is this a regression? It is a regression because it changes the text in the Kubuntu installer to the point that the message is unclear to the average user what exactly a "Minimal install" is.

If you want to introduce something so late, especially for an LTS, there can't be regressions or if there are they need to be corrected by the original committer. Otherwise, the original commiter is essentially introducing regression with the response "deal with it".

In my opinion, this is unacceptable because it adds work onto flavours without necessary time to solve it as well as finishing the already planned work.

I think the overall idea could work well but making an exception for something that improves the install for one section of the codebase but adds a regression to another is not acceptable, especially for an LTS.

Iain Lane (laney) wrote :

OK, let's wait until at least the string is fixed in the KDE frontend or preferably the new UI is implemented there too.

Changed in ubiquity (Ubuntu):
status: Confirmed → Incomplete
Matthew Paul Thomas (mpt) wrote :

Seems like a simple way to avoid the Kubuntu regression would be to use new strings, rather than changed strings, for the minimal install radio button.

Separately from the Kubuntu problem, this looks like an improvement, thank you Jeremy. The only tweak I’d request is that while the radio button labels correctly are not complete sentences, captions should always be complete sentences with periods. (This is for consistency when captions are sometimes two or more sentences, for example the third-party software one.)

(BTW, neither <https://wiki.ubuntu.com/Bugs/Bug%20statuses> nor <https://wiki.ubuntu.com/FreezeExceptionProcess> gives “Not completely fixed yet” as a definition of “Incomplete”.)

Changed in ubiquity (Ubuntu):
status: Incomplete → In Progress
Adolfo Jayme (fitojb) wrote :

Re: the new strings, using "install" as a noun is not appropriate, IMHO.

Jeremy Bicha (jbicha) wrote :

Adolfo, thank you for your feedback.

As a native US English speaker, I think "install" is perfectly acceptable to use as a noun when referring to an installation of software. I don't think it would be an improvement to switch it back to "installation".

Jeremy Bicha (jbicha) wrote :

Matthew, thank you for your review.

I like the radio button descriptions without periods better, but I will defer to your preference at this point in our release cycle. I'll update the strings to include the periods on Monday (if it's still not been done) after we see what the Kubuntu developers come up with.

Adolfo Jayme (fitojb) wrote :

1) Wiktionary says it is informal and 2) it creates one more fuzzy string.

Matthew Paul Thomas (mpt) wrote :

All else being equal, I prefer whichever variant has fewest syllables. But Adolfo makes good points, thank you. Spec updated to use “Normal installation” and “Minimal installation”.

Jeremy Bicha (jbicha) wrote :

Merge proposal updated to incorporate KDE changes from Rik Mills, and incorporate the improvements suggested by Matthew Paul Thomas and Adolfo James.

I'm attaching an updated screenshot.

Jeremy Bicha (jbicha) on 2018-03-28
description: updated
Iain Lane (laney) wrote :

> (BTW, neither <https://wiki.ubuntu.com/Bugs/Bug%20statuses> nor <https://wiki.ubuntu.com/FreezeExceptionProcess> gives “Not completely fixed yet” as a definition of “Incomplete”.)

Thanks for your feedback. I'm on the release team and I wanted to make sure it was clear that the freeze exception request was not approved after that comment. I believe setting the bug report to Incomplete conveyed that.

Rik Mills (rikmills) wrote :

I have run through a couple of installs with Jememy's updated PPA packages, which incorporate the latest changes, and all seems good with the KDE/Qt front end from that.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubiquity - 18.04.4

---------------
ubiquity (18.04.4) bionic; urgency=medium

  [ Mathieu Trudel-Lapierre ]
  * debian/control: have ubiquity-frontend-gtk Depends: explicitly on
    sensible-utils, which is used to start a browser when slideshow or
    release notes links are clicked. (LP: #1724224)
  * Automatic update of included source packages: netcfg 1.142ubuntu6,
    partman-efi 71ubuntu2, user-setup 1.63ubuntu5.

  [ Sebastien Bacher ]
  * gui/gtk/stepPrepare.ui: updated the non-free-software text to not
    include the mention to the MP3 fluendo codecs since those have
    been superseeded in the recent gstreamer version.
  * Rename the "Preparing to install Ubuntu" installer step to "Updates and
    other software" according to the design, thanks Matthew for pointing
    out the inconsistency

  [ Didier Roche ]
  * Collect and dump telemetry info as a log file: (LP: #1755456)
    Those installation telemetry info are for optional upload later on.

  [ Jeremy Bicha ]
  * Update "Updates and Other Software" page to match the spec (LP: #1758082)

  [ Rik Mills ]
  * Update KDE/Qt "Updates and Other Software" page in line with GTK changes,
    with some tweaks appropriate for Kubuntu usage in Bionic
  * Mark the sddm.conf file for OEM so that it can be removed by oem-config
    if unchanged.

 -- Mathieu Trudel-Lapierre <email address hidden> Thu, 29 Mar 2018 13:42:51 -0400

Changed in ubiquity (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers