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

chore: fix casing #36735

Merged
merged 1 commit into from
Nov 11, 2024
Merged

chore: fix casing #36735

merged 1 commit into from
Nov 11, 2024

Conversation

yin1999
Copy link
Member

@yin1999 yin1999 commented Nov 11, 2024

Description

fix casing

@yin1999 yin1999 requested a review from a team as a code owner November 11, 2024 14:31
@yin1999 yin1999 requested review from sideshowbarker and removed request for a team November 11, 2024 14:31
@github-actions github-actions bot added Content:WebAPI Web API docs size/xs [PR only] 0-5 LoC changed labels Nov 11, 2024
Copy link
Contributor

Preview URLs

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Thank you! Out of curiosity, is it reporting as a flaw? The links are working on prod: https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollWidth

@Josh-Cena
Copy link
Member

@bsmth: no, because domxref is smart enough to normalize casing. The reason is because we often use element.getBoundingClientRect to indicate it's not a static method, so in this case it's likely intentional.

@sideshowbarker sideshowbarker merged commit b477881 into mdn:main Nov 11, 2024
7 checks passed
@yin1999 yin1999 deleted the patch-1 branch November 12, 2024 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs size/xs [PR only] 0-5 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants