[Icons] X-Ubuntu-Splash-Color=#ffffff yields #f5f5f5

Bug #1449628 reported by Michael Terry
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Medium
Unassigned
Ubuntu UX
Fix Released
Medium
Jouni Helminen
unity8 (Ubuntu)
Fix Released
Undecided
Albert Astals Cid

Bug Description

I tried using a white-background app icon with a white-background splash color, but they didn't render the same. I took a screenshot and discovered that unity8 was rendering a white splash (#ffffff) as #f5f5f5.

Switching my app icon to use that same off white as its background fixed the visual glitch. But why does unity8 do that?

Related branches

Revision history for this message
Albert Astals Cid (aacid) wrote :

We're doing the same the SDK is doing, see MainViewStyle.qml that is a "copy" of MainViewStyle.qml of the Ambience theme (hurray not). There there is a Image of background_paper.png on top of the color, if you open background_paper.png you'll see it's basically adding some alpha, so it is "by design" that the color you see is not the one you specified.

Changed in unity8 (Ubuntu):
status: New → Invalid
Revision history for this message
Michael Terry (mterry) wrote :

Shouldn't the icon be below that paper background then?

I feel like from the app designer point of view, if they use #fff in both the "inputs" to Ubuntu Touch (X-Ubuntu-Splash-Color= and Icon=), when Ubuntu Touch puts the two together, they should look the same. Maybe they can be "papered over", but if so, both should be papered over.

Changed in unity8 (Ubuntu):
status: Invalid → New
Revision history for this message
Albert Astals Cid (aacid) wrote :

To be honest i'm undecided too.

I agree with you it's weird that the developer says "use this color" and we show a different one.

On the other hand it's the same case than in the MainView of the SDK is doing so it makes sense that if people are using a backgroundColor in their MainView they can use the same color in the splash and actually get the same color in the splash than in the MainView.

I agree it's weird that if you use the same color on the icon than in the splash background it is weird it looks different in the icon than in the background, but it on the other hand the icon is one of the things people are usually very strict about, so modifying the colors of people's icons is also not the best of the ideas :/

So basically we need some decision on wheter we should either:
 * Not change the background color specified by the developer with "papering" (and thus not matching what the SDK MainView does)
  * It's ok changing the colors of the icon of the application

Changed in ubuntu-ux:
assignee: nobody → Jouni Helminen (jounihelminen)
importance: Undecided → Medium
status: New → Triaged
Changed in ubuntu-ux:
assignee: Jouni Helminen (jounihelminen) → Matthieu James (tiheum)
summary: - X-Ubuntu-Splash-Color=#ffffff yields #f5f5f5
+ [Icons] X-Ubuntu-Splash-Color=#ffffff yields #f5f5f5
Revision history for this message
Albert Astals Cid (aacid) wrote :

unity8 waiting for design -> incomplete

Changed in unity8 (Ubuntu):
status: New → Incomplete
Matthieu James (tiheum)
Changed in ubuntu-ux:
assignee: Matthieu James (tiheum) → Jouni Helminen (jounihelminen)
Revision history for this message
Jouni Helminen (jounihelminen) wrote :

default background colour changed to white with Tim Peeters

Changed in ubuntu-ux:
status: Triaged → Fix Released
Changed in unity8 (Ubuntu):
status: Incomplete → Triaged
Changed in unity8 (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
status: Triaged → In Progress
Changed in canonical-devices-system-image:
status: New → Fix Committed
importance: Undecided → Medium
milestone: none → ww46-2015
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.11+16.04.20151104-0ubuntu1

---------------
unity8 (8.11+16.04.20151104-0ubuntu1) xenial; urgency=medium

  [ Albert Astals Cid ]
  * Add an Item that proxies for old and new audio roles (LP: #1493851)
  * CroppedImageMinimumSourceSize: Fix 'Binding loop detected for
    property "imageAspectRatio"'
  * Make the scope settings list scroll to text visible area (LP:
    #1499084)
  * Move to importing QtQuick 2.4 & friends
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Andrea Cimitan ]
  * Add sharing widget to zoomable image and video playback
  * Make zoomable and video playback widgets edge to edge
  * Updated all 1.3 UbuntuShape to use new APIs where possible
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ CI Train Bot ]
  * Resync trunk.

  [ Christopher Lee ]
  * Quick spelling fix in process control output.

  [ Daniel d'Andrada ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Josh Arenson ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)

  [ Michael Terry ]
  * Avoid showing the shutdown dialog when turning on the screen if your
    device is under heavy load. (LP: #1508563)
  * Handle lifecycle policy exceptions ourselves, instead of letting
    qtmir do it for us and allow non-Touch apps to opt-out of the Touch
    lifecycle.

  [ Michael Zanetti ]
  * Fixes for the panel buttons (LP: #1510360, #1504269, #1431566,
    #1443319)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Michał Sawicz ]
  * Add missing copyright to Cursor.qml
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

  [ Nick Dedekind ]
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

 -- Michał Sawicz <email address hidden> Wed, 04 Nov 2015 14:58:41 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
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.