Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#17887 closed defect (bug) (fixed)

New stars for gray theme shouldn't have white borders

Reported by: helenyhou's profile helenyhou Owned by: ocean90's profile ocean90
Milestone: 3.2 Priority: normal
Severity: normal Version: 3.2
Component: Graphic Design Keywords: has-patch
Focuses: Cc:


The new stars in the plugin install screen are fantastic, but look funky when using the gray theme because they still have white borders on each side. Patch adds border color #f9f9f9 to colors-fresh.css, does not remove #fff from plugin-install.css - order seems fine.

Related: #17428

Attachments (3)

colors-fresh.diff (77.7 KB) - added by helenyhou 13 years ago.
17887.patch (1.5 KB) - added by ocean90 13 years ago.
17887.diff (1.3 KB) - added by dd32 13 years ago.

Download all attachments as: .zip

Change History (12)

#1 follow-up: @ocean90
13 years ago

  • Milestone changed from Awaiting Review to 3.2

Patch is okay, next time you doesn't need to patch the compressed files, dev is enough.

#2 @dd32
13 years ago

The colour specifications should be moved out of into their colour specific sheets IMO.

#3 @dd32
13 years ago

  • Keywords needs-patch added; has-patch removed

actually. To complicate it, The Stars on the Plugin Information dialogue need the #fff background, whilst the list table needs the #f9f9f9 background, all whilst using the grey scheme. needs-patch based on that.

13 years ago

#4 @ocean90
13 years ago

  • Keywords has-patch added; needs-patch removed
  • Owner set to ocean90
  • Status changed from new to accepted

#5 in reply to: ↑ 1 @helenyhou
13 years ago

Replying to ocean90:

Patch is okay, next time you doesn't need to patch the compressed files, dev is enough.

Good to know; gotta learn somewhere, I guess!

New patch works for me (of course).

13 years ago

#6 @dd32
13 years ago

attachment 17887.diff added

Just a simplified version of ocean90's patch.

Tested it in the 3 locations we use the stars in both Grey and Blue schemes. (Plugin Info dialogue, Plugin Listing page, Theme Install details pane), retains generic white formatting to fit in any other locations plugins may be using it (but if they are, they'll need to update the filenames anyway)

#7 @ocean90
13 years ago

Just a simplified version of ocean90's patch.

Sure, no idea why I haven't done this.

#8 @azaozz
13 years ago

+1 for 17887.diff.

#9 @dd32
13 years ago

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

In [18362]:

Correct the background colour for the stars. Props ocean90 & helenyhou. Fixes #17887

Note: See TracTickets for help on using tickets.