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

Update timing-adjustable.html #2584

Closed
wants to merge 1 commit into from
Closed

Conversation

bruce-usab
Copy link
Contributor

Editorial, clarifying that ten times in Extend bullet does not imply ten times as much time.

Closes PR #1040.
Closes PR #2581.

Editorial, clarifying that <q>ten times</q> in Extend bullet does not imply ten times as much time.
@awkawk
Copy link
Member

awkawk commented Aug 10, 2022

@bruce-usab My review on this is that the text is clear and I agree with what it says, but since I also believe that this is what it says already and since the Understanding document doesn't really clarify it sufficiently we should fix it in Understanding rather than changing the normative text.

Copy link
Member

@awkawk awkawk left a comment

Choose a reason for hiding this comment

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

(see comment above - I didn't follow the expected Github review process)

@GreggVan
Copy link

GreggVan commented Aug 10, 2022 via email

@bruce-usab
Copy link
Contributor Author

My review on this is that the text is clear and I agree with what it says

I agree.

but since I also believe that this is what it says already and since the Understanding document doesn't really clarify it sufficiently we should fix it in Understanding rather than changing the normative text.

My preference is this edit, but I am also okay with leaving this SC alone. FWIW, I do not believe this particular pernicious misunderstanding can be corrected in Understanding.

@bruce-usab
Copy link
Contributor Author

bruce-usab commented Aug 10, 2022

@alastc — please dispose as you see fit. This PR is probably a good candidate for wontfix and close.

@bruce-usab bruce-usab requested a review from alastc August 10, 2022 21:40
@bruce-usab
Copy link
Contributor Author

This would be a major normative change - rewriting the SC to mean something other than what it was originally meant to be.

@GreggVan — it may be the case that we did not end up writing what you (and others) originally meant to have written. As written, the bullets are independent from each other. My proposed change (in this PR) is editorial on its face.

Current 2.2.1 Extend bullet:

The user is warned before time expires and given at least 20 seconds to extend the time limit with a simple action (for example, "press the space bar"), and the user is allowed to extend the time limit at least ten times;

Could have been:

The user is warned before time expires and given at least 20 seconds to extend the time limit with a simple action (for example, "press the space bar") and the user is allowed enough opportunities to extend the time limit to at least ten times the duration of the original time limit;

@GreggVan
Copy link

GreggVan commented Oct 11, 2022 via email

@bruce-usab
Copy link
Contributor Author

Closing as we did not get AGWG concurrence on the underlying tension between RAR and how SC is understood.

@bruce-usab bruce-usab closed this Jul 27, 2023
@patrickhlauke patrickhlauke deleted the bruce-usab-221-patch branch July 28, 2023 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants