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

Is the distinction between "s" and "S" insufficient from an accessibility point of view #62

Closed
ndw opened this issue Apr 5, 2022 · 5 comments
Assignees
Labels
enhancement An improvement to an existing feature
Milestone

Comments

@ndw
Copy link
Contributor

ndw commented Apr 5, 2022

No description provided.

@ndw
Copy link
Contributor Author

ndw commented Apr 7, 2022

The nonterminal S is only used once and is never serialized so its name is irrelevant.

In the interest of accessibility, I suggest we rename it WS.

@ndw ndw added this to the Version 1.0 milestone Apr 8, 2022
@ndw ndw added the enhancement An improvement to an existing feature label Apr 8, 2022
@cmsmcq
Copy link
Contributor

cmsmcq commented Apr 11, 2022

What kind of accessibility issue are we worrying about? Visual acuity (because lowercase and uppercase S are very similar)? Or screen readers? or ... ?

Since the salient difference between s and S is (or was, last time I looked) that s is for optional whitespace and S is for required whitespace, I wonder if RS ('required space') or ws-plus or something similar might be a better name than WS.

@ndw
Copy link
Contributor Author

ndw commented Apr 11, 2022

It was @LinguaCelta that raised the point that they are visually too similar. Subsequent experimentation with grammars has persuaded me that this is true irrespective of accessibility criteria.

I was trying to keep the name short, I thought WS filled the bill. I could equally go with RS. I'm a little less fond of the somewhat longer ws-plus or s-required, etc. family of choices, but if there's a consensus favorite, I won't fuss.

@ndw
Copy link
Contributor Author

ndw commented Apr 12, 2022

The group favors RS.

@ndw
Copy link
Contributor Author

ndw commented Apr 16, 2022

Resolved in the 15 April 2022 draft.

@ndw ndw closed this as completed Apr 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

No branches or pull requests

3 participants