WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 4 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:

Description

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 5 years ago.
WP.Upgreade.Notification.for.2.9.patch (1.5 KB) - added by Simek 4 years ago.
new patch for 2.9

Download all attachments as: .zip

Change History (12)

comment:1 Denis-de-Bernardy5 years ago

  • Milestone changed from 2.8.2 to 2.9

comment:2 scribu5 years ago

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

comment:3 janeforshort5 years ago

  • Milestone changed from 2.9 to Future Release

Punting due to feature freeze.

comment:4 Simek4 years ago

  • Milestone changed from Future Release to 3.0

Simek4 years ago

new patch for 2.9

comment:5 follow-up: westi4 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.

comment:6 janeforshort4 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.

comment:7 in reply to: ↑ 5 Simek4 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.

comment:8 matt4 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.

comment:9 iammattthomas4 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.

comment:10 westi4 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.