Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#8306 closed defect (bug) (fixed)

Left over color issue?

Reported by: ydekproductions's profile ydekproductions Owned by:
Milestone: 2.7 Priority: normal
Severity: normal Version: 2.7
Component: UI Keywords:
Focuses: Cc:

Description

I think there has been a self over style from a previous revision, but I'm not sure.

@@ line 238

.form-table td,
.form-table th,
#wpbody-content .describe td,
#wpbody-content .describe th {
	border-bottom-color: #f9f9f9;
}

That is in colors-fresh.css, and the f9f9f9 color is slightly off from the background of the table which is now white. It has been bugging me, so I started to look into it and found that to be the culprit. Either just change it to #fffff, or remove the border all together.

I'm attaching a screen shot to see what I'm talking about. Look between the rows, it's slightly gray and set off from the rest of the table. I'm doubting it's intentional.

Attachments (3)

Picture 6.png (3.8 KB) - added by ydekproductions 16 years ago.
fix-color.diff (373 bytes) - added by ydekproductions 16 years ago.
fix-border-colors.diff (752 bytes) - added by ydekproductions 16 years ago.

Download all attachments as: .zip

Change History (9)

#1 @ydekproductions
16 years ago

Actually, now I see that the style is used differently in different places.

For example, the screen shot I took is from the Add Link page where the table is set on a white background, but on the General Settings page, it's set on the #f9f9f9 page background color.

I updated my patch to add a selector for just the ones inside a .postbox.

#2 @kpdesign
16 years ago

  • Version set to 2.7

I see the same issue as in your screenshot (FF3). The recommended change would need to be made to both stylesheets, as I can reproduce this with both the Classic and Fresh themes.

#3 @ydekproductions
16 years ago

There, that fixes both themes. :)

#4 @kpdesign
16 years ago

I am still seeing this with both the Classic and Fresh themes. Perhaps it's supposed to be that way?

#5 @ydekproductions
16 years ago

Nobody has commit the patch yet. :(

#6 @azaozz
16 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [9942]) Remove left-over background color, fixes #8306

Note: See TracTickets for help on using tickets.