-
Notifications
You must be signed in to change notification settings - Fork 107
CSS property not applied: font-variant-numeric: oldstyle-nums
#294
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 is supported (there might be a few caveats with inheritance, but they shouldn't trigger in the case of this book) koreader/koreader#5821 (comment) The issue with this book is that our CSS parser is confused by some stuff at the start of epub/css/core.css, which makes it skip the first declaration (bug koreader/koreader#7199): @charset "utf-8";
@namespace epub "http://www.idpf.org/2007/ops"; /* this is what we don't handle well */
dummy {} /* add a first dummy declaration and the next one works */
body{
font-variant-numeric: oldstyle-nums;
hyphens: auto;
color: blue; /* just to be sure it applies */
-epub-hyphens: auto;
} |
I see. Should this issue be closed then in favor of the linked koreader bug? |
@virxkane : I'm working on this CSS parsing issue (#294 (comment)) - so you don't need to (seeing you're browsing all the issues here, you might get the idea to go at it :) |
Ok. But, I was not going to :) |
The property is being used in the core CSS of standardebooks.org epubs, see for example https://standardebooks.org/ebooks/p-g-wodehouse/jeeves-stories, and it is not applied to the rendered text even if the font supports oldstyle-nums. I'm not familiar with the project and therefore the following assumption may be wrong, but a search reveals that the feature should be supported given that

font-variant: small-caps
found nearby is being properly rendered.Here pictured in comparison cr3 (left) and ebook-viewer of calibre which renders
oldstyle-nums
as intended:1915 and 1925 -- oldstyle numerals, P. G. Wodehouse and Standard Ebooks project -- small capitals.
The text was updated successfully, but these errors were encountered: