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

Expand translatable properties section #1316

Closed
benoitjchevalier opened this issue Aug 27, 2020 · 4 comments · Fixed by #2043
Closed

Expand translatable properties section #1316

benoitjchevalier opened this issue Aug 27, 2020 · 4 comments · Fixed by #2043
Assignees
Labels
editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo good first issue
Projects
Milestone

Comments

@benoitjchevalier
Copy link

Looks like the translatable section is missing a few properties. It defines:

  • aria-label
  • aria-placeholder
  • aria-roledescription
  • aria-valuetext

I think it's missing:

  • aria-colindextext
  • aria-description
  • aria-rowindextext

The braille ones are a bit more complex but my understanding is that they should also be translated since they won't be translated by assistive technologies:

  • aria-braillelabel
  • aria-brailleroledescription

I can do a PR if those additions make sense.

@jnurthen
Copy link
Member

Yep - you are right. We should add the first 3.
I'd like advice on the braille ones from @cookiecrook @sinabahram and @pkra
My first thought is that yes they should but we should add some text that states that an automated translation of these should not be attempted - but I would defer to any of the above on this.

@jnurthen jnurthen added this to the ARIA 1.3 milestone Sep 3, 2020
@cookiecrook
Copy link
Contributor

Also, @joanmarie @michael-n-cooper should there be a new translatableString (strawman name) value type in addition to the existing string value type?

@cookiecrook
Copy link
Contributor

If so, then the translatable section could be generated from ReSpec.

Regarding the braille attr, I think non-braille values for aria-braillelabel are probably okay to translate. I think it's unlikely that a translation of braille cell-conserving abbreviations (e.g. "btn" for "button" or "g" for "group") would be useful.

@cookiecrook
Copy link
Contributor

As for the more rarely used Braille Pattern values for those attributes, I think it's unlikely any existing translation engine would try to translate them. For example, even if you knew the language (English) aria-braillelabel="⠏" could mean "p" in Grade 1 Uncontracted, or "people" in Grade 2 Contracted. It'd be impossible to back translate without knowing the table, so I don't think there is any risk of it in the near term.

So the choices regarding the Unicode Braille Pattern range are:

  1. Add a recommendation or note that mentions UAs should not attempt to translate substrings that include the Unicode Braille Pattern range.
  2. Don't mention it at all. We can add a note later if it becomes a problem.

I lean toward option 2. There's very little need to mention it in the spec because it's unlikely to be a problem. Explaining the problem space just adds more complexity to the spec without much gain.

@jnurthen jnurthen self-assigned this May 6, 2021
@spectranaut spectranaut added this to James Nurthen in ARIA 1.3 Jun 7, 2022
@jnurthen jnurthen added good first issue feature may add new concept(s) to ARIA which will require implementations or APG changes labels Aug 2, 2022
@chlane chlane self-assigned this Aug 3, 2022
@jnurthen jnurthen added editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo and removed feature may add new concept(s) to ARIA which will require implementations or APG changes labels Sep 16, 2022
@spectranaut spectranaut moved this from James Nurthen to Good first issues in ARIA 1.3 Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo good first issue
Projects
ARIA 1.3
Good first issues
Development

Successfully merging a pull request may close this issue.

4 participants