Skip to content

[Ordered Comments] Flush to the spec#3844

Merged
nex3 merged 1 commit intomainfrom
flush-ordered-comments
Apr 24, 2024
Merged

[Ordered Comments] Flush to the spec#3844
nex3 merged 1 commit intomainfrom
flush-ordered-comments

Conversation

@nex3
Copy link
Contributor

@nex3 nex3 commented Apr 20, 2024

See #2848

@nex3 nex3 requested a review from Goodwine April 20, 2024 00:38
* If `statement` is an `@import` rule, insert a copy of `statement` in `css`
after the last `@import` rule, or at the beginning of `css` if it doesn't
contain any `@import` rules.
* If `statement` is a marked comment, ignore it.
Copy link
Member

@Goodwine Goodwine Apr 23, 2024

Choose a reason for hiding this comment

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

I don't see where statement gets marked, I think "them mark it" is missing in the bullet point below 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's marked on line 210, above.

Copy link
Member

Choose a reason for hiding this comment

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

That bullet marks a "comment" rather than a statement

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right—this bullet checks if statement is a comment that is marked.

Copy link
Member

Choose a reason for hiding this comment

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

🤦🏽‍♂️ for some reason I read this as "If statement is marked, ignore it" lol

* If `statement` is an `@import` rule, insert a copy of `statement` in `css`
after the last `@import` rule, or at the beginning of `css` if it doesn't
contain any `@import` rules.
* If `statement` is a marked comment, ignore it.
Copy link
Member

Choose a reason for hiding this comment

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

🤦🏽‍♂️ for some reason I read this as "If statement is marked, ignore it" lol

@nex3 nex3 merged commit 09c89ec into main Apr 24, 2024
@nex3 nex3 deleted the flush-ordered-comments branch April 24, 2024 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants