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

update from official repo #3

Merged
merged 47 commits into from
Aug 25, 2017
Merged

update from official repo #3

merged 47 commits into from
Aug 25, 2017

Conversation

gijsbotje
Copy link
Owner

No description provided.

XhmikosR and others added 30 commits August 17, 2017 12:11
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)
Fixes #23360 by restoring just the offset class generation to our grid
framework mixins. Also restores the `make-col-offset` mixin.

Docs have been restored to illustrate this behavior and merged with the
newer margin utilities examples.
…he top

I never use this dang list and it just falls out of date far too easily. Dropping for my own sanity.
1. Instead of having two selectors, we use the `.list-group-item` class and have coverage for list items and anchors.
2. Rather than override all border values, we only override what we set with `border-top`.
pat270 and others added 17 commits August 21, 2017 18:20
…-bg`, `$table-inverse-hover-bg`, `$table-inverse-border-color`, `$table-inverse-color` to `$table-dark-bg`, `$table-dark-accent-bg`, `$table-dark-hover-bg`, `$table-dark-border-color`, `$table-dark-color` for consistency
…e-dark`, `.thead-dark`, `.thead-light` respectively
make outer `.modal-dialog` pass through events for custom click
handling, counteract it for the actual `.modal-content`
* Adjust margin for sticky elements on modal

Previously white space was visible to the right of sticky elements due to right padding being added to the body. This fixes #23661.

* Add unit tests for margin of sticky elements on modal
@gijsbotje gijsbotje merged commit 4356d08 into gijsbotje:v4-dev Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.