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

We need a new Favicon #2171

Closed
SoftVision-CosminMuntean opened this issue Feb 15, 2017 · 5 comments
Closed

We need a new Favicon #2171

SoftVision-CosminMuntean opened this issue Feb 15, 2017 · 5 comments
Assignees
Milestone

Comments

@SoftVision-CosminMuntean

[Notes]:

  • The Page Shot tab favicon is also hardly visible on Firefox High Contrast themes and other 3rd party darker themes.
  • On specific themes the Page Shot favicon is hardly visible even if the tab is not focused.

[Affected versions]:

  • Aurora 53.0a2

[Affected Platforms]:

  • All Windows
  • All Mac

[Prerequisites]:

[Steps to reproduce]:

  1. Open the Developer Edition bowser with the profile from prerequisite and navigate to any website.
  2. Click the Page Shot button from the browser toolbar.
  3. Perform a selection an save it.
  4. Observe the Page Shot tab favicon.

[Expected result]:

  • The Page Shot favicon is perfectly visible.

[Actual result]:

  • The Page Shot favicon is hardly visible.

[Additional Notes]:

  • Attached a screen shot with the issue:

untitled

@ianb ianb added this to the Page Shot in 54 milestone Feb 15, 2017
@ianb ianb modified the milestones: Page Shot in 54, Sprint 54.2 Mar 9, 2017
@dannycoates
Copy link
Contributor

dannycoates commented Mar 15, 2017

We might be able to use the browser_style thing for this. https://developer.mozilla.org/en-US/Add-ons/WebExtensions/manifest.json/browser_action

(I don't know if it only affects a panel or the button too)

@pdehaan
Copy link
Collaborator

pdehaan commented Mar 15, 2017

I think @ianb added the browser_style thing to addon/webextension/manifest.json in #2377:

"browser_style": false

Although the I believe false is the default value. So sayeth the docs:

Although this key defaults to false, it's recommended that you include it and set it to true.

@ianb
Copy link
Contributor

ianb commented Mar 15, 2017

browser_style won't do anything, I'm afraid, it just applies to panels. There isn't a panel, so I added it just to make a warning shut up.

Note though that this ticket is for the favicon, not the toolbar, so this is entirely a website issue. Maybe we want a background on our favicon?

@ghost ghost modified the milestones: Sprint 54.2, Sprint 54.3 Mar 20, 2017
@jaredhirsch
Copy link
Member

If this bug is about the colors in the favicon, that doesn't seem like a beta blocker. Removing that label; feel free to re-add it if I'm missing something.

@ghost ghost modified the milestones: Sprint 54.3, Sprint 54.4 Mar 31, 2017
@ghost ghost modified the milestones: Sprint 54.4, Sprint 54.5 Apr 17, 2017
@SoftVision-CosminMuntean
Copy link
Author

Since the new Firefox Compact themes will enter in Firefox 54 release, I have also checked how the Screenshots tab favicon behaves on them. It seems that we have the same issue described above, using "Compact Dark" theme.

Here is a screenshot with Screenshots tab favicon on Compact Dark theme:
favicon

@ghost ghost modified the milestones: Sprint 54.5, Sprint 54.6 May 1, 2017
@ghost ghost assigned johngruen May 1, 2017
@ghost ghost removed this from the Sprint 54.6 milestone May 12, 2017
@ghost ghost modified the milestones: Screenshots in 55, Sprint 54.6 May 12, 2017
@johngruen johngruen changed the title Page Shot tab favicon is hardly visible on darker themes if the tab is focused We need a new Favicon Jun 12, 2017
@johngruen johngruen modified the milestones: 55.1, General Release 55 Jun 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants