-
Notifications
You must be signed in to change notification settings - Fork 125
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
Add mark role #1133
Add mark role #1133
Conversation
Closes issue #508
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: We need to add the role to the role categorization. See the bulleted lists found in https://w3c.github.io/aria/#roles_categorization
@joanmarie @jnurthen I've updated the PR to match the HTML spec text and add mark to the role categorization section. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See inline comment about breaking up definition into more than one paragraph.
The HTML text for mark element is pretty obscure without examples. (i.e. why "one document"? And the "previously unexpected scrutiny" sentence is really long...):
I kinda preferred the original description:
Or the "similar to HTML but more clearly written" MDN text for mark element:
If we keep the HTML description, it really needs to be clarified with an example or two. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joanmarie Can you offer guidance on these remarks? I agree with Carolyn that the original HTML spec text is not ideal. I don't think the MDN text is ideal either. @anyone feel free to suggest some text.
The HTML text for mark element is pretty obscure without examples. (i.e. why "one document"? And the "previously unexpected scrutiny" sentence is really long...):
The mark element represents a run of text in one document marked or highlighted for reference purposes, due to its relevance in another context.
When used in a quotation or other block of text referred to from the prose, it indicates a highlight that was not originally present but which has been added to bring the reader's attention to a part of the text that might not have been considered important by the original author when the block was originally written, but which is now under previously unexpected scrutiny. When used in the main prose of a document, it indicates a part of the document that has been highlighted due to its likely relevance to the user's current activity.I kinda preferred the original description:
A mark contains a run of content, marked or highlighted to indicate that other content refers to it, or to indicate special relevance to the user's current activity.
Or the "similar to HTML but more clearly written" MDN text for mark element:
The HTML
<mark>
element represents text which is marked or highlighted for reference or notation purposes, due to the marked passage's relevance or importance in the enclosing context.
Typical use cases for<mark>
include:
- When used in a quotation (
<q>
) or block quote (<blockquote>
), it generally indicates text which is of special interest but is not marked in the original source material, or material which needs special scrutiny even though the original author didn't think it was of particular importance. Think of this like using a highlighter pen in a book to mark passages that you find of interest.- Otherwise,
<mark>
indicates a portion of the document's content which is likely to be relevant to the user's current activity. This might be used, for example, to indicate the words that matched a search operation.If we keep the HTML description, it really needs to be clarified with an example or two.
Yep. :)
Not sure... and what if it changes? Maintenance hassle...
I was caught off guard by "a run of text in one document", which would naturally be followed by I also liked the MDN idea of breaking up that dense HTML paragraph into a list of "Typical use cases". And I found the analogy, "Think of this like using a highlighter pen in a book to mark passages that you find of interest.", quite helpful. |
I personally like the MDN text. What do others think? |
@joanmarie @jnurthen @carmacleod How about this? It is quite similar to the MDN text. |
Nice! I like it. Only a couple of nits, mentioned above. |
Co-Authored-By: Carolyn MacLeod <[email protected]>
Co-Authored-By: Carolyn MacLeod <[email protected]>
@carmacleod Thanks, good catches. What do you think now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Ship it! :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
* Add mark role Co-Authored-By: Carolyn MacLeod <[email protected]>
Closes issue #508
Preview | Diff