-
Notifications
You must be signed in to change notification settings - Fork 28
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
Upgrade Bootstrap to version 5 #254
Comments
it used Bootstrap CDN ?? |
It's a front-end dependency see https://github.com/wagtail/sphinx_wagtail_theme/blob/main/package.json#L23 |
Just a thought. Is it worth looking at something more lightweight instead of Bootstrap. Maybe too lightweight - https://open-props.style/ - but super flexible with CSS Custom Properties as the core method of designs. Not sure on other options, maybe we need to review the actual Bootstrap parts used. Also need to consider the ability to customise and override things for others. |
Currently the Theme is using Bootstrap 4 and it would be good to upgrade to version 5.
Split from #252 / #250
It may be worth considering a small clean up of unused styles in the process. See #64
The text was updated successfully, but these errors were encountered: