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

Document the moz-document breaking change on the sass website. #667

Merged
merged 2 commits into from Sep 7, 2022

Conversation

stof
Copy link
Contributor

@stof stof commented Sep 3, 2022

Currently, the link in the deprecation warning goes to a website that does not even exist anymore, relying on the web-archive. And this website does not give any details about the Sass breaking change.

This references this breaking change on the website like the others (still linking to the Firefox deprecation announcement on web-archive as part of the explanation).

Copy link
Member

@Goodwine Goodwine left a comment

Choose a reason for hiding this comment

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

Can you also add an entry in data/documentation.yml?

---

Sass has historically supported a special parsing for the `@-moz-document` rule.
As [Firefox dropped support for them], Sass will also drop support.
Copy link
Member

Choose a reason for hiding this comment

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

Per this comment - sass/dart-sass#378 (comment)
Sass won't drop support entirely, only for the special parsing in favor of interpolation within unknown at-rules

@stof stof force-pushed the moz_document_rule branch 2 times, most recently from 42c7281 to d89ab68 Compare September 7, 2022 07:43
@stof
Copy link
Contributor Author

stof commented Sep 7, 2022

@Goodwine updated

@Goodwine Goodwine merged commit df332e0 into sass:main Sep 7, 2022
@stof stof deleted the moz_document_rule branch September 7, 2022 22:00
asaf400 pushed a commit to asaf400/ass-site that referenced this pull request Apr 18, 2024
)

* Document the moz-document breaking change on the sass website.
asaf400 pushed a commit to asaf400/ass-site that referenced this pull request Apr 18, 2024
)

* Document the moz-document breaking change on the sass website.
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.

None yet

2 participants