Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lighthouse 2.0.0 FF issues #2320

Closed
XhmikosR opened this issue May 20, 2017 · 8 comments
Closed

Lighthouse 2.0.0 FF issues #2320

XhmikosR opened this issue May 20, 2017 · 8 comments

Comments

@XhmikosR
Copy link
Contributor

I'm using Firefox 54 beta on Windows 7 64-bit

  1. Some of the new SVG icons don't show up here
  2. The score on the sidebar shows NaN

screen shot 2017-05-20 at 20 01 33-fullpage

@patrickhulce
Copy link
Collaborator

re-opening for the second half of this issue that wasn't addressed by #2361 "Some of the new SVG icons don't show up here"

@patrickhulce patrickhulce reopened this May 25, 2017
@evenstensberg
Copy link
Contributor

I'll pick it up tomorrow, sorry!

@evenstensberg
Copy link
Contributor

Last item to fix is the bugs due to details and summary in firefox, where chrome uses ::-webkit-details-marker to completely forget about this issue

@XhmikosR
Copy link
Contributor Author

@ev1stensberg: can't we do what the MDN article suggests? https://developer.mozilla.org/en/docs/Web/HTML/Element/details

@evenstensberg
Copy link
Contributor

I was actually tempted to correct the manual audits class in <summary> to be compliant to the passed audits, in order to at least make the colors depend, instead of implicitly. Regardless, this is an issue with ::-webkit-details-marker and list-item styling that differs in chrome from FF.
skjermbilde 2017-05-29 kl 15 29 38
skjermbilde 2017-05-29 kl 15 29 26

@evenstensberg
Copy link
Contributor

Actually, let me take the first things I said back. We use the psudo-element to style. In firefox, doesn't seem to work cause display doesn't get updated / ::-webkit-details-marker doesn't apply the marker changes, as we've disabled the native bullet point/arrow. This is a hack ❌

@evenstensberg
Copy link
Contributor

Could we move this to #2385 ?

@XhmikosR
Copy link
Contributor Author

XhmikosR commented May 30, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants