Skip to content

ArrayBuffer, binary arrays - #529

Merged
leviding merged 15 commits into
masterfrom
unknown repository
Dec 5, 2019
Merged

ArrayBuffer, binary arrays#529
leviding merged 15 commits into
masterfrom
unknown repository

Conversation

@ghost

@ghostghost commented Nov 20, 2019

Copy link
Copy Markdown

目标章节:4-binary/01-arraybuffer-binary-arrays

当前上游最新 commit:54984506

所做更改如下:

文件名参考上游 commit更改(理由)
article.mdc5ce557新增翻译文件
o1-concat/task.md6fb4aab新增翻译文件
_js.view/solution.js6fb4aab新增翻译文件
_js.view/source.js6fb4aab新增翻译文件

@iliakaniliakan mentioned this pull request Nov 20, 2019
@ghostghost changed the title ArrayBuffer, binary arrays[WIP] ArrayBuffer, binary arraysNov 29, 2019
@ghostghost changed the title [WIP] ArrayBuffer, binary arraysArrayBuffer, binary arraysNov 29, 2019
@KerbalHao

Copy link
Copy Markdown
Collaborator

还请麻烦您修改一下 PR 内容,写上修改的文件和参考的上游 commit 等

Comment thread4-binary/01-arraybuffer-binary-arrays/01-concat/_js.view/solution.js Outdated
@javascript-translate-bot

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@ghost

Copy link
Copy Markdown
Author

还请麻烦您修改一下 PR 内容,写上修改的文件和参考的上游 commit 等

已修改,谢谢~

/done

Comment thread4-binary/01-arraybuffer-binary-arrays/article.md
Comment thread4-binary/01-arraybuffer-binary-arrays/article.md

@KerbalHaoKerbalHao left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

感谢翻译,校对完毕

@ghost

ghost commented Dec 2, 2019

Copy link
Copy Markdown
Author

感谢校对,已修改~

/done

@levidingleviding left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

存在几点问题,麻烦译者 @bemself 注意一下,并进行修改:

  1. 格式细节问题需要仔细
  2. 翻译成中文的,那就不要用英文的引号来扩中文汉字,符号也要相应改为中文全角
  3. 空格等细节格式问题
  4. 168 行的 2<sup>8</sup> 是 28,也就是 2 的八次方,不是取余吧?

@KerbalHao 麻烦校对者也注意,在审核翻译的同时,也要审核格式问题。

Comment thread4-binary/01-arraybuffer-binary-arrays/01-concat/_js.view/solution.js Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/01-concat/_js.view/solution.js Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/01-concat/_js.view/source.js Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/01-concat/task.md Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/article.md Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/article.md Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/article.md Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/article.md Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/article.md Outdated
Comment thread4-binary/01-arraybuffer-binary-arrays/article.md Outdated
@javascript-translate-bot

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@leviding

leviding commented Dec 4, 2019

Copy link
Copy Markdown
Member

麻烦两位 @bemself @KerbalHao 都看下校对意见,现根据校对意见修改一遍,修改的同时需要检查是否还有类似问题,一并修改,辛苦了。

bemselfand others added 7 commits December 4, 2019 19:35
resolve review comment
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
@ghost

ghost commented Dec 4, 2019

Copy link
Copy Markdown
Author

@leviding 感谢校对,已修改~

/done

@ghost

ghost commented Dec 5, 2019

Copy link
Copy Markdown
Author

@KerbalHao 感谢校对,已修改。

/done

@javascript-translate-bot
javascript-translate-bot requested a review from a teamDecember 5, 2019 12:26
@leviding
leviding merged commit 34f4904 into javascript-tutorial:masterDec 5, 2019
@javascript-translate-bot

Copy link
Copy Markdown

Error: the article already translated in the Progress Issue #324⁉️

@levidingleviding added DONE and removed needs +1 labels Dec 5, 2019
Sign up for freeto 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.

3 participants

@KerbalHao@javascript-translate-bot@leviding