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

Add container for busuanzi counter widget #1154

Merged
merged 6 commits into from
Sep 15, 2019

Conversation

Liam0205
Copy link
Contributor

@Liam0205 Liam0205 commented Sep 6, 2019

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines.
  • Tests for the changes was maked (for bug fixes / features).
    • Muse | Mist have been tested.
    • Pisces | Gemini have been tested.
  • Docs in NexT website have been added / updated (for features).

PR Type

What kind of change does this PR introduce?

  • Bugfix.
  • Feature.
  • Code style update (formatting, local variables).
  • Refactoring (no functional changes, no api changes).
  • Build related changes.
  • CI related changes.
  • Documentation content changes.
  • Other... Please describe:

What is the current behavior?

Issue resolved: N/A

For busuanzi counter, if the script is not correctly loaded and executed, the display would be broken. Example:

阅读次数:   // here is supposed to be the page_pv counter value.

What is the new behavior?

For busuanzi counter, if the script is not correctly loaded and executed, the container span will remain hidden, rather than broken display.

  • Screenshots with this changes: N/A
  • Link to demo site with this changes: N/A

How to use?

Automatically enable, if busuanzi is enabled.

Does this PR introduce a breaking change?

  • Yes.
  • No.

This will hide the `span` until `busuanzi`'s script is correctly loaded.
@welcome
Copy link

welcome bot commented Sep 6, 2019

Thanks so much for opening your first PR here!

@CLAassistant
Copy link

CLAassistant commented Sep 6, 2019

CLA assistant check
All committers have signed the CLA.

@Liam0205 Liam0205 force-pushed the dev-busuanzi_container branch from 3376bb6 to 00cd72e Compare September 6, 2019 07:19
@ivan-nginx ivan-nginx self-requested a review September 6, 2019 07:48
@stevenjoezhang stevenjoezhang changed the title [busuanzi] add container for page_pv/site_pv/site_uv Add container for busuanzi counter widget Sep 15, 2019
@stevenjoezhang stevenjoezhang added this to the v7.4.1 milestone Sep 15, 2019
@stevenjoezhang stevenjoezhang merged commit 4f757e4 into theme-next:master Sep 15, 2019
@welcome
Copy link

welcome bot commented Sep 15, 2019

Congrats on merging your first pull request here! 🎉 How awesome!

@Liam0205 Liam0205 deleted the dev-busuanzi_container branch September 15, 2019 14:07
@1v9
Copy link
Member

1v9 commented Sep 20, 2019

@all-contributors please add @Liam0205 for code

@allcontributors
Copy link
Contributor

@1v9

I've put up a pull request to add @Liam0205! 🎉

tongluyang pushed a commit to tongluyang/hexo-theme-next that referenced this pull request Nov 19, 2019
* [layout] _macro/post.swig: add container to busuanzi page_pv.

* [layout] _partials/analytics/busuanzi-counter.swig: add container to busuanzi site_pv & site_uv.

This will hide the `span` until `busuanzi`'s script is correctly loaded.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants