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 course format for accessibility #25

Merged
merged 8 commits into from
May 30, 2023
Merged

Update course format for accessibility #25

merged 8 commits into from
May 30, 2023

Conversation

heiskr
Copy link
Contributor

@heiskr heiskr commented May 30, 2023

Update course format for accessibility, removing details/summary usage and instead replacing the content of README.md. cc https://github.com/github/skills/issues/115

@heiskr heiskr requested a review from a team May 30, 2023 17:37
@heiskr heiskr merged commit ecd6e77 into main May 30, 2023
@heiskr heiskr deleted the a11y branch May 30, 2023 22:33
echo "Remove 'open' from any <details> tags"
sed -r 's/<details id=([0-9]+) open>/<details id=\1>/g' README.md > tmp
mv tmp README.md
echo "TODO UPDATE NEEDED"
Copy link
Contributor

Choose a reason for hiding this comment

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

I will open a PR for this.

@Ljaidi
Copy link

Ljaidi commented Jul 4, 2023

Update course format for accessibility, removing details/summary usage and instead replacing the content of README.md. cc https://github.com/github/skills/issues/115
5551900j@gmail.com

@skills skills locked and limited conversation to collaborators Jul 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants