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.

Sign up for GitHub

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

Jump to bottom

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

Closed
3 tasks done
Walkline80 opened this issue Jun 30, 2021 · 14 comments · Fixed by #308
Closed
3 tasks done

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

Walkline80 opened this issue Jun 30, 2021 · 14 comments · Fixed by #308
Labels
Bug Something isn't working Solved
Milestone
8.6.1

Comments

@Walkline80
Copy link

Walkline80 commented Jun 30, 2021 •
edited
Loading

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

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 





















 




















@Walkline80
Walkline80
added
the

Bug

Something isn't working
label
Jun 30, 2021












@welcome












 








Copy link
 

 













welcome
bot

commented
Jun 30, 2021
















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


































 













@PaperStrike












 








Copy link
 

 







Member








PaperStrike

commented
Jun 30, 2021

•






edited








 Loading






















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


































 













@stevenjoezhang












 








Copy link
 

 







Member




















可能是这个问题？ #243


































 













@Walkline80












 








Copy link
 

 







Author






















可能是这个问题？ #243




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


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



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


































 













@stevenjoezhang












 








Copy link
 

 







Member




















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


































 













@Walkline80












 








Copy link
 

 







Author






















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




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


































 













@ljcbaby












 








Copy link
 

 







Member








ljcbaby

commented
Jul 1, 2021

•






edited








 Loading






















更新到 #243 合并后的commit就出现了这个问题

我不应该没看前面的测试就合并到

我自己的https://blog.ljcbaby.top也是同样的问题


































 




















@ljcbaby
ljcbaby
mentioned this issue

Jul 1, 2021









 Merged

















12 tasks















@ljcbaby












 








Copy link
 

 







Member








ljcbaby

commented
Jul 1, 2021
















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


































 


















@stevenjoezhang
stevenjoezhang
added this to the
8.6.1 milestone
Jul 1, 2021












@Walkline80












 








Copy link
 

 







Author






















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




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


































 



















ljcbaby
added a commit
that referenced
this issue

Jul 2, 2021














@ljcbaby
 






Fix scroll in sidebar (#308)
























5dc546c










See #307



















@ljcbaby












 








Copy link
 

 







Member








ljcbaby

commented
Jul 2, 2021
















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


































 













@stevenjoezhang












 








Copy link
 

 







Member




















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


































 



















@stevenjoezhang
stevenjoezhang
added this to the 8.6.1 milestone
Jul 3, 2021












@stevenjoezhang












 








Copy link
 

 







Member




















@Walkline80 能否帮忙测试一下这个 Pull Request: #360

进入theme/next目录执行


git fetch
git checkout -b sidebar origin/sidebar



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


































 













@Walkline80












 








Copy link
 

 







Author






















@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


预览地址在这里


































 
















lingyf
pushed a commit
to lingyf/hexo-theme-next
that referenced
this issue

Jan 27, 2022














@ljcbaby
 






Fix scroll in sidebar (next-theme#308)























e5610a4










See next-theme#307



















@github-actions












 








Copy link
 

 

























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.


































 


















@github-actions
github-actions
bot
locked as resolved and limited conversation to collaborators
Aug 31, 2022













 










Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.









 







Assignees



No one assigned







Labels





Bug

Something isn't working

Solved











Projects







None yet


 







Milestone




 
8.6.1













Development



Successfully merging a pull request may close this issue.




 
Fix scroll in sidebar

 
next-theme/hexo-theme-next




 











4 participants





@Walkline80
 
@stevenjoezhang
 
@PaperStrike
 
@ljcbaby