Skip to content

Han Support https://hanzi.pro/ #1558 #1222 #698#1598

Merged
iissnan merged 2 commits intoiissnan:masterfrom
geekrainy:feature/han
Apr 19, 2017
Merged

Han Support https://hanzi.pro/ #1558 #1222 #698#1598
iissnan merged 2 commits intoiissnan:masterfrom
geekrainy:feature/han

Conversation

@geekrainy
Copy link
Copy Markdown
Contributor

No description provided.



{% if theme.han %}
<link rel="stylesheet" media="all" href="//cdnjs.cloudflare.com/ajax/libs/Han/3.2.7/han.min.css">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cloudflare的CDN在国内很慢吧?
建议不要直接引用CDN,直接将资源文件放在主题里,同时提供CDN选项,可以参考其他资源文件。

@iissnan iissnan merged commit 3a390ee into iissnan:master Apr 19, 2017
@iissnan
Copy link
Copy Markdown
Owner

iissnan commented Apr 19, 2017

Thanks. 👍

@geekrainy geekrainy deleted the feature/han branch April 19, 2017 04:33


<div class="post-body" itemprop="articleBody">
<div class="post-body han-init-context" itemprop="articleBody">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is the switch for han-init-context?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's just a hook. When Han is not enabled, it will not have any effect.

Copy link
Copy Markdown
Collaborator

@ivan-nginx ivan-nginx Apr 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But it's will be in default class style. It was hard to do this?

<div class="post-body{% if theme.han %} han-init-context{% endif %}" itemprop="articleBody">

Copy link
Copy Markdown
Contributor Author

@geekrainy geekrainy Apr 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your suggestion. :) 756c49c

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In ideal, this switches must be anywhere on any custom options.

@ivan-nginx
Copy link
Copy Markdown
Collaborator

References to #1558 #1222 #698.

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.

4 participants