Opened 14 years ago
Closed 14 years ago
#17887 closed defect (bug) (fixed)
New stars for gray theme shouldn't have white borders
Reported by: | helenyhou | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 3.2 | Priority: | normal |
Severity: | normal | Version: | 3.2 |
Component: | Graphic Design | Keywords: | has-patch |
Focuses: | Cc: |
Description
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)
Change History (12)
#2
@
14 years ago
The colour specifications should be moved out of plugin-install.dev.css into their colour specific sheets IMO.
#3
@
14 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.
#4
@
14 years ago
- Keywords has-patch added; needs-patch removed
- Owner set to ocean90
- Status changed from new to accepted
#5
in reply to:
↑ 1
@
14 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).
#6
@
14 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)
Patch is okay, next time you doesn't need to patch the compressed files, dev is enough.