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

sidebar menu - creating horizontal scroll bar #190

Closed
lb- opened this issue Aug 17, 2022 · 0 comments · Fixed by #191
Closed

sidebar menu - creating horizontal scroll bar #190

lb- opened this issue Aug 17, 2022 · 0 comments · Fixed by #191

Comments

@lb-
Copy link
Member

lb- commented Aug 17, 2022

The first thing that jumps out is that the sidebar menu has a horizontal scroller.

Preview: https://wagtail--9013.org.readthedocs.build/en/9013/

Originally posted by @zerolab in wagtail/wagtail#9013 (comment)
lb- pushed a commit to lb-/sphinx_wagtail_theme that referenced this issue Aug 17, 2022
multiple minor styling fixes
8954073 
- fix gap after content within admonitions, avoid styling every last element inside, only the container - fixes wagtail#107
- use variable for the max-width of the version picker (300px on bootstrap variables)
- add layout scss to fix some global style issues (already on Wagtail core)
- add .sidebar-container so that we can have a max width on navbar - fixes wagtail#99
- use .sidebar-container to allow for border-right on larger devices to match designs - fixes wagtail#59
- resolve styling issues with sidebar container & overflow scroll - fixes wagtail#190
- avoid using coded styles when bootstrap utility classes are suitable in the sidebar & search box
- add high contrast focus outline to focusable (tabbable) elements - fixes wagtail#23
@lb- lb- mentioned this issue Aug 17, 2022
Merged
lb- pushed a commit to lb-/sphinx_wagtail_theme that referenced this issue Aug 17, 2022
multiple minor styling fixes
20354f9 
- fix gap after content within admonitions, avoid styling every last element inside, only the container - fixes wagtail#107
- use variable for the max-width of the version picker (300px on bootstrap variables)
- add layout scss to fix some global style issues (already on Wagtail core)
- add .sidebar-container so that we can have a max width on navbar - fixes wagtail#99
- use .sidebar-container to allow for border-right on larger devices to match designs - fixes wagtail#59
- resolve styling issues with sidebar container & overflow scroll - fixes wagtail#190
- avoid using coded styles when bootstrap utility classes are suitable in the sidebar & search box
- add high contrast focus outline to focusable (tabbable) elements - fixes wagtail#23
@lb- lb- closed this as completed in #191 Aug 18, 2022
@lb- lb- closed this as completed in 5301999 Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix multiple minor styling issues lb-/sphinx_wagtail_theme
1 participant
@lb-