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.

Sign up for GitHub

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

Jump to bottom

Core media type fonts #872

Closed
mattgarrish opened this issue Nov 12, 2018 · 2 comments
Closed

Core media type fonts #872

mattgarrish opened this issue Nov 12, 2018 · 2 comments
Assignees
@mattgarrish
Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: has PR The issue is being processed in a pull request
Milestone
4.2.0-beta

Comments

@mattgarrish
Copy link
Member

This is a placeholder issue until w3c/epub-specs#1158 is resolved. Either we need to update the allowed core media types for font/woff2 and application/font-sfnt, or we should stop enforcing fallbacks for font resources.
@mattgarrish mattgarrish added spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: accepted Ready to be further processed labels Nov 12, 2018
@mattgarrish mattgarrish mentioned this issue Nov 13, 2018
Closed
@mattgarrish
Copy link
Member Author

The final word on this is that we shouldn't require fallbacks for font resources. I believe this is the case already for a number of font types, but needs to be expanded to cover the newer font/* types.

@mattgarrish mattgarrish self-assigned this Feb 15, 2019
@mattgarrish mattgarrish mentioned this issue Feb 15, 2019
Merged
@rdeltour rdeltour added this to the 4.2.0-beta milestone Feb 19, 2019
@rdeltour rdeltour added status: has PR The issue is being processed in a pull request and removed status: accepted Ready to be further processed labels Feb 19, 2019
rdeltour pushed a commit that referenced this issue Feb 19, 2019
@mattgarrish @rdeltour
feat: update recognized font Core Media types
ac2f1bd 
Add the following media types as recognized Core Media Types for fonts:
- font/otf
- font/ttf
- font/woff
- font/woff2
- application/font-sfnt

Also add a WOFF2 declaration to the core media type test.

Fix #872, Fix #339
@rdeltour
Copy link
Member

Fixed in #972.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mattgarrish mattgarrish

Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: has PR The issue is being processed in a pull request
Projects
None yet
Milestone
4.2.0-beta
Development

No branches or pull requests
2 participants
@rdeltour @mattgarrish