-
Notifications
You must be signed in to change notification settings - Fork 115
Enhancement: Removing unused @chaset at-rulers and possible encoding convertions #4
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
Comments
It happened many times for me to have to deploy utf8 CSS (that contain for example utf8 chars for I think we should just put one at the top (see my example before) if one If I'm not sure removing the charset is a good idea, but handling conversion is a nice one. |
I think that simplest way is to think about any file without But you are probably right, if we removed Something like this:
|
Exactly. I'll update this a bit
|
One thing I think we won't need to handle is conversion, this will be done automatically because we will read with declared encoding (eg: read |
I think that original is more bulletproof. :) Consider this situation:
If we leave |
Good point.
|
Yes, exactly. |
@MoOx Does this make sense now? |
Why not now? |
cssnext issue was closed and I always use only utf-8 :) |
Yeah but you are not alone? What is the problem of keeping this issue open? |
@MoOx Will be fixed in 10 or more version ... :) |
Nobody asked for this in a while, I guess we can close... |
Never used
@charset
at-rules myself and not really understand why somebody may want to make css files with different@charset
values (importing css from external libraries perhaps?), but specificaition allows it.Good idea will probably to do the following: If any of imported files have
@charset
, then convert file to UTF-8 from this charset and remove@charset
at-rule.But it is not really a urgent feature, it can be done sometimes later.
The text was updated successfully, but these errors were encountered: