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

Requirements editorial changes #619

Merged
merged 11 commits into from Mar 21, 2024
Merged

Requirements editorial changes #619

merged 11 commits into from Mar 21, 2024

Conversation

jspellman
Copy link
Contributor

@jspellman jspellman commented Mar 31, 2022

This PR combines multiple editorial updates that tackle 9 issues.

The "diff" preview version is probably easier to read, red for removed, green for added, yellow for updated.

Then need to work together changed to They need
to clarify because the commenter was not clear on the intent.
Changed ul to be one comma separated sentence. Changed ol to be sentence structure for each item.
I also removed all the "The" before the guideline names as required in Manual of Style.
Also fixed the first acronym use of WCAG3
requirements/index.html Outdated Show resolved Hide resolved
@gundulaniemann
Copy link

Somehow I miss the change history. The youngest entry carries no date, and the youngest but one is from 2019.
I suggest to format the change history as youngest change is the first mentioned, and then they listed by point in time.

Contentwise, I would like to understand why the section severity was deleted, and you the last of the points in "Accessibility guidelines should" was deleted.

@alastc
Copy link
Contributor

alastc commented Feb 27, 2024

Hi @gundulaniemann, the change history isn't updated by this PR, please raise a separate issue.

@alastc alastc merged commit 9585af4 into main Mar 21, 2024
5 checks passed
github-actions bot added a commit that referenced this pull request Mar 21, 2024
SHA: 9585af4
Reason: push, by alastc

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Mar 21, 2024
SHA: 9585af4
Reason: push, by alastc

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Mar 21, 2024
SHA: 9585af4
Reason: push, by alastc

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Mar 21, 2024
SHA: 9585af4
Reason: push, by alastc

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants