Skip to content

Remove attribute selectors from buttons#28988

Merged
XhmikosR merged 3 commits intomasterfrom
master-mc-remove-attribute-selectors
Jul 12, 2019
Merged

Remove attribute selectors from buttons#28988
XhmikosR merged 3 commits intomasterfrom
master-mc-remove-attribute-selectors

Conversation

@MartijnCuppens
Copy link
Member

We needed this, back in the days when this was still present in our code (see #4666):

bootstrap/less/forms.less

Lines 102 to 108 in fd49d4a

// Help out input buttons
input[type="button"],
input[type="reset"],
input[type="submit"] {
width: auto;
height: auto;
}

But nowadays it's redundant.

@XhmikosR XhmikosR requested a review from mdo July 4, 2019 21:40
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 feel like this had something to do with Normalize back in the day, but I don’t see anything relevant there still today, so likely safe to remove.

@XhmikosR XhmikosR merged commit d479fd8 into master Jul 12, 2019
@XhmikosR XhmikosR deleted the master-mc-remove-attribute-selectors branch July 12, 2019 10:15
patrickhlauke pushed a commit that referenced this pull request Jul 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants