从8.2更新到8.5后发现侧边栏目录部分全部出现垂直滚动条

Walkline80 commented 3 years ago

Please follow this Issue template to provide relevant information, such as source code repository, website URL and screenshots, which will help us investigate. 请按照此 Issue 模版提供相关信息，例如源码仓库、网站链接和屏幕截图，这将有助于我们进行调查。

Issue Checklist

[x] I am using NexT version 8.0 or later.
[x] I have already read the Troubleshooting page of Hexo and Troubleshooting page of NexT.
[x] I have already searched for current issues, which does not help me.

Expected behavior

Actual behavior

Links to demo site with this issue:
Links to repository or source code of the blog:
Screenshots:

Steps to reproduce the behavior

Environment Information

Node.js and NPM Information

hexo: 5.4.0
hexo-cli: 4.3.0
os: win32 10.0.19043
node: 12.14.0
v8: 7.7.299.13-node.16

Package dependencies Information

Hexo Configuration

NexT Configuration

发现8.2之后增加如下配置文件，不过设置以后也是无效

body_scrollbar:
  # Place the scrollbar over the content.
  overlay: false
  # Present the scrollbar even if the content is not overflowing.
  stable: false

Other Information

welcome[bot] commented 3 years ago

Thanks for opening this issue, maintainers will get back to you as soon as possible!

PaperStrike commented 3 years ago

这个 body_scrollbar 只是作用于 <body> 页面主体的滚动条的，和侧边目录关系不大。请尽量提供更多关键信息，如测试页面、测试仓库、安装方式（GitHub or npm）等，以帮助其他人排查问题。

stevenjoezhang commented 3 years ago

可能是这个问题？ https://github.com/next-theme/hexo-theme-next/pull/243

Walkline80 commented 3 years ago

可能是这个问题？ #243

不太确定是不是那个问题啊，我看通篇都在讨论开源协议。。。。

back2top:
  enable: true
  # Back to top in sidebar.
  sidebar: false
  # Scroll percent label in b2t button.
  scrollpercent: false

查了一通发现是侧栏的back2top按钮的问题，禁用就好了

stevenjoezhang commented 3 years ago

8.6.0版本刚刚发布了，更新一下试试

Walkline80 commented 3 years ago

8.6.0版本刚刚发布了，更新一下试试

进入theme/next目录执行git pull之后执行hexo clean最后hexo d问题依旧，实际效果请访问这里

ljcbaby commented 3 years ago

更新到 #243 合并后的commit就出现了这个问题我不应该没看前面的测试就合并到我自己的https://blog.ljcbaby.top也是同样的问题

ljcbaby commented 3 years ago

尝试一下 #308 ，应该能解决问题

Walkline80 commented 3 years ago

尝试一下 #308 ，应该能解决问题

可以了，问题已解决，感谢！~

ljcbaby commented 3 years ago

已经合并，等待新npm版本发布

stevenjoezhang commented 3 years ago

如果发现 8.6.0 还有什么其他的严重问题，可以回复一下。今天晚些时候会发布 8.6.1 版本。

stevenjoezhang commented 3 years ago

@Walkline80 能否帮忙测试一下这个 Pull Request: https://github.com/next-theme/hexo-theme-next/pull/360 进入theme/next目录执行

git fetch
git checkout -b sidebar origin/sidebar

这里把侧栏的高度计算重做了，但不知道会不会导致以前的问题再次出现。谢谢！

Walkline80 commented 3 years ago

@Walkline80 能否帮忙测试一下这个 Pull Request: #360 进入theme/next目录执行
git fetch
git checkout -b sidebar origin/sidebar
这里把侧栏的高度计算重做了，但不知道会不会导致以前的问题再次出现。谢谢！

测试了一下，好像没有问题

$ git fetch
$ git checkout -b sidebar origin/sidebar
$ cd ../..
$ hexo s
INFO  Validating config
INFO  ==================================
  ███╗   ██╗███████╗██╗  ██╗████████╗
  ████╗  ██║██╔════╝╚██╗██╔╝╚══██╔══╝
  ██╔██╗ ██║█████╗   ╚███╔╝    ██║
  ██║╚██╗██║██╔══╝   ██╔██╗    ██║
  ██║ ╚████║███████╗██╔╝ ██╗   ██║
  ╚═╝  ╚═══╝╚══════╝╚═╝  ╚═╝   ╚═╝
========================================
NexT version 8.7.0
Documentation: https://theme-next.js.org
========================================
INFO  Start processing
INFO  Hexo is running at http://localhost:4000 . Press Ctrl+C to stop.

$ hexo d

预览地址在这里

github-actions[bot] commented 2 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. It is possible issue was solved or at least outdated. Feel free to open new for related bugs.

next-theme / hexo-theme-next