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

Migrate to Bootstrap 5.0.0.alpha3 #8

Merged
merged 1 commit into from
Nov 16, 2020

Conversation

stevenbuccini
Copy link

  • Implement new .form-label class. Docs say:

Form labels now require the .form-label class. Sass variables are now available to style form labels to your needs. See #30476

  • Drop .form-group. This method has been removed in Bootstrap 5. The maintainers replaced it in their docs with .mb-3 and I have done the same.

@stevenbuccini stevenbuccini changed the base branch from master to bootstrap4 November 16, 2020 01:56
@afomera afomera changed the base branch from bootstrap4 to bootstrap5 November 16, 2020 18:59
@afomera
Copy link
Owner

afomera commented Nov 16, 2020

hey @stevenbuccini thanks for this PR. I've rebased this onto a Bootstrap 5 branch for now until it's released.

(I want to keep BS4's branch as BS4)

When BS5 is out officially, I would be okay making the main/master branch BS5. Does that sound good to you?

@afomera afomera merged commit 099801f into afomera:bootstrap5 Nov 16, 2020
@stevenbuccini
Copy link
Author

I think that's a great plan! I simply used that as the base branch to reduce the noise in the PR. Thanks for the quick turnaround.

@stevenbuccini stevenbuccini deleted the bootstrap5.0.0.alpha3 branch November 16, 2020 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants