"Post update" button disabled status doesn't show

[r-a-y]

In global.js, need to add the "disabled" class to the "Post update" submit buttons and also handle removing the class when it's re-enabled.

Also, the current input.disabled style lacks visual presence. Needs to be darker.

No time for patch at the moment.

Related: #3165.

[DJPaul]

We need to update the JS, where we set things like button.prop('disabled', true), to also set a disabled class, as <= IE8 doesn't support input:disabled

[DJPaul]

Since we are using disabled to throttle people submitting updates, the styling should only be applied once the (first) message gets sent sucesfully. I've made this change in the javascript, but the button's background picks up a white? styling from somewhere while it's working.

And we need to come up with new button for disabled, in general.

[boonebgorges]

You forgot to remove the 'disabled' class after the 8 second timeout. See 3360-2.patch.

What does "a new button for disabled" mean? Different text? Is this something we need for 1.3?

[DJPaul]

When you click on a button, and the AJAX requests happen, the spinner appears (all good). At this point, the visual appearance of the button is that it looks like a white rectangle with a border. In the case of this "8 second button", after the AJAX completes, and the button is not clickable, it looks extremely similar to a button that you *can* click.

[boonebgorges]

Right. I tried adding a 'title' attribute so that at least you would have a hover tooltip, but it appears that 'disabled' makes tooltips not work as well. What if we change the button text to "Please wait"? Or to "Please wait (x)" where x is a 87654321 countdown timer?

[DJPaul]

I've never really liked the hidden 8 second delay. Let's make it work like Twitter's site: no button visible until you focus in the textarea, then it appears. Then after it submits, the button hides again and focus is removed.

[boonebgorges]

Fine by me. You want to patch it, Señor JavaScript?

[DJPaul]

That's done, but the pending / disabled button styles still need updating for the recent changes to the theme's colours.

[DJPaul]

(See r4814)

[boonebgorges]

I really like r4814 btw.

[DJPaul]

Current issue is the a.loading / input.loading CSS. When that's applied to one of the buttons, its background-image selector overrides the button's "background" background.

See ​http://i51.tinypic.com/n3stis.png

Left is the live state, middle is loading, right is disabled. I think the disabled button appearance needs some work. I'd almost suggest we come up with a new loading style, and use the current loading style as a new disabled style?

[boonebgorges]

(In [4886]) Applies new activity stream animations introduced in r4884 to update box when arriving via Send Public Message. Fixes #3360 again