Make WordPress Core

Opened 19 years ago

Closed 19 years ago

#1139 closed defect (bug) (fixed)

IE bug makes all class="alternate" rows show up green (like class="active")

Reported by: mc_incubus's profile MC_incubus Owned by: ryan's profile ryan
Milestone: Priority: normal
Severity: minor Version: 1.5.1
Component: Administration Keywords:
Focuses: Cc:


Plugins that are active are class="active", but every other row is also class="alternate" which is combined into class="alternate active" if the plugin is also active.

The CSS rules for those multiclasses are these:

.active.alternate td {background: #ADA;}
.active.alternate .name {background: #8B8;}

Due to a bug in IE (oh, let's not kid ourselves, they didn't even TRY to properly support CSS2), these rules are incorrectly interpreted so that all class="alternate" rows get the green color.

Ironically, I discovered this IE bug on a page by Eric Meyers, the one who proposed these CSS changes. :-)

Attachments (1)

wp-admin.css.diff (421 bytes) - added by MC_incubus 19 years ago.

Download all attachments as: .zip

Change History (6)

#1 @MC_incubus
19 years ago

  • Patch set to No

#2 @anonymousbugger
19 years ago

Oops. Just invert the order, so the rules read: td {background: #ADA;} .name {background: # 8B8;}

...but remove the space I threw in there to keep the system from turning the octothorpe-and-8B8 combination into a link to bug number 8 with a B* right after it.

#3 @MC_incubus
19 years ago

Yep, that fixes it. Of course, if we use this method for designating active/alternate rows, IE will never be able to alternate green color... but that's not a big problem.

Uploading the patch now. Thank you, anonymousbugger who I strongly suspect is Eric. :-)

#4 @MC_incubus
19 years ago

  • Patch changed from No to Yes
  • Resolution changed from 10 to 20

#5 @ryan
19 years ago

  • fixed_in_version set to 1.5.1
  • Owner changed from anonymous to rboren
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.