Skip to content

Conversation

@luo-zhan
Copy link
Contributor

PR Checklist

  • The commit message follows guidelines for NexT.
  • 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

  • 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?

The reading progress bar is a little stuck.

What is the new behavior?

Improve the accuracy of numerical value to make the progress bar smoother.

How to use?

In NexT _config.yml:

reading_progress:
  enable: true
  ...

Does this PR introduce a breaking change?

  • Yes.
  • No.

@welcome
Copy link

welcome bot commented Dec 20, 2019

Thanks so much for opening your first PR here!

@1v9 1v9 added this to the v7.7.0 milestone Dec 21, 2019
@stevenjoezhang stevenjoezhang modified the milestones: v7.7.0, 7.8.0 Dec 31, 2019
@stevenjoezhang stevenjoezhang merged commit ec80c64 into theme-next:master Jan 14, 2020
@welcome
Copy link

welcome bot commented Jan 14, 2020

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

voidking pushed a commit to voidking/hexo-theme-next-2021 that referenced this pull request Oct 28, 2020
@CLAassistant
Copy link

CLAassistant commented Apr 10, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ stevenjoezhang
❌ luo-zhan
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

4 participants