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

Translating docs/accessibility #60

Merged
merged 43 commits into from Dec 22, 2019
Merged

Translating docs/accessibility #60

merged 43 commits into from Dec 22, 2019

Conversation

smetzdev
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented May 28, 2019

Deploy preview for de-reactjs ready!

Built with commit 17529b0

https://deploy-preview-60--de-reactjs.netlify.com

@miku86 miku86 self-requested a review July 1, 2019 14:22
@miku86 miku86 self-assigned this Jul 1, 2019
Copy link
Member

@ph1p ph1p left a comment

Choose a reason for hiding this comment

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

Please re-read your translation. I've stopped at Line 231.

content/docs/accessibility.md Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
@ph1p
Copy link
Member

ph1p commented Oct 1, 2019

@r3sMetz please do something. This PR needs only some minutes to finish and we can merge it :)

@miku86 miku86 removed their assignment Oct 27, 2019
content/docs/accessibility.md Outdated Show resolved Hide resolved
content/docs/accessibility.md Outdated Show resolved Hide resolved
@ph1p ph1p added idle This PR is idle and the creator has not touched it for a long time. needs review from author First review completed. All changes are processed and a final review is required. and removed 🚧 work in progress 🚧 labels Nov 18, 2019
Sascha Metz and others added 4 commits December 17, 2019 21:33
keep suggested change at line 11

Co-Authored-By: Phil <phil@capshake.com>
keep suggested changes at line 17

Co-Authored-By: Phil <phil@capshake.com>
keep suggested changes at line 29

Co-Authored-By: Phil <phil@capshake.com>
Sascha Metz and others added 2 commits December 17, 2019 21:45
Keep suggested change at line 47

Co-Authored-By: Phil <phil@capshake.com>
Keep suggested changes at line 75

Co-Authored-By: Phil <phil@capshake.com>
@smetzdev
Copy link
Contributor Author

Sorry for the (way to long) time of beeing inactive, @ph1p i just added you on twitter, got some questions if and how to resume

@smetzdev
Copy link
Contributor Author

@ph1p I cant send you messages though, checked every channel :D

@ph1p ph1p removed the idle This PR is idle and the creator has not touched it for a long time. label Dec 19, 2019
@ph1p
Copy link
Member

ph1p commented Dec 19, 2019

Nice work! Thank you for finishing it (:
But there are still some suggestions. Maybe you didn't see them because of the collapsing feature of github.

@smetzdev
Copy link
Contributor Author

So i just have to finish these right here, and then it's done?

@ph1p
Copy link
Member

ph1p commented Dec 19, 2019

yes :D but only if you think, these suggestions are correct

Sascha Metz and others added 15 commits December 22, 2019 11:36
Accept changes on line 93

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 110

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 113

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 115

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 116

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 119

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 128

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 135

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 137

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 139

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestion on line 141

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 149

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 151

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 158

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestions on line 164

Co-Authored-By: Phil <phil@capshake.com>
@smetzdev
Copy link
Contributor Author

Think i've found all suggestions and accepted everyone but two. Am i just waiting for the suggesting persons to eventually change their requests, after debate?

Sascha Metz and others added 2 commits December 22, 2019 20:43
Accept suggestions on line 110 + 111

Co-Authored-By: Phil <phil@capshake.com>
Accept suggestion at line 145

Co-Authored-By: Phil <phil@capshake.com>
@smetzdev
Copy link
Contributor Author

Okay, back in business, i guess. Thanks for your help and patience so far!

@ph1p ph1p removed the needs review from author First review completed. All changes are processed and a final review is required. label Dec 22, 2019
@ph1p ph1p merged commit 1c880d9 into reactjs:master Dec 22, 2019
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

3 participants