-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Conversation
Deploy preview for web-dev-staging ready! Built with commit f26692f |
@lilles FYI, the |
Co-Authored-By: Joe Medley <[email protected]>
Thanks Thomas! A couple of notes: Do we have a crbug report for the link colors? We shouldn't break accessibility requirements for the default rendering of:
Since the UA color changes for system colors, forms, scrollbars etc. has not shipped yet. Did you plan to publish this before that ships? If so, it should have a note about that? |
Meta issue in the HTML Standard
Content on web.dev is not tied to Chrome, we write about general web features. Ideally it'd work in Chrome at the time of publication, but it doesn't have to. |
Co-Authored-By: Joe Medley <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for the great content @tomayac. Notes:
|
@kaycebasques Thanks, Kayce, for the tweaks.
I've opened #2532 to push it live. Are there comms from the team who should flip the
Well aware, just in this case the embed doesn't work to convey the message due to iframe styling issues (w3c/csswg-drafts#4772).
I have tweaked the title a bit #2532, the "clickbait" is improved dark mode default styling, I think that works. |
Fixes #2265
Changes proposed in this pull request: