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

v5: Drop .media #28265

Merged
merged 7 commits into from
May 8, 2019
Merged

v5: Drop .media #28265

merged 7 commits into from
May 8, 2019

Conversation

mdo
Copy link
Member

@mdo mdo commented Feb 14, 2019

The media component is basically a series of utilities that have been assigned to a more unique name. This can all be done via our other utility classes.

@mdo mdo requested a review from a team as a code owner February 14, 2019 17:54
@XhmikosR XhmikosR changed the title v5: Drop .media [WIP] v5: Drop .media Feb 14, 2019
@MartijnCuppens
Copy link
Member

Deprecate the component in v4.3.2 /cc @MartijnCuppens

We can't use the deprecate() mixin here since we can only use it in mixins of functions, not in for selectors. I guess we'll just need to document this in the migration doc.

@MartijnCuppens
Copy link
Member

About the deprecation notice: we could just add a message to the v4 docs this will be removed.

@mdo
Copy link
Member Author

mdo commented May 8, 2019

Should be ready for review now, @twbs/css-review.

@mdo
Copy link
Member Author

mdo commented May 8, 2019

@ysds Done!

@XhmikosR XhmikosR changed the title [WIP] v5: Drop .media v5: Drop .media May 8, 2019
@XhmikosR
Copy link
Member

XhmikosR commented May 8, 2019

@MartijnCuppens: LGTY?

@XhmikosR XhmikosR merged commit d806e87 into master May 8, 2019
@XhmikosR XhmikosR deleted the v5-drop-media branch May 8, 2019 18:45
crhistianramirez pushed a commit to crhistianramirez/headstart that referenced this pull request Dec 28, 2022
Dropped the .media component as it can be easily replicated with utilities. See twbs/bootstrap#28265 and the flex utilities page for an example https://getbootstrap.com/docs/5.0/utilities/flex/#media-object
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.

7 participants