search

xz / new.css-site

https://newcss.net
58 stars 7 forks source link

Combine abbr styles #6

Open erikwallace opened 4 years ago

erikwallace commented 4 years ago

Great project, I have two suggestions. There are two cursor: help declarations, it might make sense to combine them:

abbr, abbr:hover { /* Set the '?' cursor while hovering an abbreviation */ cursor: help; }

Also, where you reset margins and padding with *, would you consider adding box-sizing?

* { /* Reset margins and padding */ box-sizing: border-box; margin: 0; padding: 0; }

erikwallace commented 4 years ago

I don't contribute much to GitHub, my suggesions/questions aren't necessarily issues. Let me know if there's a better way to comment.