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

modify the bug of sidebar-inner #2180

Closed
wants to merge 1 commit into from
Closed

Conversation

iNiL0119
Copy link

@iNiL0119 iNiL0119 commented Nov 4, 2018

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines.
  • Tests for the changes have been added (for bug fixes / features).
    • Muse | Mist have been tested.
    • Pisces | Gemini have been tested.
  • Docs have been added / updated (for bug fixes / 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?

init the blog and link to localhost:4000 with Next, click the title Hello World then show the detail of the article, then scorll to bottom and refresh(MacOS commond + shift +R / commond + R) repeatedly, and you can see the class element 'sidebar-inner' has class name 'affix-bottom' with top 915px sometimes.
Issue Number(s): N/A

What is the new behavior?

Description about this pull, in several words...

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

How to use?

In NexT _config.yml:

...

Does this PR introduce a breaking change?

  • Yes.
  • No.

@ivan-nginx
Copy link
Collaborator

NexT is rebased into organization repo.
If you want new feature, please, create pull in NexT v6.x repo.
There is instructions on English or Chinese how to update from v5.1.x to v 6.x

You also may read this for details.

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.

2 participants