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

Use "it" in 2 places on order page to avoid repetition #37618

Closed
Hexstream opened this issue Jan 12, 2025 · 4 comments · Fixed by #37648
Closed

Use "it" in 2 places on order page to avoid repetition #37618

Hexstream opened this issue Jan 12, 2025 · 4 comments · Fixed by #37648
Labels
Content:CSS Cascading Style Sheets docs good first issue A good issue for newcomers to get started with.

Comments

@Hexstream
Copy link

MDN URL

https://developer.mozilla.org/en-US/docs/Web/CSS/order

What specific section or headline is this issue about?

Syntax

What information was incorrect, unhelpful, or incomplete?

This should probably be just one sentence:

Since order is only meant to affect the visual order of elements and not their logical or tab order. order must not be used on non-visual media such as speech.

Like this:

Since order is only meant to affect the visual order of elements and not their logical or tab order, it must not be used on non-visual media such as speech.


I would use "it" here too to avoid repetition:

When order is set on an element whose parent's display property is not creating a flex or grid container, the order property has no effect.

Like this:

When order is set on an element whose parent's display property is not creating a flex or grid container, it has no effect.

What did you expect to see?

Less repetition.

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

No response

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@Hexstream Hexstream added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jan 12, 2025
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Jan 12, 2025
@Josh-Cena Josh-Cena added good first issue A good issue for newcomers to get started with. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Jan 12, 2025
@Hexstream
Copy link
Author

I am extremely confused.

What is the point of reiterating my own proposal to me except in worse formatting?

Is this AI-generated?

@Josh-Cena
Copy link
Member

I don't see anything other than your post. Perhaps you saw a response that was deleted? This often happens when someone clicks quote reply but didn't type anything.

@Hexstream
Copy link
Author

Hexstream commented Jan 13, 2025

It seems a mod has deleted that message, didn't you get an email notification for it?

(edit: Come to think of it, they may have deleted their own message... 😅)

Here it is:

github-bot-probably

@Josh-Cena
Copy link
Member

Indeed, I don't have email notifications on. Would have generated hundreds of emails each day then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs good first issue A good issue for newcomers to get started with.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants