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 Bootstrap 5.3.0 #487

Merged
merged 10 commits into from
Jun 1, 2023
Merged

Update Bootstrap 5.3.0 #487

merged 10 commits into from
Jun 1, 2023

Conversation

crftwrk
Copy link
Member

@crftwrk crftwrk commented May 31, 2023

This PR has a lot changed files, but it must not reviewed each of them. Checking the overall functionality is enough and has no breaking changes to existing installations.

Demo https://dev.bootscore.me (dark-mode switcher is in the footer)

  • Update to Bootstrap 5.3
  • Changing color classes to new one, bg-light > bg-body-tertiary, text-muted > text-body-tertiary etc.
  • Adding a fallback to old color classes
  • Update versions, readme.txt and readme.md
  • Some minor bugfixes and improvements, nothing special

Tested it on some live-sites and works fine.

There is another PR #483. This updates two templates for the upcoming WooCommerce 7.8. Tested both templates with current 7.7 and this has no issues. So, I think we can merge this as well. This will prevent us to create a new release in two weeks again.

@justinkruit if you like it, merge it! As usual, there is a drafted release ready. bootscore.me site is ready as well.

Edit: From here, I think we should decouple versions from Bootstrap versions, switch to a 3 digit number and take semantic versioning serious as criticized in some issues.

Closes #489
Closes #488
Closes #490

@crftwrk crftwrk marked this pull request as draft May 31, 2023 09:00
@crftwrk crftwrk marked this pull request as ready for review June 1, 2023 09:01
@crftwrk crftwrk requested a review from justinkruit June 1, 2023 09:01
@justinkruit
Copy link
Member

@crftwrk There is a merge conflict in this one, could you take a look at that?

Also, I agree on the semantic versioning. Do we want to start that with us beginning at 5.3.0? Or do we want to start a bit fresher, make it something like v6, and go from there? Not all that good with versioning often 😅 😂

@crftwrk
Copy link
Member Author

crftwrk commented Jun 1, 2023

Yes, I check it right now

@justinkruit justinkruit merged commit b661e3c into main Jun 1, 2023
@crftwrk
Copy link
Member Author

crftwrk commented Jun 1, 2023

It was just the Stripe field for the dark mode. But it's good as it is.

Do we want to start that with us beginning at 5.3.0?

Yes, I think so. Next minor update will be 5.3.1, next major 5.4.0 and so on. No matter what Bootstrap version says. If Bootstrap is v6, we can also go to 6.x.x. This gives us better control over the features and provides a clear number to the users. Agree?

@justinkruit
Copy link
Member

I agree.

I'm currently making some very small changes, and will maybe do another round of formatting just to be sure that everything is nice for 5.3.

@crftwrk
Copy link
Member Author

crftwrk commented Jun 1, 2023

Perfect

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
2 participants