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

Remove redundant radius properties #28956

Merged
merged 2 commits into from Jun 26, 2019

Conversation

MartijnCuppens
Copy link
Member

@MartijnCuppens MartijnCuppens commented Jun 24, 2019

Some properties are already set on .list-group-item, they can be removed safely.

Demo: https://deploy-preview-28956--twbs-bootstrap.netlify.com/docs/4.3/components/list-group/#horizontal

@MartijnCuppens MartijnCuppens requested a review from a team as a code owner June 24, 2019 18:38
@MartijnCuppens MartijnCuppens added this to Inbox in v4.4.0 via automation Jun 24, 2019
@MartijnCuppens MartijnCuppens added this to Inbox in v5 via automation Jun 24, 2019
@XhmikosR XhmikosR force-pushed the master-mc-remove-redundant-radius branch from 3a72139 to 29d3dd7 Compare June 25, 2019 06:49
v5 automation moved this from Inbox to Approved Jun 26, 2019
@MartijnCuppens MartijnCuppens merged commit 4b3d263 into master Jun 26, 2019
v5 automation moved this from Approved to Shipped Jun 26, 2019
@MartijnCuppens MartijnCuppens deleted the master-mc-remove-redundant-radius branch June 26, 2019 13:11
XhmikosR pushed a commit that referenced this pull request Jun 27, 2019
XhmikosR pushed a commit that referenced this pull request Jul 10, 2019
@XhmikosR XhmikosR moved this from Inbox to Shipped in v4.4.0 Jul 10, 2019
@XhmikosR XhmikosR added the v4 label Jul 16, 2019
@mdo mdo mentioned this pull request Jul 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v4.4.0
  
Shipped
v5
  
Shipped
Development

Successfully merging this pull request may close these issues.

None yet

3 participants