Save/Publish spinners should be separate

[jane]

When you click on Save Draft, the spinner is still located next to the Publish button, and there's no visual connection between the spinner and the action it is describing. Save Draft (on screens to add new post/page, in QuickPress, etc.) should have its own spinner to improve clarity.

Raised issue originally in related ticket, which fixed padding but not location: ​http://core.trac.wordpress.org/ticket/10680

User feedback reminding of the issue: ​http://jackosborne.co.uk/articles/a-wordpress-ux-problem/

[nacin]

+100. This trips me up all the time.

[wojtek.szkutnik]

introducing an additional spinner

[JohnONolan]

Big +1 from me, looks great. Tested in Mozilla, Webkit, and IE. Screencast of patch in action here: ​http://screenr.com/32n[[BR]]

Suggest commit pending signoff from Jane.

[JohnONolan]

Damn BR tag, video here: ​http://screenr.com/32n

[JohnONolan]

On 2nd review I'd like to question why we need a new ID and inline styles add in this patch?

[nacin]

Let's move these to a class. The visibility bit should be the default in CSS as well, with JS then toggling it.

This keeps bugging me every time I save a post draft, so I'm likely going to just run with this.

[JohnONolan]

Do you mean move everything to a single class, eliminating both the multiple IDs and the inline CSS ? I'd support that.

One of the big problems with WP's CSS is that IDs are getting added in patches like this on a regular basis, bloat builds up over time with hundreds of single use IDs.

Hopefully with the CSS refactor we can make more smart/dynamic classes for use in multiple instances

[nacin]

(In [15722]) Use separate ajax spinners for the Save Draft and Publish buttons on the post form. fixes #14582 with initial patch from wojtek.szkutnik.