Remove some base element resets [#439428]

There are a few base element resets in this theme that don't make a whole lot of sense to have in there:

b, strong {
  font-weight: bold;
}
i, em, u {
  font-style: italic;
}
u {
  text-decoration: none;
}

u with a text-decoration of none, in particular, is just weird and has caused at least some folks to pull out their hair trying to figure out why things were displaying oddly. I think we should remove these and let them stay at defaults.

Comment	File	Size	Author
#3	foundation_element_reset_2.patch	5.51 KB	Tresler
#2	foundation_element-reset_439428-1.patch	812 bytes	add1sun

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

add1sun CreditAttribution: add1sun commented 20 April 2009 at 18:15

Also, setting ALL headings to the same size, and 100% at that?

h1, h2, h3, h4, h5, h6 {
  margin-top: 10px;
  margin-bottom: 0px;
  font-size: 100%;
}

Comment #2

add1sun CreditAttribution: add1sun commented 20 April 2009 at 18:16

Status:

Active

» Needs review

File	Size
foundation_element-reset_439428-1.patch	812 bytes

Attaching a patch that cleans these out.

Comment #3

Tresler CreditAttribution: Tresler commented 24 July 2009 at 18:09

File	Size
foundation_element_reset_2.patch	5.51 KB

Patch applies clean and works as advertised.

I did notice that a lot of the whitespace was odd. It looked like the grouping of common selectors was not being separated by a space between rules, which, for me made it difficult to read.

I went through style.css and cleaned it up vis-a-vis this: http://drupal.org/node/302199

Only then did I realize I'd already applied add1sun's patch, so here are the two patches combined. I would normally make this two separate patches. but here is hoping a combined patch is better than nothing.