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

Add -moz-initial keyword #243

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Mouvedia
Copy link

@Mouvedia Mouvedia commented Feb 4, 2023

@Mouvedia
Copy link
Author

@lahmatiy is there anything I can do to help this PR being merged?

@lahmatiy
Copy link
Member

@Mouvedia I'm somewhat puzzled. I delved into the history of the keyword and discovered that it became unprefixed in Firefox 19, which was released over a decade ago (February 2013). MDN indicates that it was removed in Firefox 24 (link), although I wasn’t able to corroborate this with the release notes. Given this, I'm struggling to see the necessity to support such an outdated keyword. My research hasn’t revealed where and why this keyword was initially deemed essential. Could it be that I've overlooked something?

@Mouvedia
Copy link
Author

Mouvedia commented Sep 28, 2023

@lahmatiy we are using csstree as a dependency in a rule of stylelint. Stylelint is used not only on greenfield projects but also on existing codebases that need to be maintained. These codebases may be considered old by your standards but—for example—the sass mixins that were used back then to achieve cross-browser styling shouldn't be considered invalid.

What I am advocating in essence is, if it doesn't go against csstree goals outright, it's harmless and useful to other projects, the addition should be warranted.

My research hasn’t revealed where and why this keyword was initially deemed essential.

Probably because back then every experimental properties or values were prefixed.
i.e. they were testing initial

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