WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 14 years ago

#19955 closed defect (bug) (fixed)

Update wordpress-logo.png and logo-login.png

Reported by: nacin Owned by: nacin
Priority: normal Milestone: 3.4
Component: UI Version:
Severity: normal Keywords:
Cc: Focuses:

Description

These are essentially the same graphic. We should consider combining them, unless there is a valid reason to keep the colors different.

wordpress-logo.png

http://core.trac.wordpress.org/export/19823/trunk/wp-admin/images/wordpress-logo.png

logo-login.png

http://core.trac.wordpress.org/export/19823/trunk/wp-admin/images/logo-login.png

Attachments (3)

logo-login.png​ (4.9 KB ) - added by iammattthomas 15 years ago.
Atoning for my past color sins.
19955.1.patch​ (1.1 KB ) - added by edward mindreantre 15 years ago.
Removed references to logo-login. Kept it in the $_old_files array, though.
readme.html.diff​ (531 bytes ) - added by soulseekah 15 years ago.
"Hey! I need some room to breathe here..."

Download all attachments as: .zip

Change History (14)

#1 @nacin
15 years ago

logo-login.php is much newer. It should be copied over wordpress-logo.png and all references to it removed, using wordpress-logo.png as the canonical image.

@iammattthomas
15 years ago

Atoning for my past color sins.

@edward mindreantre
15 years ago

Removed references to logo-login. Kept it in the $_old_files array, though.

#2 @edward mindreantre
15 years ago

  • Cc edward@… added
  1. Move the logo-login.png to wordpress-logo.png
  1. Apply the patch.

#3 @nacin
15 years ago

In [19937]:

Update login-logo.png to use proper color. Copy it over wordpress-logo.png. Make that one canonical, and stop using login-logo.png. Leave it for plugins. props iammattthomas. see #19955.

#4 @nacin
15 years ago

In [19938]:

Remove old logo-login.gif from manifest. props edward mindreantre, see #19955.

#5 @nacin
15 years ago

As logo-login.gif was ruthlessly deleted from #17324 and no one complained, I'm going to go ahead and remove logo-login.png here. I did a grep of the plugins repo, and the only plugin that would be affected would be ​this one, which moves replacement headers into wp-admin/images...

#6 @nacin
15 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In [19940]:

Remove unused logo-login.png after all. fixes #19955.

#7 @soulseekah
15 years ago

The new logo is 5px shorter and 14px wider! The readme.html file squished it up to 250x68. I propose to remove the inline attributes. Will check all instances of wordpress-logo.png shortly, may not fit nicely somewhere else as well.

@soulseekah
15 years ago

"Hey! I need some room to breathe here..."

#8 @soulseekah
15 years ago

  • Resolution fixed
  • Status closed → reopened

#9 follow-up: @soulseekah
15 years ago

Sorry, don't know whether to reopen or create a new ticket and list this ticket as related. Decided to go for less tickets.

#10 in reply to: ↑ 9 @nacin
15 years ago

Replying to soulseekah:

Sorry, don't know whether to reopen or create a new ticket and list this ticket as related. Decided to go for less tickets.

For simple things like this, when it is closed on a milestone that is not yet completed, this is fine.

#11 @nacin
14 years ago

  • Resolution → fixed
  • Status reopened → closed

In [20624]:

Remove img width/height from the readme.html WP logo. props soulseekah. fixes #19955. see #20293 for any follow-ups.

Note: See TracTickets for help on using tickets.