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: Correct descriptions of clientX and pageX #1233

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

Conversation

yzws1001
Copy link

@yzws1001 yzws1001 commented Dec 9, 2024

The clientX/Y and pageX/Y descriptions of mouse events are opposite

目标章节:2-ui/3-event-details/1-mouse-events-basics/article.md

当前上游最新 commitjavascript-tutorial/en.javascript.info@34a80e7

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md 修改部分错误

The clientX/Y and pageX/Y descriptions of mouse events are opposite
@CLAassistant
Copy link

CLAassistant commented Dec 9, 2024

CLA assistant check
All committers have signed the CLA.

@@ -139,7 +139,7 @@

我们已经在 <info:coordinates> 中解释过它们之间的区别。

简而言之,相对于文档的坐标 `pageX/Y` 以文档的左上角为参照物,并且同一位置的坐标不随页面的滚动而改变。相对于窗口的坐标 `clientX/Y` 以当前窗口的左上角为参照物,并且同一位置的坐标会随着页面的滚动而改变。
简而言之,相对于文档的坐标 `clientX/Y` 以文档的左上角为参照物,并且同一位置的坐标不随页面的滚动而改变。相对于窗口的坐标 `pageX/Y` 以当前窗口的左上角为参照物,并且同一位置的坐标会随着页面的滚动而改变。
Copy link
Member

Choose a reason for hiding this comment

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

这里似乎没反?client 就是针对窗口的,page 是针对文档的

@javascript-translate-bot

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

1 similar comment
@javascript-translate-bot

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

@javascript-translate-bot

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

1 similar comment
@javascript-translate-bot

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants