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

doc(zh): translate eip-107 #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Xia-Sam
Copy link

@Xia-Sam Xia-Sam commented Dec 29, 2021

翻译:EIP-107

翻译参考链接:

@Xia-Sam Xia-Sam requested a review from izayl as a code owner December 29, 2021 03:07
@vercel
Copy link

vercel bot commented Dec 29, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ethlibrary/eips/4k1iAVTWpY8NkTBA5aNLbxjEn2Yc
✅ Preview: https://eips-git-fork-xia-sam-zh-eip107-ethlibrary.vercel.app

@@ -1,76 +1,74 @@
---
eip: 107
title: safe "eth_sendTransaction" authorization via html popup
title: 通过 `html` 弹出窗口安全的 `eth_sendTransaction` 授权
Copy link
Member

Choose a reason for hiding this comment

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

-通过 `html` 弹出窗口安全的 `eth_sendTransaction` 授权
+通过 `html` 弹出窗口安全地进行 `eth_sendTransaction` 授权

Comment on lines +6 to +8
status: 停滞
type: 标准追踪
category: 接口
Copy link
Member

Choose a reason for hiding this comment

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

后面不需要翻译 status/type/category 了,只需要翻译 title 和 description(如果有的话),其他 meta 信息不需要翻译,脚本会统一进行专业名词翻译


Motivation
动力
Copy link
Member

Choose a reason for hiding this comment

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

-动力
+动机

Copy link
Member

Choose a reason for hiding this comment

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

这种 EIP 规范的标题也需要规范一下专业名词的翻译

cc @Riely007


![](../../assets/eip-107/authorization-password.png)


Specification
规格
Copy link
Member

Choose a reason for hiding this comment

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

-规格
+标准规范


The use of iframe/ window was required to have both security and user friendliness. The invisible iframe allows the dapp to execute read only calls without the need for user input, and the window ensures user approval before making a call. While we could have made it without the window mode by making the iframe confirmation use the native browser `window.confirm` dialog, this would have prevented the use of a more elegant confirmation popup that the current design allows. It also happens to be that the `window.confirm` is not safe in some browsers, as it gives focus to the accept option and can be triggered automatically (https://bugs.chromium.org/p/chromium/issues/detail?id=260653).
使用 `iframe/window` 需要同时具有安全性和用户友好性。不可见的 `iframe` 允许 `dapp` 在不需要用户输入的情况下执行只读调用,并且窗口确保在调用之前得到用户的批准。虽然我们可以在没有窗口模式的情况下通过使用原生浏览器的 `window.confirm` 对话框进行 `iframe` 确认来实现它,但这会阻止使用当前设计允许的更优雅的确认弹出窗口。也恰好是 `window.confirm` 在某些浏览器中不安全,因为它把焦点放在接受选项上并且可以自动触发 (https://bugs.chromium.org/p/chromium/issues/detail?id=260653).
Copy link
Member

Choose a reason for hiding this comment

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

中文句号

===============
In order to implement this design, the following html file or an equivalent one needs to be served at the url \<node url\>/authorization.html
为了实现这种设计,需要在 `url` \<node url\>/authorization.html 提供以下 `html` 文件或等效的文件
Copy link
Member

Choose a reason for hiding this comment

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

中文句号


That's it.
那就是全部了。
Copy link
Member

Choose a reason for hiding this comment

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

-这就是全部了
+以上。

@izayl
Copy link
Member

izayl commented Dec 29, 2021

@Xia-Sam 后面可以直接通过 preview 链接来预览你的翻译成果

比如这次的:

https://eips-git-fork-xia-sam-zh-eip107-ethlibrary.vercel.app/zh/eip-107.html

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.

2 participants