aria icon indicating copy to clipboard operation
aria copied to clipboard

editorial: remove vague "meaningful" and parenthetical example

Open pkra opened this issue 2 years ago • 4 comments

Following up on #2062. Closes #2068

In this negation, it seems very vague (what is non-meaningful-but-not-empty alt?) and I think a clear "SHOULD NOT add alt" is better.

Otherwise, if there's a use case for "non-meaningful text alternative" then perhaps this can be added to the parenthetical example.

To further reduce confusion, the parenthical example using an empty alt attribute was also removed.


Preview | Diff

pkra avatar Oct 20 '23 12:10 pkra

if we really don't intend to allow alt="" on role="none" then fine - but I'm not sure that is the intent. If it is then we need to change the example too.

E.g., nu validator flags it as error "An img element which has an alt attribute whose value is the empty string must not have a role attribute."

pkra avatar Nov 02 '23 08:11 pkra

Deploy Preview for wai-aria ready!

Name Link
Latest commit 4a7c3c7076db19ce03d64626866d932bbf525638
Latest deploy log https://app.netlify.com/sites/wai-aria/deploys/679b31982869c30008c89c3f
Deploy Preview https://deploy-preview-2069--wai-aria.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.

netlify[bot] avatar Jan 30 '25 07:01 netlify[bot]

@jnurthen I've added your suggestion. Can I merge this?

pkra avatar Jan 30 '25 08:01 pkra

@jnurthen another ping.

@scottaohara are you still ok with this adjustment?

pkra avatar Feb 09 '25 14:02 pkra

couldn't fix the merge conflicts so I added a new PR: https://github.com/w3c/aria/pull/2505

in order to close/merge this :)

spectranaut avatar Apr 01 '25 17:04 spectranaut

closing, merged replace PR

spectranaut avatar Apr 01 '25 17:04 spectranaut