New admin bar WP logo is strangely large, + icon missing hover state

[johnjamesjacoby]

See #19404 and r19516.

The new, bigger WP logo makes the other icons look too small. If we're going big on the WP icon, all the icons should go big too (similar to the main OS X menu bar, Apple logo VS the tray icons.)

The "Add" menu icon is missing a hover state.

[johnjamesjacoby]

Updated sprite with hover state for add menu

[ocean90]

The "Add" menu icon is missing a hover state.

At the moment only the WP icon has a hover state, search, update and add not.

[johnjamesjacoby]

Replying to ocean90:

The "Add" menu icon is missing a hover state.

At the moment only the WP icon has a hover state, search, update and add not.

That's the problem this patch fixes. On hover, the Add menu drops down. When down, the background color changes from dark gray to white. When white, the light gray plus graphic doesn't currently have a dark equivalent. This is obvious when looking at the current iteration of the admin bar.

[jane]

When you have 2 separate issues like this, please put them on separate tickets for easier tracking.

Re the big icon: This was discussed in IRC, and was a Matt request. Think of the Apple icon compared to the file menu labels. Being bigger than the others was intentional. Wontfix.

Re hover state for Add New, I agree it would make the most sense for the + sign to change color along with the font for New. Should fix.

Grr. See how putting two issues on one ticket messes things up? :)

[koopersmith]

The patch doesn't apply cleanly, but when applied manually, it works.

[nacin]

The W logo could probably use a fresh coat of paint by chexee or emireoflight. It looks like the aliasing is off, a bit pixelated, rough around the edges.

Looking into the patch for the + icon.

[nacin]

Patch refreshed. Any reason why we need to re-specify background-image and -position? Is it there for reset purposes?

[johnjamesjacoby]

Replying to nacin:

Patch refreshed. Any reason why we need to re-specify background-image and -position? Is it there for reset purposes?

Was in my patch as it's how the logo worked; your revised patch fixes that. Can't imagine it's needed?

[johnjamesjacoby]

Replying to jane:

Grr. See how putting two issues on one ticket messes things up? :)

If the WP logo changeset had its own ticket to reference, this wouldn't be so messed. :)

[jane]

Replying to johnjamesjacoby:

If the WP logo changeset had its own ticket to reference, this wouldn't be so messed. :)

That's not true. You could have made two new tickets instead of one. I know you are just yanking my chain a little because we're friends, but new contributors reading this won't realize that and may actually believe that's a valid reason for putting two issues on one ticket.

[nacin]

Replying to jane:

Replying to johnjamesjacoby:

If the WP logo changeset had its own ticket to reference, this wouldn't be so messed. :)

That's not true. You could have made two new tickets instead of one. I know you are just yanking my chain a little because we're friends, but new contributors reading this won't realize that and may actually believe that's a valid reason for putting two issues on one ticket.

True. In fairness, jjj has a separate and valid point. All commits should have a corresponding ticket. He should know, considering I ribbed him on being guilty of this over on bbPress. :-)

[koopersmith]

Tested attachment:19475.2.diff​. It works just as well, and the CSS is cleaner. I like.

[azaozz]

Yes, attachment:19475.2.diff​ looking good here too, tested all the way to IE7 rtl.

[nacin]

In [19578]:

Hover state for the Add New admin bar icon. props johnjamesjacoby. fixes #19475.