-
Notifications
You must be signed in to change notification settings - Fork 237
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
corrections to Character key shorts and Reflow #606
Conversation
Hi @mbgower - Read through it, looks good to me! |
guidelines/sc/21/reflow.html
Outdated
|
||
<p class="note">Examples of content which require two-dimensional layout are images, maps, diagrams, | ||
<p class="note">Examples of parts of content which require two-dimensional layout are images, maps, diagrams, |
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.
I don't think that this is necessary. In WCAG content can be a piece of the page (see the "change of context" def for an example).
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.
If you don't put in "parts of" then it needs to be "requires" > "Examples of content which requires..."
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.
I'd change the verb to "requires"
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.
I prefer the change to "requires" as well.
<p class="note">Examples of parts of content which require two-dimensional layout are images, maps, diagrams, | |
<p class="note">Examples of content which requires two-dimensional layout are images, maps, diagrams, |
@@ -13,9 +13,9 @@ <h4>Reflow</h4> | |||
</ul> | |||
<p>Except for parts of the content which require two-dimensional layout for usage or meaning.</p> | |||
|
|||
<p class="note">320 CSS pixels is equivalent to a starting viewport width of 1280 CSS pixels wide at 400% zoom. For web content which are designed to scroll horizontally (e.g. with vertical text), the 256 CSS pixels is equivalent to a starting viewport height of 1024px at 400% zoom.</p> | |||
<p class="note">320 CSS pixels is equivalent to a starting viewport width of 1280 CSS pixels wide at 400% zoom. For web content which is designed to scroll horizontally (e.g., with vertical text), 256 CSS pixels is equivalent to a starting viewport height of 1024px at 400% zoom.</p> |
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.
Yikes.
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.
Oh dear, I wonder if that got changed from "web pages which are designed" and not updated the is/are.
@@ -14,7 +14,7 @@ <h4>Character Key Shortcuts</h4> | |||
<dd>A <a>mechanism</a> is available to turn the shortcut off;</dd> | |||
|
|||
<dt>Remap</dt> | |||
<dd>A mechanism is available to remap the shortcut to use one or more non-printable keyboard characters (e.g. Ctrl, Alt, etc);</dd> | |||
<dd>A mechanism is available to remap the shortcut to include one or more non-printable keyboard keys (e.g., Ctrl, Alt);</dd> |
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.
Agree on the e.g./etc.
I think that use/include an be explained in understanding and probably needs to be clarified whether include or use is the term used/included in the SC text.
My thoughts:
"keyboard keys" sounds a bit odd, would just "keys" work? I haven't found anyone mentioning this as confusing, but if it's changed, just 'keys' would probably do it.
"Use" makes the most sense to me, if we changed it I would suggest "add" instead. I don't remember the ins & outs of this one from the time, but was there a reason it couldn't be this?
I don't think that is necessary, 'content' conveys the meaning and the addition makes it less clear to me. I realise that it says 'parts of the content' above, it's hard to explain, but that was added specifically to convey that that you can have pages with content, and a part of that page/content can scroll in both directions. When in the note it talks about 'content', the context of the examples sets the right level for interpretation. |
I vote for "keyboard keys". A keyboard is a board of keys, so it is normal to say keyboard keys even if it seems odd.
"add" makes it seem like a shortcut of ONLY non-printing character keys might not be ok, but it is.
I made this comment on the code in the pull request, so we agree. |
Just to clarify, in Reflow, for "parts" the issue is that there is not agreement as it is written right now. "content which require" is not grammatical. If "parts of" is not added in, than it needs to become "content which requires" |
@awkawk is there a reason this isn't part of this week's survey? |
@mbgower Forgot. Just added and will notify the 3 people who took the survey already. |
Accepted by WG on 2/12/2019 |
For whatever reason, these changes were not incorporated, so I'm re-addressing. I believe the Reflow ones were agreed on with the change Andrew put in. So for Shortcuts... Existing:
I definitely prefer either "include" or "add" instead of "use". An author can't have a shortcut that is just Ctrl; it's a modifier key. So it seems to me saying the shortcut "uses" continues to imply something other than we want. I can live with "add", but it just seems a little clunky. Not going to die on this one, but... In the keyboard characters versus keyboard keys versus keys discussion, it seems like we have agreement it shouldn't be characters. I kinda get Alastair's keyboard key quibble, and I think we have clarifty we're talking about keys (e.g., Character Key Shortcuts), so I'm leaning to "keys". We have agreement that the ", etc" can go. So in Mike's world, we end up with:
|
@awkawk looks like there's a travis build failure, which is why this didn't come across? |
Travis build is something only Michael pays attention to I think, I started digging in once but it is generally noise, and doesn't prevent a merge. (Conflicts would, but not travis.) |
@awkawk and @michael-n-cooper I'm assuming that because of the travis error, this never happened. There's actually a very odd history. I see that Andrew committed one of the changes for the second Note on Reflow on Feb 12,
Tthe other grammar fudge in Reflow has never had any controversy, and yet wasn't committed.
becomes
|
On the Character Keys changes, it looks like we got consensus on making
read as
@michael-n-cooper is it easiest for me just to make a new PR for these two things? |
@awkawk I see you did the merge, but I did want to point out that one of the modifications that I thought had been pulled out was not (changing "use" to "include"). With this merge it now reads "include". I feel like this is my mess, so suggest I make a new PR for this and include the Character key shortcut line change in the same one. Does that work? |
@mbgower It was "use" before and that was one of the suggested changes. I don't care which way that word goes, seems very editorial. You can make a PR if you want, but I'll sleep fine if you don't. :) Also, these changes will be in the editor's draft but won't be in the public WCAG 2.1 until we republish with errata. |
The errata requests for these two SCs are as follows:
For Character Key Shortcuts, there are three changes. The most important change is the replacement of "characters" by "keys" in the first bullet. Simply, modifier keys are not characters. This change does not alter the interpretation but merely addresses the inappropriate use of a word that could potentially cause confusion.
The second suggestion is more open to discussion, and concerns replacing "use" with the word "include". This is a suggested edit to address the literal meaning of the text. The purpose of the mechanism is not to replace a character key with a non-printable key, but to create a keyboard shortcut combination -- e.g., instead of A use Ctrl+A. "Include" better asserts this notion.
Finally the parenthetical information has been updated to match what I believe is the acceptable style for e.g., (including a comma) and eliminating the "etc" which is superfluous in the context of an example. There is no supposition that the example is an exhaustive list, therefore the etc is unnecessary. This is a trivial change, only offered here with the assumption that if the text is being altered, such slight style questions may as well be addressed.
Reflow's changes are primarily concerned with number agreement. "content...are designed" becomes "content...is designed"
The first note has also been changed, but here rather than make "content which require" be "content which requires" we have added in "parts of" so that it reads "parts of content which require". This solution mirrors language in the SC exception.
A trivial style change has also been added for two occurrences -- the comma after e.g.,