-
Notifications
You must be signed in to change notification settings - Fork 286
Add note about logo/logotypes contrast to 1.4.3, 1.4.6, and 1.4.11 understanding #4402
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
base: main
Are you sure you want to change the base?
Changes from all commits
60558b0
1dc7634
b4471c1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -396,6 +396,19 @@ <h4>Required for Understanding</h4> | |||||||||||||||||||||||||||||||||||||
<li> The information is available in another form, such as in a table that follows the graph, which becomes visible when a "Long Description" button is pressed.</li> | ||||||||||||||||||||||||||||||||||||||
<li> The graphic is part of a logo or brand name (which is considered "essential" to its presentation).</li> | ||||||||||||||||||||||||||||||||||||||
</ul> | ||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||
<div class="note"> | ||||||||||||||||||||||||||||||||||||||
<p>Logos and logotypes are exempted from contrast requirements when they are purely used as | ||||||||||||||||||||||||||||||||||||||
<em>graphical objects</em>, under the assumption that they must comply with stricter color choices mandated | ||||||||||||||||||||||||||||||||||||||
by corporate identity guidelines. | ||||||||||||||||||||||||||||||||||||||
However, this is not the case when they also act as <em>user interface components</em> | ||||||||||||||||||||||||||||||||||||||
(such as links or other interactive controls). In these cases, authors should choose | ||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think people might read your writing as best practice or advice. "If the logo is a user interface component, you should ...." There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i intended it in the RFC sense
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. will discuss in the working group if we need to use stronger language (such as "authors must either choose ... or ...") There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In fact, if the logo is visible by some contrasting parts, it should be sufficient if the text inside is given by an alternative like a tooltip. Where the tooltip should also explain the functionality. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Suggestion: |
||||||||||||||||||||||||||||||||||||||
a variant of the logo or logotype that has sufficient contrast, if allowed by the | ||||||||||||||||||||||||||||||||||||||
corporate identity guidelines. | ||||||||||||||||||||||||||||||||||||||
Alternatively, authors should provide an equivalent <em>user interface component</em> | ||||||||||||||||||||||||||||||||||||||
which serves the same purpose and does meet the contrast requirements.</p> | ||||||||||||||||||||||||||||||||||||||
Comment on lines
+401
to
+409
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
I understand the value and meaning of should in RFCs. I hope this slight rephrasing and reformatting makes the text more explicit and understandable. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Visually, the code looks like you want that update as two paragraphs, but there's only one set of There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Noting that making this categorical statement will no doubt ruffle some feathers, as it "seems" to introduce a normative failure by the backdoor that some will argue wasn't as cut and dry there before...which is why I was treading lightly (in one of the linked issues there was some mention of "exempting logos was on purpose as it would lead to too many failures" or words to that effect) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. happy to take it as additional suggestion to the WCAG 2.x backlog group though for further discussion There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @erikkroes I updated your suggestion above for clarity (keeping the line breaks where they were, so it's clearer what your change is), and assuming you did want the first para to be actually split into two as well There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've opened this issue for clarification, so maybe I tread less lightly 😄 The split might not be needed indeed. I think it helped make my point in this conversation, but it should be fine without. |
||||||||||||||||||||||||||||||||||||||
</div> | ||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||
</section> | ||||||||||||||||||||||||||||||||||||||
<section> | ||||||||||||||||||||||||||||||||||||||
<h4>Gradients</h4> | ||||||||||||||||||||||||||||||||||||||
|
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.
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.
A suggestion to avoid "should".