li { margin: 0; }

Question

li { margin: 0; }

jbcaprell opened this issue 5 years ago · comments

Caveat: I only mention this because of the code golf-y nature of your reset.

I’m not actually aware of any user agent, Blink, Gecko/Quantum, WebKit, or EdgeHTML (whose stylesheet is, technically, closed-source, whatever sense that makes) that applies a margin to li elements, and so this line may be unnecessary:

https://github.com/hankchizljaw/modern-css-reset/blob/12e77f9a895255554e339b3de2c09b12f085dea3/src/reset.css#L23

Jonathan Caprell · Answer 1 · Thu Dec 05 2019 07:12:42 GMT+0800 (China Standard Time)

(This is also true of figcaption, I think)

https://github.com/hankchizljaw/modern-css-reset/blob/12e77f9a895255554e339b3de2c09b12f085dea3/src/reset.css#L25

Andy Bell · Answer 2 · Mon Dec 09 2019 05:05:27 GMT+0800 (China Standard Time)

PR 100% welcome for this :)