sphinx-book-theme icon indicating copy to clipboard operation
sphinx-book-theme copied to clipboard

Fix .bd-sizebar-secondary and .secondary-toggle breakpoint

Open sinsong opened this issue 1 year ago • 1 comments

My friend find that .bd-sidebar-secondary collapse before .secondary-toggle appear. Which means, viewport width between 1200px (xl breakpoint) and 992px (lg breakpoint), both sidebar and toggle button disappear.

I find that a rule come from pydata-sphinx-theme interact with .bd-sidebar-secondary cause collapse less that 1200px (exactly at 1199.98px). With the breakpoint xl.

So this PR is for synchronize .secondary-toogle button with .bd-sidebar-secondary collapse through align breakpoint to xl.

sinsong avatar Jan 14 '24 16:01 sinsong

Thanks for submitting your first pull request! You are awesome! :hugs:
If you haven't done so already, check out EBP's Code of Conduct and our Contributing Guide, as this will greatly help the review process.
Welcome to the EBP community! :tada:

welcome[bot] avatar Jan 14 '24 16:01 welcome[bot]

This matches my read of the upstream code, which sets $breakpoint-sidebar-secondary: xl;: https://github.com/pydata/pydata-sphinx-theme/blob/05fd48a8159d6d4046a06df6688efae0dacf34eb/src/pydata_sphinx_theme/assets/styles/variables/_layout.scss#L21-L22

agoose77 avatar Jun 12 '24 12:06 agoose77