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 typos in usage/configuration.rst #12024

Merged
merged 1 commit into from
Mar 7, 2024
Merged

Fix typos in usage/configuration.rst #12024

merged 1 commit into from
Mar 7, 2024

Conversation

afranke
Copy link
Contributor

@afranke afranke commented Feb 28, 2024

I’m guessing this section was copied over from the previous one and these bits got forgotten when updating.

I’m also wondering whether there is a missing “at an” before “earlier or lazier step”. I can amend this PR if that’s confirmed.

@picnixz
Copy link
Member

picnixz commented Mar 7, 2024

I think "at an" is indeed needed. Could you also fix this in the CSS section please? However I do not understand what they mean by "lazier" step in this case.

Btw do we use JavaScript or Javascript?

@picnixz picnixz added the awaiting:response Waiting for a response from the author of this issue label Mar 7, 2024
@afranke
Copy link
Contributor Author

afranke commented Mar 7, 2024

I think "at an" is indeed needed. Could you also fix this in the CSS section please?

Done!

However I do not understand what they mean by "lazier" step in this case.

I assume it refers to lazy evaluation and the phrase means “will be loaded only when it is needed”.

Btw do we use JavaScript or Javascript?

JavaScript. 🙂

@picnixz
Copy link
Member

picnixz commented Mar 7, 2024

I assume it refers to lazy evaluation and the phrase means “will be loaded only when it is needed”.

Yes but I am wondering what is the connection with the priority itself (for me only the "earlier" is relevant)

@picnixz
Copy link
Member

picnixz commented Mar 7, 2024

Anyway I think everything is fine. Thank you for your work !

@picnixz picnixz merged commit 1281b15 into sphinx-doc:master Mar 7, 2024
7 checks passed
@picnixz picnixz removed the awaiting:response Waiting for a response from the author of this issue label Mar 7, 2024
@afranke afranke deleted the css-js-typos branch March 7, 2024 20:04
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 7, 2024
@AA-Turner AA-Turner added this to the 7.3.0 milestone Jul 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants