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

Add helpers to utilities bundle #32324

Merged
merged 12 commits into from
Dec 18, 2020

Conversation

mjansing
Copy link
Contributor

@mjansing mjansing commented Dec 3, 2020

Bootstrap bundles utility classes in a dedicated bootstrap-utilities deliverable (bootstrap-utilities.css) but the deliverable does not contain all utilities mentioned in the documentation (e.g. .sticky-top, .stretched-link or .text-truncate). These missing utilities are located in scss/helpers/ and currently not part of it. This PR adds them.

@mjansing mjansing requested a review from a team as a code owner December 3, 2020 20:19
@mjansing mjansing force-pushed the add-helpers-to-utilities-bundle branch from 152d31b to 1194c75 Compare December 4, 2020 07:20
Adds currently missing utilities classes which are located in scss/helpers to
boostrap-utilities deliverable.
@mjansing mjansing force-pushed the add-helpers-to-utilities-bundle branch from 1194c75 to 6bde754 Compare December 4, 2020 07:21
Copy link
Member

@mdo mdo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm into it. Feels like a natural fit.

@ffoodd
Copy link
Member

ffoodd commented Dec 9, 2020

Shouldn't it be mentionned in the docs?

@XhmikosR
Copy link
Member

XhmikosR commented Dec 9, 2020

@ffoodd like where? It seems utilities are already mentioned in https://getbootstrap.com/docs/5.0/getting-started/contents/#css-files. Feel free to push to this branch if you have any ideas.

@XhmikosR XhmikosR requested a review from ffoodd December 9, 2020 07:01
@ffoodd
Copy link
Member

ffoodd commented Dec 9, 2020

Don't really know, maybe we didn't mention helpers anywhere before?

It should at least appear in the migration guide since people importing only our utilities in a custom build will now import helpers incidentally, what they might have done explicitly before.

@XhmikosR
Copy link
Member

@mjansing can you please take care of the above comment?

@mjansing
Copy link
Contributor Author

@XhmikosR I added a migration note.

ffoodd
ffoodd previously requested changes Dec 16, 2020
site/content/docs/5.0/migration.md Outdated Show resolved Hide resolved
Copy link
Member

@mdo mdo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants