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

Remove IE6/7 prefix support #699

Closed
cupcakearmy opened this issue Mar 30, 2022 · 5 comments
Closed

Remove IE6/7 prefix support #699

cupcakearmy opened this issue Mar 30, 2022 · 5 comments

Comments

@cupcakearmy
Copy link

cupcakearmy commented Mar 30, 2022

Hey there, first of all: love the project!

Building with parcel fails because of:

*/
/* Nicolas Gallaghers Clearfix solution
   Ref: http://nicolasgallagher.com/micro-clearfix-hack/ */
.cf:before, .cf:after { content: " "; display: table; }
.cf:after { clear: both; }
.cf { *zoom: 1; }     <------------------------------------------------------- This here
.cl { clear: left; }
.cr { clear: right; }
.cb { clear: both; }
.cn { clear: none; }
/*

The prefix breaks moders css parsers as it's not valid CSS. Is IE6/7 support still relevant?
Would be great to drop that.

Related Issue: parcel-bundler/lightningcss#39

@leaumar
Copy link

leaumar commented Jun 23, 2022

Been the thorn in my side for months... Can't find a better replacement for basscss than tachyons (hucssley doesn't have a stable release yet), don't wanna use clunkier build tools than parcel, don't wanna keep using outdated parcel versions.

Judging from the git dates, this project isn't active anymore?

#701 has done the work already

@cupcakearmy
Copy link
Author

It's basically abbandoned at this point unfortunately :/

@jzwood
Copy link

jzwood commented Nov 7, 2022

Agreed! Let's get rid of

.cf {
    *zoom: 1;
}

@theslantedroom
Copy link

theslantedroom commented Oct 30, 2023

I guess this is abandoned... just had to copy the file into my repo and edit them, then uninstall tachyons

@mrmrs
Copy link
Member

mrmrs commented Nov 7, 2023

It isn't abandoned, it would be a breaking change to v4 to remove so this will not be removed in v4.

@mrmrs mrmrs closed this as completed Nov 7, 2023
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

No branches or pull requests

5 participants