About this code:
|
a#{$parent}, |
|
button#{$parent} { |
|
@include hover-focus { |
|
background-color: darken($color, 10%) !important; |
|
} |
|
} |
I think we should remove it. IMO focus and hover background states don't belong in these utilities and we are also not defining an active state here. We have the buttons with backgrounds, if this doesn't cover the needs I think the developer will need to write custom code, because it's quite specific.
I guess we can just remove the mixin, just use the background-color property in v5 and use the deprecate mixin to deprecate this in v4.
Thoughts?
About this code:
bootstrap/scss/mixins/_background-variant.scss
Lines 9 to 14 in 8896827
I think we should remove it. IMO focus and hover background states don't belong in these utilities and we are also not defining an active state here. We have the buttons with backgrounds, if this doesn't cover the needs I think the developer will need to write custom code, because it's quite specific.
I guess we can just remove the mixin, just use the
background-colorproperty inv5and use thedeprecatemixin to deprecate this inv4.Thoughts?