Closed Bug 1604917 Opened 5 years ago Closed 5 years ago

GeckoView removes leading zeros from colors in Web App Manifest

Categories

(GeckoView :: General, defect, P1)

73 Branch
Unspecified
All
defect

Tracking

(firefox73 fixed)

RESOLVED FIXED
mozilla73
Tracking Status
firefox73 --- fixed

People

(Reporter: tigeroakes, Assigned: snorp)

References

Details

(Whiteboard: [geckoview:m1912])

Attachments

(1 file)

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:73.0) Gecko/20100101 Firefox/73.0

Steps to reproduce:

Open https://app.starbucks.com as a PWA in Fenix Nightly

Actual results:

When opening the Starbucks PWA in Fenix, the theme color is not used.

GeckoView's onWebAppManifest provides a theme color of #A862 instead of #00A862, removing the leading zeros.

https://github.com/mozilla-mobile/fenix/issues/5585

Expected results:

Color values should get padded at the start with zeros so that 6-digit colors are given to Android Components/Fenix

Assignee: nobody → snorp
Priority: -- → P1
Whiteboard: [geckoview:m1912]
Attachment #9116823 - Attachment description: Bug 1604917 - Ensure WebManifest colors are expressed in #AARRGGBB format. → Bug 1604917 - Coerce WebManifest colors to rrggbbaa ?marcosc,jdescottes
Pushed by jwillcox@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e8adf9ababe0 Coerce WebManifest colors to rrggbbaa ?marcosc,jdescottes r=marcosc,jdescottes
Status: UNCONFIRMED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla73
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: