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

Replace f4 and f47 #4166

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Replace f4 and f47 #4166

wants to merge 9 commits into from

Conversation

fstrr
Copy link
Contributor

@fstrr fstrr commented Dec 6, 2024

Closes #3823

F4 and F47 should be replaced. CSS' text-decoration:blink is deprecated and has no modern browser support, and the HTML blink element is obsolete.

This new failure technique replaces those two techniques.

Copy link

netlify bot commented Dec 6, 2024

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 3aaa4ed
🔍 Latest deploy log https://app.netlify.com/sites/wcag2/deploys/67f00c3badf97b00089610f4
😎 Deploy Preview https://deploy-preview-4166--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ljoakley ljoakley self-assigned this Feb 14, 2025
@bruce-usab
Copy link
Contributor

Touched upon during backlog call 2/21, but did not finish the discussion.

@mbgower
Copy link
Contributor

mbgower commented Apr 1, 2025

Since there was little support/feedback as part of the AGWG review period, and this is a larger scale change (a new technique), I'm moving to discussion for a review at Friday's TF. If anyone has a chance to review carefully before Friday, please thumbs up if you support

Copy link
Member

@scottaohara scottaohara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

while i left a couple of comments, i'd approve this merging even if the suggestions weren't incorporated.

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

Successfully merging this pull request may close these issues.

Do we need F47?
5 participants