accname icon indicating copy to clipboard operation
accname copied to clipboard

Update to Condition for step 2C

Open OisinNolan opened this issue 5 years ago • 5 comments

  • This PR broadens the condition for step 2E such that it applies to controls embedded in nodes that allow name from content as well as controls that are embedded in labels.
  • Embedded in label and Embedded in node that allows name from content may then be merged and simplified to descendant of node whose accessible name / description is being computed, as is done in the PR.

This PR is motivated by the fact that many (all?) AccName implementations interpret step 2E in this manner. See example: image


:boom: Error: 500 Internal Server Error :boom:

PR Preview failed to build. (Last tried on May 16, 2021, 3:07 PM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

:rotating_light: Spec Generator - Spec Generator is the web service used to build specs that rely on ReSpec.

:link: Related URL


😭  Sorry, there was an error generating the HTML. Please report this issue!
Specification: https://rawcdn.githack.com/OisinNolan/accname/beeb3d59364035373d642da35b6a6dc4e3bf323d/index.html?isPreview=true&publishDate=2021-05-16
ReSpec version: 25.6.0
File a bug: https://github.com/w3c/respec/
Error: Error: Evaluation failed: Timeout: document.respec.ready didn't resolve in 27421ms.
    at ExecutionContext._evaluateInternal (/u/spec-generator/node_modules/puppeteer/lib/cjs/puppeteer/common/ExecutionContext.js:221:19)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async ExecutionContext.evaluate (/u/spec-generator/node_modules/puppeteer/lib/cjs/puppeteer/common/ExecutionContext.js:110:16)
    at async generateHTML (/u/spec-generator/node_modules/respec/tools/respecDocWriter.js:276:12)
    at async toHTML (/u/spec-generator/node_modules/respec/tools/respecDocWriter.js:92:18)
    at async Object.generate [as respec] (/u/spec-generator/generators/respec.js:14:44)
    at async /u/spec-generator/server.js:205:44

If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.


Preview | Diff

OisinNolan avatar Aug 28 '20 16:08 OisinNolan

@jnurthen please add to next week's agenda.

MelSumner avatar May 16 '21 15:05 MelSumner

Next meeting is on 27 may. No meeting this week due to GAAD.

jnurthen avatar May 16 '21 15:05 jnurthen

Also @accdc I think I have resolved the merge conflicts so this is probably ready for you to review. 👍

MelSumner avatar Mar 16 '22 22:03 MelSumner

I agree, The text "whose value can be adjusted by the user" should be removed, and simply refer to control which will include all such variations.

accdc avatar Oct 11 '23 21:10 accdc

How does this PR relate to #183 ?

jnurthen avatar Nov 03 '23 23:11 jnurthen

We've moved ACCNAME spec to a mono repo: https://github.com/w3c/aria

If you still want this to be considered for ACCNAME, please re-open this PR there.

MelSumner avatar Jul 08 '24 16:07 MelSumner