Revise update notification bubble colour

[johnonolan]

Move from the WordPress 2.5 orange (d54e21) to a new one put forward by the UI group (ee7600) - ​http://twitpic.com/1meikk/full .

This was originally just for the background of the update notification bubble however due to the way it was coded it was a lot easier to blanket implement it so that link hover states are bundled in!

[johnonolan]

Screenshot of patch: ​http://drp.ly/12Fa6M

[nacin]

We use this orange in blue/classic as well, as well as a few other spots:

./wp-admin/css/colors-classic.dev.css:232:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:268:	background-color: #d54e21;
./wp-admin/css/colors-classic.dev.css:361:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:377:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:434:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:501:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:509:	background-color: #d54e21;
./wp-admin/css/colors-classic.dev.css:579:	color: #d54e21 !important;
./wp-admin/css/colors-classic.dev.css:1289:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:1395:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:1416:	color: #d54e21;
./wp-admin/css/colors-classic.dev.css:1669:	color: #d54e21;
./wp-admin/css/dashboard.dev.css:148:	color: #d54e21;
./wp-admin/css/install.dev.css:25:	color: #d54e21;
./wp-includes/js/tinymce/wp-mce-help.php:47:		color: #d54e21;
./wp-includes/js/tinymce/wp-mce-help.php:58:		color: #d54e21;

Are we ready to move to this orange for links as well? Just confirming. It's arguably a little light.

[Denis-de-Bernardy]

Replying to nacin:

It's arguably a little light.

+1. My eyes are hurting.

[ocean90]

-10 for #ee7600.

[johnonolan]

I'm on the fence with this one, I can see both sides. Let's see what Jane says and go from there!

[jane]

I like it better than the burnt orange for the # bubble, but don't like it for the text b/c it's a bit too light. However, since we're still potentially going to adjust the on-state background color, maybe this one should wait for that, as the orange may or may not be relevant at that point.

[nacin]

Move out of 3.0 then?

[jane]

We're shooting to get it done before RC. No ticket to link to yet, though. John's working on it with a new contributor.

[johnonolan]

As a followup to Jane's comment this ticket is dependent on #13486 being completed first

[ocean90]

A blue version: ​http://grab.by/4w4u, see ​#13486#comment:12

[jane]

I think we're going with gray instead of blue after all, and moving to a dark gray bubble instead of orange goes along with that.

[johnonolan]

+1, I'll work on that next

[johnonolan]

New patch for grey bubbles

[johnonolan]

Screenshot of all states ​http://drp.ly/14lHeA

[jane]

The blue bubble on hover seems a little weird... I think it might look better either do orange, so all hover is the same in the menu and it matches the adjacent text, or to leave it gray, so it doesn't look like the bubble is an independent link.

[johnonolan]

Revised patch without hover state

[Denis-de-Bernardy]

what's wrong with the current color?

[nacin]

Replying to Denis-de-Bernardy:

what's wrong with the current color?

It's from the 2.5 design, doesn't fit with the rest of the admin because of that, and needs a new coat of paint.

[johnonolan]

Jane - what about the hover-state of the comment-count bubble in the "in response to" column of the comment moderation screen? Currently goes blue > orange - do you want to go blue > grey?

[jane]

That one can stay orange since it is actually a link, and if it was going to change, it would be on a different ticket, since it's not update related. One's a notification and one's a graphical link, so the bubble on comments should act like the links.

[nacin]

(In [14918]) New admin menu notification bubble color. props JohnONolan, fixes #13474.

[Viper007Bond]

For what it's worth, I'm not a fan. It sticks out like a sore thumb.

[nacin]

Replying to Viper007Bond:

For what it's worth, I'm not a fan. It sticks out like a sore thumb.

Pretty sure that's the idea ;)

[nacin]

This changed the color of the "Install Update Now" button (or other related text) for the plugin thickbox. Probably need to switch classes.

[nacin]

[14980]

Use orange for the plugin install thickbox button, not the new gray for update notification bubbles. fixes #13474.