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.

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

Update to Bootstrap 4 #384

Closed
kulbhushanchand opened this issue Aug 11, 2018 · 13 comments
Closed

Update to Bootstrap 4 #384

kulbhushanchand opened this issue Aug 11, 2018 · 13 comments

Comments

@kulbhushanchand
Copy link

There are some issues when using latest bootstrap from cdnjs server. Here is the link to closed issue .
The theme works well on bootstrap ver 3.3.2 for both js and css.
However if I may request to update the repository to use the latest bootstrap, would it be possible, and right now I have not the expertise to do so.

@OCram85
Copy link
Contributor

OCram85 commented Sep 21, 2018

Upgrading to bootstrap 4 would cause a complete rework of the theme. Since bootstrap 4 itself is a rework of the framework, the majority of components changed.

@kulbhushanchand
Copy link
Author

Thanks for the feedback. Is it possible to make it free from bootstrap via custom css, like this theme - https://github.com/mmistakes/minimal-mistakes

@OCram85
Copy link
Contributor

OCram85 commented Sep 24, 2018

I personally think using the bootstrap framework for beautiful-jekyll has 2 huge advantages:

  • Customization to individual needs is much easier with a detailed underlying component based framework.
  • Using bootstrap guarantees you the help of a big community.

Like already said, it's only my personal opinion. But maybe it could be worth thinking about a reboot for this theme with bootstrap 4 - at a given time.

@daattali
Copy link
Owner

@kulbhushan-chand thanks for your suggestions. I don't think the theme will ever be completely standalone, using bootstrap as a core theme adds a lot of value and consistency. While I don't see the removal of bootstrap happening, upgrading to bootstrap 4 may happen in the future

@kulbhushanchand
Copy link
Author

@OCram85 @daattali Thanks for your feedback. You are right about bootstrap. Hope we will soon see the latest addition of bootstrap 4.

@danielmpetrov
Copy link
Contributor

Would an update to the latest Bootstrap 3 (v3.3.2 -> v3.4.1) be a welcome PR?

@daattali
Copy link
Owner

@danielmpetrov is there a big benefit to upgrading within bootstrap 3? I'm not too familiar with the new features of 3.4 vs 3.3

@danielmpetrov
Copy link
Contributor

@daattali Mostly bug fixes. It will also enable the theme to upgrade jQuery to version 3, which brings security fixes. There is an open PR for this as well - #357

@daattali
Copy link
Owner

Ah yes I do see that comment you made there. Are there any regression bugs or behaviour changes? I'd be happy to update to the latest bootstrap 3 if you can test it out a bit and make sure there are no breaking changes!

@danielmpetrov
Copy link
Contributor

Since it's a point release, there wouldn't be any breaking changes, but I share your concern @daattali. I can go through it in the upcoming days and open a PR if it works out.

@daattali
Copy link
Owner

daattali commented Jul 23, 2019 via email

@daattali daattali changed the title Update request to use the latest bootstrap Update to Bootstrap 4 Apr 2, 2020
@daattali daattali mentioned this issue May 1, 2020
@daattali
Copy link
Owner

daattali commented May 1, 2020

I've started the work on migrating to bs4 in a bs4 branch. It's a lot more work than I imagined, there are a lot of components that break and need to be rewritten. If anybody wants to help, PRs are welcome.

@daattali
Copy link
Owner

daattali commented May 2, 2020

Beautiful Jekyll now uses bootstrap 4.4.1

@daattali daattali closed this as completed May 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants