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

Accessibility 2021 edits #2576

Merged
merged 5 commits into from
Nov 23, 2021
Merged

Accessibility 2021 edits #2576

merged 5 commits into from
Nov 23, 2021

Conversation

tunetheweb
Copy link
Member

Closes #2147
Makes progress on #2525

Standard enough copy edit. Will call out a couple of points of note.

@tunetheweb tunetheweb added the editing Content excellence label Nov 22, 2021
@tunetheweb tunetheweb added this to the 2021 Analysis milestone Nov 22, 2021
@tunetheweb tunetheweb requested a review from alextait1 November 22, 2021 21:38
Copy link
Member Author

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

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

Some comments below on some changes you might want to review. The rest should be standard I think.

src/content/en/2021/accessibility.md Outdated Show resolved Hide resolved
src/content/en/2021/accessibility.md Show resolved Hide resolved
src/content/en/2021/accessibility.md Show resolved Hide resolved
src/content/en/2021/accessibility.md Show resolved Hide resolved
src/content/en/2021/accessibility.md Show resolved Hide resolved
Copy link
Member

@rviscomi rviscomi left a comment

Choose a reason for hiding this comment

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

Nits to remove "please"

src/content/en/2021/accessibility.md Outdated Show resolved Hide resolved
src/content/en/2021/accessibility.md Outdated Show resolved Hide resolved
Co-authored-by: Rick Viscomi <[email protected]>
Copy link
Contributor

@alextait1 alextait1 left a comment

Choose a reason for hiding this comment

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

Some small nit comments but LGTM 🚢

src/content/en/2021/accessibility.md Show resolved Hide resolved
src/content/en/2021/accessibility.md Outdated Show resolved Hide resolved
src/content/en/2021/accessibility.md Outdated Show resolved Hide resolved
src/content/en/2021/accessibility.md Outdated Show resolved Hide resolved
@tunetheweb tunetheweb merged commit f88361f into main Nov 23, 2021
@tunetheweb tunetheweb deleted the accessibility-2021-edits branch November 23, 2021 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editing Content excellence
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Accessibility 2021
3 participants