1708534 - Input text colour should be primary and not secondary colour in Edit Bookmarks panel

Status: VERIFIED FIXED

(Firefox :: Bookmarks & History, defect, P2)

Description

[Amy Lee [:amylee] UX]

Attachment: Screen Shot 2021-04-29 at 2.49.56 PM.png

Title text colour should be primary and not secondary colour in Edit Bookmarks panel when you input text in the text field. Currently it's set to secondary text colour in the Titles input field. When you type in "Tags" it should also be primary text colour.

Light them text - #15141A
Dark theme text - #FBFBFE

Comment 1

[Emma Malysz]

Attachment: Bug 1708534, increase spacing beneath labels in edit bookmark panel

Comment 2

[Phabricator Automation]

Comment on attachment 9221443 [details]
Bug 1708534, increase spacing beneath labels in edit bookmark panel

Revision D114903 was moved to bug 1708424. Setting attachment 9221443 [details] to obsolete.

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1708534 - Use the specified input colours for dark and light built-in themes. r?jaws!

It looks like we erroneously used the placeholder colour for the dark theme
input colour. This updates the dark theme to use the correct colour. It also
updates the light theme to use the same input colours as defined in the
light variation of the system theme.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/633b705d14eb
Use the specified input colours for dark and light built-in themes. r=jaws

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/633b705d14eb

Comment 6

[Romain Testard [:RT]]

Can we please uplift to Beta 89?

Comment 7

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 9221644 [details]
Bug 1708534 - Use the specified input colours for dark and light built-in themes. r?jaws!

Beta/Release Uplift Approval Request

• User impact if declined: Users will get slightly incorrect colours for input fields in the browser UI for the built-in light and dark themes.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Just updates some colour definitions in our theme manifests - very low risk.
• String changes made/needed: None.

Comment 8

[Pascal Chevrel:pascalc]

Comment on attachment 9221644 [details]
Bug 1708534 - Use the specified input colours for dark and light built-in themes. r?jaws!

Approved for 89 beta 15, thanks.

Comment 9

[Pascal Chevrel:pascalc]

https://hg.mozilla.org/releases/mozilla-beta/rev/938e3c0e9038

Comment 10

[Ciprian Georgiu, Desktop QA]

I've reproduced the bug using the info provided in comment 0, on an affected Nightly build (2021-04-29).

The issue is verified as fixed on the latest builds: Beta 89.0b15 and Nightly 90.0a1. Tested under macOS 10.15, Windows 10 x64 and Ubuntu 18.04 x64.