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

handling fallback content for the meter element #401

Open
wants to merge 12 commits into
base: gh-pages
Choose a base branch
from

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented May 4, 2022

resolves #27

this PR looks to treat the fallback content (the subtree) of the meter element as aria-valuetext.

However, an author updates the value of the meter element, but does not update the subtree/fallback content, then the value needs to instead take preference over the fallback content, as the fallback contnet would no longer represent the true value of the meter element.

Feedback / changes welcome.


Preview | Diff

resolves #27

this PR looks to treat the fallback content (the subtree) of the meter element as `aria-valuetext`.

However, an author updates the value of the meter element, but does not update the subtree/fallback content, then the `value` needs to instead take preference over the fallback content, as the fallback contnet would no longer represent the true value of the `meter` element.
@scottaohara scottaohara marked this pull request as ready for review May 4, 2022 10:59
@jnurthen jnurthen requested review from cyns and cookiecrook May 5, 2022 17:19
index.html Outdated Show resolved Hide resolved
Co-authored-by: James Craig <cookiecrook@users.noreply.github.com>
@scottaohara
Copy link
Member Author

scottaohara commented Mar 22, 2023

TODO: browser bugs need to be filed

  • Chromium
  • Gecko
  • Webkit

@cookiecrook
Copy link
Collaborator

FYI, my suggested placeholder text "[text content? innerText? computed nameFrom:content]" was not intended to be committed literally. Sorry for the confusion. Should this be changed to a "Draft PR" until what-to-call-that is resolved?

@scottaohara scottaohara marked this pull request as draft March 23, 2023 16:08
index.html Outdated Show resolved Hide resolved
@scottaohara scottaohara marked this pull request as ready for review October 18, 2023 12:30
<td>
<td class="comments">
<div>
If the `meter` element's subtree contains text content, user agents SHOULD expose this content, along with the element's initial value, by exposing the fallback content via the <a class="core-mapping" href="#ariaValueText">`aria-valuetext`</a> mapping.
Copy link
Collaborator

@cookiecrook cookiecrook Nov 4, 2023

Choose a reason for hiding this comment

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

element's subtree contains text content, user agents SHOULD expose this content

Should this use "text node"?

Or innerText?

along with the element's initial value, by exposing the fallback content

I dunno what this means... Does this mean concat the innerText and aria-valuenow, or convert aria-valuenow to a percentage and then concat? Does "along with" mean prepend? append? join(", ")?

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

Successfully merging this pull request may close these issues.

HTML-AAM: how to map Inner Phrasing Content for <meter>?
2 participants