]> git.ipfire.org Git - thirdparty/bootstrap.git/commit
Remove `color` property from `badge` class 23529/head
authorJon Wolski <jon.wolski@groups360.com>
Thu, 17 Aug 2017 19:25:38 +0000 (14:25 -0500)
committerJon Wolski <jon.wolski@groups360.com>
Fri, 18 Aug 2017 14:35:00 +0000 (09:35 -0500)
commitf07d870381885f1901f7de78457761a5c13d64b3
tree326c4720988508bab4904c4977e69cf240047c7e
parent74d96392d2370c5e0f54380ddb5c7a09d6c13ac7
Remove `color` property from `badge` class

I suggest removing `color: #FFF` from the `badge` class, because it is
provided by the `badge-<modifier>` classes along with a
background-color, whereas there is no corresponding background-color
provided on the `badge` class itself.

Background
----------

According to the documentation:

> [badge] Requires one of the contextual, color modifier classes for `color` and
> `background-color`.

That is, color and background color should be provided by a modifier
class, rather than the badge class itself.

e.g. `<span class='badge badge-primary'>4</span>`

In general, providing a `color` property without also providing a
`background-color` should be avoided. [W3 QA tips]

In this context, this is a problem only if someone creates a `badge`
without providing a modifier. Such an omition may be difficult to trouble-shoot,
because this will most likely yield white text on a white background.

[W3 QA tips](https://www.w3.org/QA/Tips/color)
scss/_badge.scss