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.

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

feat: Make web fonts available #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ghostrick
Copy link

Do not expand the link tag if it contains a specific domain.
https://amp.dev/ja/documentation/guides-and-tutorials/develop/style_and_layout/custom_fonts/

@t-jindai
Copy link
Owner

@ghostrick Thank you for your PR and sorry for late reply...

According to the document, There are 2 ways to use custom webfonts.
One is link tag and another is font-face.

IMO current implementation uses font-face way so there is no problem to use webfonts.

Could you explain why you want to use link tag solution instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants