377110 - Strip out some un-needed rules from Camino-only forms.css

Status: RESOLVED FIXED

Description

[Stuart Morgan]

Created attachment 261216 [details] [diff] [review]
cleanup

I went through the rules in the Camino-only section of forms.css and pulled out a few that no longer appear to be necessary given the current state of trunk. This should make it a bit easier to see what's left to implement in the theme code so we can eventually rip the rest out without causing regressions.

Comment 1

[Josh Aas]

Comment on attachment 261216 [details] [diff] [review]
cleanup

Fine with me if it doesn't make camino unusable - the smaller the CSS diff between camino and Firefox the better.

Comment 2

[philippe (part-time)]

While you are at it, you could also remove the background-color rule from
input,
input[disabled] {
  padding: 0px 1px 0px 1px;
  font-size: 11px;
  margin: 1px 1px 1px 1px;
  background-color: -moz-field;
}

That one line causes disabled textfields to paint as gfx, for which I just filed bug 377124.

Comment 3

[Stuart Morgan]

Created attachment 261258 [details] [diff] [review]
v2

Yep, thanks. I thought I tried that and ended up with transparent fields, but apparently not since it works for me now.

Comment 4

[Stuart Morgan]

Landed.