Closed Bug 1703898 Opened 3 years ago Closed 3 years ago

Proton: External theme differences in search bar in toolbar

Categories

(Firefox :: Address Bar, defect, P1)

defect

Tracking

()

VERIFIED FIXED
89 Branch
Tracking Status
firefox87 --- unaffected
firefox88 --- unaffected
firefox89 --- verified

People

(Reporter: muirpablo, Assigned: mak)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [proton-address-bar] [priority:2a])

Attachments

(2 files)

Attached image lake.jpg

Affected versions

Nightly 89.0a1

tested on platforms

Win 7 64-bit, Ubuntu 20. windows 10 64bit

Preconditions
user.js with
user_pref("browser.proton.enabled", true);
user_pref("browser.proton.toolbar.enabled", true);
user_pref("browser.proton.urlbar.enabled", true);

themes to use
https://addons.mozilla.org/es/firefox/addon/northern-lake-by-madonna/?utm_source=addons.mozilla.org&utm_medium=referral&utm_content=search

https://addons.mozilla.org/es/firefox/addon/purple-pastel-cloud-aesthetic/?utm_source=addons.mozilla.org&utm_medium=referral&utm_content=search

Steps to reproduce

  1. launch firefox
  2. check and install both themes
  3. enable search bar in toolbar ( in settings > search)
  4. type something in search bar in toolbar

Expected result
theme background should match release

Actual result
background color is different
in northern lake its almost all black
in pastel theme, its showing white

Hi romain, could you please set a priority for this here?

Severity: -- → S3
Flags: needinfo?(rtestard)
Whiteboard: [proton-address-bar]

the search bar should follow the urlbar panel colors more or less. that text seems problematic though

Flags: needinfo?(rtestard)
Priority: -- → P2
Whiteboard: [proton-address-bar] → [proton-address-bar] [priority:2a]

The second theme is correct imo, it follows perfectly the urlbar panel colors.

The problem here is the first theme being unreadable.

Assignee: nobody → mak
Status: NEW → ASSIGNED
Priority: P2 → P1
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/6799ad273be4
Fix search bar panel text color with themes. r=desktop-theme-reviewers,dao
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

verified fix on windows10 64bit, ubuntu 20 and MacOs 10.15 using firefox nightly 89.0a1

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: