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

Fix grammar and construction in 2.1.10 #1672

Merged
merged 2 commits into from Sep 23, 2020

Conversation

@galacemiguel
Copy link
Contributor

@galacemiguel galacemiguel commented Dec 11, 2019

There were various grammatical and construction-related errors in the Window sizes and scrolling article. I did my best to correct them.

Please let me know if I made a mistake myself! Thank you!

@lex111 lex111 requested a review from paroche Dec 19, 2019
Copy link
Collaborator

@paroche paroche left a comment

All good or at least OK by me (as far as I noticed) except as noted.

2-ui/1-document/10-size-and-scroll-window/article.md Outdated Show resolved Hide resolved
2-ui/1-document/10-size-and-scroll-window/article.md Outdated Show resolved Hide resolved
2-ui/1-document/10-size-and-scroll-window/article.md Outdated Show resolved Hide resolved
2-ui/1-document/10-size-and-scroll-window/article.md Outdated Show resolved Hide resolved
2-ui/1-document/10-size-and-scroll-window/article.md Outdated Show resolved Hide resolved
@iliakan
Copy link
Member

@iliakan iliakan commented Dec 27, 2019

/me stands with @paroche on the review comments, please update the PR :)

@iliakan
Copy link
Member

@iliakan iliakan commented Jan 18, 2020

@galacemiguel Could you please fix the PR according to the suggestions?

@iliakan
Copy link
Member

@iliakan iliakan commented Sep 23, 2020

ping @galacemiguel 😮

@galacemiguel
Copy link
Contributor Author

@galacemiguel galacemiguel commented Sep 23, 2020

ping @galacemiguel 😮

Sorry, this one slipped my mind. 😅

Getting to it now! @iliakan

@galacemiguel galacemiguel force-pushed the galacemiguel:patch-1 branch from d53ba7c to d3ad4c3 Sep 23, 2020
@galacemiguel
Copy link
Contributor Author

@galacemiguel galacemiguel commented Sep 23, 2020

Hi, @iliakan! I've committed d3ad4c3 to revert and apply some changes based on the suggestions.

Let me know if there's anything I missed.

@iliakan
Copy link
Member

@iliakan iliakan commented Sep 23, 2020

Wow! So I've got to review it again! ;)

@iliakan
Copy link
Member

@iliakan iliakan commented Sep 23, 2020

Great, thanks!

@iliakan iliakan merged commit 8cb4268 into javascript-tutorial:master Sep 23, 2020
1 check passed
1 check passed
license/cla Contributor License Agreement is signed.
Details
@galacemiguel galacemiguel deleted the galacemiguel:patch-1 branch Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.