-
-
Notifications
You must be signed in to change notification settings - Fork 101
Convert Basic theme from Boostrap 3 to Boostrap 4 and use sass instead of lessc (also fixes #1770) #1908
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
Conversation
Removing draft status as it is unclear when a new XStatic-Bootstrap release will be available (see pending MR). A workaround for using "./m css" would be copying scss files to xstatic/pkg/bootstrap/data/ in the local python package cache. |
Testing on Windows 10 fails with:
The basic theme.css file was updated:
|
@RogerHaase Thanks for looking into the PR. Did you actually copy the bootstrap 4.5.3 scss files into the XStatic-Bootstrap site-packages folder as mentioned in my comment above? Running "./m css" requires those files to be installed but they currently are not part of the latest XStatic-Bootstrap release.
With the scss files installed I was able to run the command without errors and produce the CSS files for the Basic theme. The sass invocation will log a good amount of deprecation warnings though. My plan was to fix this stuff, when finally switching to Bootstrap 5 (current) in a separate PR. |
After copying scss directory I was able to successfully run To repeat the process in more detail:
I am doing a b3 release this week. I think it would be best to merge this after the release. |
@roland-ruedenauer Please resolve the conflict, then I will merge this. |
@roland-ruedenauer IMO you can remove |
6aa1ec6
to
10945f3
Compare
I've rebased the changes onto the master branch |
This PR should fix the remaining issues listed in #1770.
Problem with running
./m css
for now:The package XStatic-Bootstrap installed with moin does not contain the
scss
files required by the sass compiler.The CSS files for the Basic theme were created by locally patching the bootstrap package before running
./m css
.This is the reason I chose to use the draft status for this PR.
A new release of XStatic-Bootstrap should provide the required
scss
files.Regarding the use of Font Awesome icons:
Making use of the regular icons instead of the solid ones would probably make the Basic theme look a bit nicer.
Please note that Boostrap switched from Less to Sass with version 4 already. Moin was using the Bootstrap 4 CSS files even before this PR.
It would be good if someone could test the changes made in this PR and check, if there are still issues left to fix when using the Basic theme.