Skip to content

Fix: translation errrors#940

Merged
leviding merged 4 commits intojavascript-tutorial:masterfrom
uknfire:fix
Oct 5, 2021
Merged

Fix: translation errrors#940
leviding merged 4 commits intojavascript-tutorial:masterfrom
uknfire:fix

Conversation

@uknfire
Copy link
Copy Markdown
Contributor

@uknfire uknfire commented Aug 20, 2021

目标章节

  • 2-ui/5-loading/01-onload-ondomcontentloaded/article.md
  • 2-ui/2-events/01-introduction-browser-events/04-move-ball-field/task.md
  • 5-network/06-fetch-api/article.md
  • 6-data-storage/01-cookie/article.md

当前上游最新 commitjavascript-tutorial/en.javascript.info@4d01fc2

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
2-ui/5-loading/01-onload-ondomcontentloaded/article.md 修改翻译错误
2-ui/2-events/01-introduction-browser-events/04-move-ball-field/task.md 修改翻译错误
5-network/06-fetch-api/article.md 修改翻译错误
6-data-storage/01-cookie/article.md 改进翻译

…aded/article.md

original text: What happens if we set the DOMContentLoaded handler after the document is loaded?
…vents/04-move-ball-field/task.md

original text: When the page is scrolled, nothing should break.
original text: The keepalive option indicates that the request may “outlive” the webpage that initiated it.
"outlive" means live longer than ...
original text: 1. A samesite=lax cookie is sent if both of these conditions are true 2.So, what samesite=lax does, is to basically allow the most common “go to URL” operation to have cookies. E.g. opening a website link from notes that satisfy these conditions.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Aug 20, 2021

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Member

@leviding leviding left a comment

Choose a reason for hiding this comment

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

Thanks.

@leviding leviding merged commit ecb2c8d into javascript-tutorial:master Oct 5, 2021
@leviding leviding added DONE and removed needs +1 labels Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants