Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix: Correct descriptions of clientX and pageX #1233

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

Open
yzws1001 wants to merge 1 commit into javascript-tutorial:master
base: master
Choose a base branch
Loading
from yzws1001:patch-1

Conversation

@yzws1001
Copy link

@yzws1001 yzws1001 commented Dec 9, 2024
edited
Loading

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

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

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

本 PR 所做更改如下:

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

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

CLAassistant commented Dec 9, 2024
edited
Loading

CLA assistant check
All committers have signed the CLA.

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

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

@songhn233 songhn233 Jan 4, 2025

Choose a reason for hiding this comment

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

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

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

1 similar comment

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

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

1 similar comment

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

Reviewers

1 more reviewer

@songhn233 songhn233 songhn233 requested changes

Reviewers whose approvals may not affect merge requirements

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /