Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#10431 closed enhancement (fixed)

Fresh look for WP update notification on the top of page

Reported by: Simek Owned by:
Milestone: 3.0 Priority: normal
Severity: minor Version: 2.8
Component: UI Keywords: has-patch
Focuses: Cc:


Fresh look for WP update notification on the top of page.

Additional - shorter form for "moz-border-radius" in "wp-admin.css" if possible.

Preview: http://i28.tinypic.com/2cokfpw.png

Attachments (2)

better.WP.update.notifiaction.patch (2.6 KB) - added by Simek 6 years ago.
WP.Upgreade.Notification.for.2.9.patch (1.5 KB) - added by Simek 6 years ago.
new patch for 2.9

Download all attachments as: .zip

Change History (12)

#1 @Denis-de-Bernardy
6 years ago

  • Milestone changed from 2.8.2 to 2.9

#2 @scribu
6 years ago

Perhaps making the notification more noticeable (like a red border or something) would be good, given the recent security hubub.

#3 @janeforshort
6 years ago

  • Milestone changed from 2.9 to Future Release

Punting due to feature freeze.

#4 @Simek
6 years ago

  • Milestone changed from Future Release to 3.0

6 years ago

new patch for 2.9

#5 follow-up: @westi
6 years ago

  • Cc iammattthomas added
  • Keywords tested removed

This looks like a nice small improvement to me.

Removing the Tested flag as it is unclear what browser combinations this css change has been tested in.

Could you give more detail on what combinations you have tested it in.

I assume it will effectively fallback to the current UI if the browser doesn't support border-radius.

#6 @janeforshort
6 years ago

I'm going to do a UI review with Matt Thomas (who skinned 2.7 with the visual style) the 2nd week of January and ID all the things that need touch-ups for 3.0. Alerts have needed this since 2.7, as they didn't get a new style with everything else. Since the UI will be getting touched for the merge with MU, I'd like to hold off on UI tickets until after the review with MT so that everything will be consistent in 3.0.

#7 in reply to: ↑ 5 @Simek
6 years ago

Replying to westi:

Patch tested on IE 8, 7, 6; FF 2, 3, 3.5; Opera 9, 10; Safari 3, 4; Chrome 2, 3, 4 using Spoon Browsers Sandbox (http://spoon.net/browsers/).

Only in IE and Opera was a problem with non radius corners, as you expected. Everything else looks good.

#8 @matt
6 years ago

I think this is better than we have now, so worth bringing in. Thank you for updating the patch Simek!

Since we're in the early stages of 3.0 UI might be in flux, especially pending Jane and MT's meeting, but that's no reason not to make improvements in the meantime. Every little bit helps, and there are probably dozens of small improvements we could make around WP like this. Having a strong starting place will make our revisions better.

#9 @iammattthomas
6 years ago

I like the rounded corners since they match the admin style better. I'm guessing the centered alignment of the text in the screenshot is unintentional since it's not in the patch; we should definitely leave that left-aligned though.

#10 @westi
6 years ago

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

(In [12530]) Round the corners of the Update nag message. Fixes #10431 props Simek.

Note: See TracTickets for help on using tickets.