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.

Sign up for GitHub

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

Jump to bottom

I think "focus" does not belong here #37889

Open
juanlanus opened this issue Jan 30, 2025 · 1 comment · May be fixed by #37902
Open

I think "focus" does not belong here #37889

juanlanus opened this issue Jan 30, 2025 · 1 comment · May be fixed by #37902
Labels
Content:WebAPI Web API docs goal: clarity (Experimental label) Issues about unclear/confusing/inconcise content. needs info Needs more information to review or act on.

Comments

@juanlanus
Copy link

MDN URL

https://developer.mozilla.org/en-US/docs/Web/API/Selection/anchorOffset

What specific section or headline is this issue about?

main

What information was incorrect, unhelpful, or incomplete?

Hi There,
In the paragraph:

In the case of Selection.anchorNode being another type of node, Selection.anchorOffset returns the number of Node.childNodes the selection's focus is offset within the Selection.anchorNode.

it seems to me that the word "focus" might be confusing.
Actually, it confused me and prompted me to re-read it all.
Because "focus" is associated with the end of the selection.

Regards, and thanks for the excellent work.

Juan Lanus

What did you expect to see?

Something like "selection start".

Do you have any supporting links, references, or citations?

No response

Do you have anything more you want to share?

Regards, and thanks for the excellent work.

Juan Lanus

MDN metadata

Page report details
@juanlanus juanlanus added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jan 30, 2025
@github-actions github-actions bot added the Content:WebAPI Web API docs label Jan 30, 2025
vladboisa added a commit to vladboisa/content that referenced this issue Jan 31, 2025
@vladboisa
docs: swap a word in sentence
0ac21fb 
Swap the sentence for more "clear" version


Fixes mdn#37889
@vladboisa vladboisa linked a pull request Jan 31, 2025 that will close this issue
Open
vladboisa added a commit to vladboisa/content that referenced this issue Jan 31, 2025
@vladboisa
docs: swap a word in sentence
a539687 
Swap the sentence for more "clear" version


Fixes mdn#37889
@vladboisa
Copy link
Contributor

Hi, thanks for opening this issue. I'd like to help resolve it.
Let's discuss which version is better? Or just change to "selection start" ?

@Josh-Cena Josh-Cena added needs info Needs more information to review or act on. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Feb 1, 2025
@caugner caugner added the goal: clarity (Experimental label) Issues about unclear/confusing/inconcise content. label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Content:WebAPI Web API docs goal: clarity (Experimental label) Issues about unclear/confusing/inconcise content. needs info Needs more information to review or act on.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: swap a word in sentence vladboisa/content
4 participants
@caugner @juanlanus @Josh-Cena @vladboisa