Add comment to clarify vertical-align warning (#4434)

* Add comment to clarify vertical-align warning

* fix typo

* Update preflight.css

Co-authored-by: Adam Wathan <adam.wathan@gmail.com>
This commit is contained in:
Jason Safaiyeh 2021-05-24 10:46:50 -05:00 committed by GitHub
parent 3876cffbbd
commit ea3892fc91
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -207,11 +207,20 @@ samp {
}
/**
* Make replaced elements `display: block` by default as that's
* the behavior you want almost all of the time. Inspired by
* CSS Remedy, with `svg` added as well.
* 1. Make replaced elements `display: block` by default as that's
* the behavior you want almost all of the time. Inspired by
* CSS Remedy, with `svg` added as well.
*
* https://github.com/mozdevs/cssremedy/issues/14
* https://github.com/mozdevs/cssremedy/issues/14
*
* 2. Add `vertical-align: middle` to align replaced elements more
* sensibly by default when overriding `display` by adding a
* utility like `inline`.
*
* This can trigger a poorly considered linting error in some
* tools but is included by design.
*
* https://github.com/jensimmons/cssremedy/issues/14#issuecomment-634934210
*/
img,
@ -222,8 +231,8 @@ audio,
iframe,
embed,
object {
display: block;
vertical-align: middle;
display: block; /* 1 */
vertical-align: middle; /* 2 */
}
/**