Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

PATCH: default value for CSS property "background-color"

#122

I bumped into this testing with a large website. It did (I forget the exact mechanism) (getComputedStyle" or something ...).backgroundColor.toLower() == ...

It didn't matter what backgroundColor was, as long as it was a string.

Commit http://github.com/client9/env-js/commit/a5e28767cac052aaa51165c19fc... did this to src/css/properties.js

@ - backgroundColor: null, + backgroundColor: 'rgb(0,0,0)', @

You may not like my choice of default. Modern browsers seem to use rbga(0,0,0,0) or transparent.
I'm happy with any string, including ''

And if you this default should be null (that's cool), see my next ticket.

thankee!

--nickg

Reported by nickg · March 20th, 2010 @ 03:54 AM

State: resolved
Milestone: none
Assigned to: nickg nickg

Activity

  1. nickg
    nickg
    • State changed from new to resolved
    • Assigned user changed from Thatcher to nickg

    March 21st, 2010 @ 12:04 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.