-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
add hover text for PV UV #110
Conversation
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.
Please, add i18n
support.
update |
For example, here is hexo-theme-next/layout/_macro/post.swig Line 211 in 82ac7af
This mean what this value will be read from current language in hexo-theme-next/languages/zh-CN.yml Line 31 in 82ac7af
E.g. So, no need to add language value into NexT config. Need just add Need to refactor all previous langs values from config to languages files:
P.S. And maybe add hexo-theme-next/languages/_en.yml Lines 53 to 57 in 82ac7af
|
i have localized about NO, i saw your suggestion above, It seems no need to add |
My opinion is to just leave the Just provide the Because there is a grammar issue: When we say Site visited by 100 people, translated into Chinese will be 网站被 100 人所访问 And in
But in
Due to the grammar difference, I suggest not to touch the Eg:
|
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number(s): #108
What is the new behavior?
Add hover text for some confusing icon. In the above case , someone maybe be confusing with two date.
Hover text, such as “created” “modified” maybe more favorable in many cases.
How to use?
In NexT
_config.yml
:Does this PR introduce a breaking change?