-
Notifications
You must be signed in to change notification settings - Fork 16
Support parsing @font-face
#39
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
Conversation
|
@LaurenzV PTAL. The current implementation only targets |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, maybe add a similar test like font_face_02_mixed_with_rules but where font-face is not the first rule?
I think for now we don't need general at-rule support, the font-face one is by far the most important one.
1f1af7d to
afaeef9
Compare
afaeef9 to
fbf1e15
Compare
|
Thank you, I've added the new test |
|
Doc failure needs to be fixed and then it should be good to go. |
|
Actually, let me just fix it with a separate PR. |
|
Hmm, is it possible that you haven't rebased onto the newest main? I can't find the line. |
error[E0557]: feature has been removed --> src/lib.rs:42:29 | 42 | #![cfg_attr(docsrs, feature(doc_auto_cfg))] | ^^^^^^^^^^^^ feature has been removed | = note: removed in 1.92.0; see <rust-lang/rust#138907> for more information = note: merged into `doc_cfg` error: Compilation failed, aborting rustdoc
Fixed: |
|
Nevermind, I was the one looking at an old version. 😅 |
|
Thanks! |
In the future, we may also add
@font-palette-valuesand@font-feature-values.Part of linebender/resvg#541