-
Notifications
You must be signed in to change notification settings - Fork 60
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
Encoding of CSS files #1628
Comments
There was a discussion about dropping UTF-16 in #50 It was also reconfirmed for CSS in a telecon discussion (the referenced thread in google groups appears to be lost to time now, though) The only concern I'd have with making a change is whether we risk breaking existing content. Maybe we should just advise against the use rather than disallow it? |
I had the same concern... |
@xfq @mattgarrish would it be possible to change the text as follows: from
to
@xfq please follow the link to deprecated to see what it means in the context of EPUB 3.3 However, I see UTF-16 mentioned in two more places in the document: I believe the media type registration must stay, because it is "legal" to use UTF-16 (because it is just deprecated). I would think, however, that the same change should be made on the XML case as for CSS. WDYT? |
Isn't this a bit of a contradiction? If it must be encoded in utf-8, then that negates using utf-16 at all. This is kind of an oddball case since we're only deprecating half the requirement. If we go this route, maybe it might make more sense to phrase it along the lines of: "It MUST be encoded in UTF-8 or UTF-16, but the use of UTF-16 is now deprecated." That would be consistent with not invalidating any utf-16 content that might be out there while recommending it no longer be used. |
You are right. Let us go this way if we get the blessing of the WG (on the call tomorrow) |
It appears the general web direction is to use utf-8 exclusively, so a broader recommendation against using utf-8 for xml, too, seem in keeping with that trend. The Encoding spec says this:
UTF-16 is now considered a legacy encoding format. It also appears that only a fraction of a percent of web content uses it (although that's not necessarily a good barometer of publishing use). |
Just for reference... CSS says stylesheets should be UTF-8
|
The issue was discussed in a meeting on 2021-04-23 List of resolutions:
View the transcript1.1. CSS files encodingSee github issue #1628. See github pull request #1645. Dave Cramer: first, encoding of CSS files Ivan Herman: not sure how much books currently use UTF16, but epubcheck having to warn about this means that some changes will have to be made to epubcheck Matt Garrish: i'm not sure how much epubcheck even looks at CSS Brady Duga: are there enough positive benefits to go ahead with this? Dave Cramer: we're trying to work with i18n WG Brady Duga: is it deprecated in CSS? Dave Cramer: they say you SHOULD use UTF8 Brady Duga: i would say "you should use UTF8", but i'm not sure about deprecating UTF16 Ivan Herman: i suspect there are tools and scripts which simply do not work with UTF16 Brady Duga: i'd be surprised if there are a lot of tools out there that don't support both encodings Matt Garrish: we are trying to align with the web, and what the other specs are saying Dave Cramer: i understand the concerns with deprecation Matt Garrish: that puts us more or less back in the same place, there's going to be an epubcheck warning Dave Cramer: and then people also have to go look up the definition of "deprecated", so maybe not saying that is more straightforward Ivan Herman: for sake of argument, Rust for example, which is gaining popularity, only uses UTF8
Ivan Herman: but yes, let's change the PR
|
CSS files in an EPUB package MUST be encoded in UTF-8 or UTF-16. The i18n WG supports the idea of using UTF-8 only. Why is UTF-16 mentioned here?
The text was updated successfully, but these errors were encountered: