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

Change breadcrumb item padding variable name #28125

Merged
merged 2 commits into from
Feb 11, 2019

Conversation

astrahov
Copy link
Contributor

Breadcrumb item use only padding-left and padding-right.

$breadcrumb-item-padding -> $breadcrumb-item-padding-x

@astrahov astrahov requested a review from a team as a code owner January 28, 2019 09:14
@MartijnCuppens
Copy link
Member

@astrahov, this might indeed be a bit confusing. However, we can't just change the name because this will be a breaking change.

I'm going to put this on hold for now, we'll come back to it when we're starting to work on v5.

@XhmikosR XhmikosR changed the base branch from v4-dev to v4-without-jquery February 11, 2019 10:28
@XhmikosR XhmikosR removed the on-hold label Feb 11, 2019
@XhmikosR XhmikosR merged commit 3dd1d25 into twbs:v4-without-jquery Feb 11, 2019
@astrahov astrahov deleted the patch-3 branch October 25, 2019 05:11
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.

3 participants