Featured snaps doesn't align well when part page

Bug #1769810 reported by Robert Ancell
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gnome-initial-setup (Ubuntu)
Fix Released
Medium
Robert Ancell
Bionic
Fix Released
Medium
Robert Ancell
Cosmic
Fix Released
Medium
Robert Ancell

Bug Description

[Impact]
When there is not a full page of featured apps they don't align to the grid.

[Test Case]
1. Run initial setup:
$ /usr/lib/gnome-initial-setup/gnome-initial-setup --existing-user
2. Go to the "Ready to go" page
3. Scroll to the last page in the featured apps

Expected result:
The last page is aligned the same as the other page(s), even if it doesn't have a full 15 apps to show.

Observed result:
The apps are spread out.

[Regression Potential]
Solved by adding spacer elements to fill out page. Could cause other bugs.

Revision history for this message
Robert Ancell (robert-ancell) wrote :

Screenshot of issue. Note that this depends on how many snaps are being featured at the time of testing.

Changed in gnome-initial-setup (Ubuntu Cosmic):
status: New → Fix Committed
assignee: nobody → Robert Ancell (robert-ancell)
Changed in gnome-initial-setup (Ubuntu Bionic):
importance: Undecided → Medium
Changed in gnome-initial-setup (Ubuntu Cosmic):
importance: Undecided → Medium
Changed in gnome-initial-setup (Ubuntu Bionic):
status: New → Triaged
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-initial-setup - 3.28.0-2ubuntu8

---------------
gnome-initial-setup (3.28.0-2ubuntu8) cosmic; urgency=medium

  * debian/patches/0001-Add-Ubuntu-mode-with-special-pages.patch:
    - Remove some dead code
    - Fix alignment of featured snaps (LP: #1769810)
    - Update background images to be more transparent (LP: #1769811)
    - Make changes page translatable (LP: #1766277)
    - Fix crash handling errors in livepatch page (LP: #1768743)
  * debian/patches/0001-Ensure-stamp-file-if-the-user-quit-the-wizard.patch:
    - Split out of 0001-Add-Ubuntu-mode-with-special-pages.patch

 -- Robert Ancell <email address hidden> Wed, 09 May 2018 16:08:27 +1200

Changed in gnome-initial-setup (Ubuntu Cosmic):
status: Fix Committed → Fix Released
Iain Lane (laney)
Changed in gnome-initial-setup (Ubuntu Bionic):
assignee: nobody → Robert Ancell (robert-ancell)
Changed in gnome-initial-setup (Ubuntu Bionic):
status: Triaged → Fix Committed
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Robert, or anyone else affected,

Accepted gnome-initial-setup into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/gnome-initial-setup/3.28.0-2ubuntu6.16.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-needed verification-needed-bionic
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Confirmed 3.28.0-2ubuntu6.16.04.1 correctly lays out featured snaps.

tags: added: erification-done-bionic
removed: verification-needed verification-needed-bionic
tags: added: verification-done-bionic
removed: erification-done-bionic
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-initial-setup - 3.28.0-2ubuntu6.16.04.1

---------------
gnome-initial-setup (3.28.0-2ubuntu6.16.04.1) bionic; urgency=medium

  * debian/patches/0001-Add-Ubuntu-mode-with-special-pages.patch:
    - Add legal notice for livepatch page (LP: #1773140)
    - Typo fix in the "help improve ubuntu" page (lp: #1768549)
    - Fix crash if fail to get installed snaps (LP: #1768744)
    - Fix alignment of featured snaps (LP: #1769810)
    - Update background images to be more transparent (LP: #1769811)
    - Make changes page translatable (LP: #1766277)
    - Correctly update the "What's new" graphic (LP: #1768557)
    - Fix crash handling errors in livepatch page (LP: #1768743)
    - Remove some dead code
  * debian/patches/0001-Ensure-stamp-file-if-the-user-quit-the-wizard.patch:
    - Split out of 0001-Add-Ubuntu-mode-with-special-pages.patch

 -- Robert Ancell <email address hidden> Thu, 31 May 2018 17:00:47 +1200

Changed in gnome-initial-setup (Ubuntu Bionic):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

The verification of the Stable Release Update for gnome-initial-setup has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

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.