Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

The "Firefox Notes" toolbar button is not visible when the browser's dark theme is activated #583

Closed
Softvision-MariusComan opened this issue Jan 16, 2018 · 8 comments
Labels
bug Firefox Bug Firefox changes required to implement [QA]:Verified fixed Label for QA to mark verified fixed issues

Comments

@Softvision-MariusComan
Copy link

[Affected versions]:

  • Firefox 57.0.4 and up

[Affected Platforms]:

  • All Windows
  • All Mac
  • All Linux

[Prerequisites]:

  • Have a Firefox profile with the latest version of the "Firefox Notes" add-on (2.2.0dev custom build from 01.16.2018) installed.
  • Have the "Dark" theme enabled from about:addons->Themes.

[Steps to reproduce]:

  1. Open the browser with the profile from prerequisites.
  2. Observe the "Firefox Notes" toolbar button.

[Expected result]:

  • The button is visible.

[Actual result]:

  • The button blends in with the background.

[Notes]:

  • The issue is not reproducible with the TxP version of "Firefox Notes".
  • Attached a screen recording of the issue:
    rec of the issue
@Natim
Copy link
Collaborator

Natim commented Jan 17, 2018

When we merged it it was looking like that:

@Natim
Copy link
Collaborator

Natim commented Jan 18, 2018

I can reproduce on Linux but we can't on Mac (we tried with release/beta and nightly versions) I am suspecting a Firefox bug.

@Natim Natim added the Firefox Bug Firefox changes required to implement label Jan 18, 2018
@Natim
Copy link
Collaborator

Natim commented Jan 18, 2018

We can reproduce on Windows.

@Natim
Copy link
Collaborator

Natim commented Jan 18, 2018

It seems to work only on retina displays...

@sebastienbarbier
Copy link
Collaborator

Could solve this issue by adding in manifest "theme_icons" size 16 in the list. Will push a fix soon.

@sebastienbarbier sebastienbarbier added Firefox Bug Firefox changes required to implement and removed Firefox Bug Firefox changes required to implement labels Jan 18, 2018
@Natim Natim closed this as completed in 59b4209 Jan 18, 2018
@Natim
Copy link
Collaborator

Natim commented Jan 18, 2018

@Natim Natim reopened this Jan 18, 2018
@Natim Natim closed this as completed Jan 18, 2018
@Natim
Copy link
Collaborator

Natim commented Jan 18, 2018

@Softvision-MariusComan
Copy link
Author

The issue is no longer reproducible with the latest version of the "Firefox Notes" add-on (2.2.0dev custom build from 01.25.2018) installed, on the latest Firefox release and Nightly.

@Softvision-MariusComan Softvision-MariusComan added the [QA]:Verified fixed Label for QA to mark verified fixed issues label Jan 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Firefox Bug Firefox changes required to implement [QA]:Verified fixed Label for QA to mark verified fixed issues
Projects
None yet
Development

No branches or pull requests

3 participants