Remove reset.css from Bartik [#702402]

Comment	File	Size	Author
#7	combine-styles-reset.patch	1.08 KB	emmajane

Comment #1

jensimmons commented 8 February 2010 at 03:02

Hmmm, this is interesting. I don't usually use reset stylesheets, but did for Bartik, in part because reset.css-es are being used in other places in Drupal core. I started out with the full-length Eric Meyer reset, but over time stripped most of it out. Maybe now I should just move what's left into styles.css and have there be one less css file.

I also don't think this is a very big deal — it does work fine as is.

What do other people think?

Log in or register to post comments

Comment #2

Anonymous (not verified) commented 9 February 2010 at 06:26

There seems to be an 'argument' for using a reset css as best practise from what I have read online.

eg http://sixrevisions.com/css/css-tips/css-tip-1-resetting-your-styles-wit...

I would however have this as the beginning of the main style sheet instead of separately.

Log in or register to post comments

Comment #3

dodorama commented 9 February 2010 at 14:14

I'd say that the main argument of the article you linked (avoid browser inconsistencies) is not always valid. As pointed by jensimmons in this comment while talking about rounded corner support on IE, websites don't need to look the same in every browser. Moreover I'm pretty sure that the most annoying differences (like the margins on paragraphs) are styled, anyway, in the main stylesheet resulting in duplicated code. I believe that some (soft) reset styles (if needed) can be moved inside the main stylesheet as jensimmons suggested.