Provide an API for aria-live region update announcements that module developers may leverage

Patch applies nicely locally.

I didn't have enough experience getting to the console easily, but with Jesse's help I was able to verify that it worked in Safari with VoiceOver & Chrome with http://www.chromevox.com

With Safari, it's not as obvious where the console is, it's now:
"the speech bubble tab at the top of the sidebar on the left side of the screen and then click on the Current Log entry in the sidebar."

This is definitely something that needs to be available for everyone, but I quite like the centralization of this function.

I like the parameter priority too:

   * @param {String} priority
   *   - A string to indicate the priority of the message. Can be either
   *   'polite' or 'assertive'. Polite is the default.
   * @see http://www.w3.org/WAI/PF/aria-practices/#liveprops

Sometimes ARIA docs are horrible to read, but that looks like a reasonably clear one. Nice choice.

Sounds great! This is a very useful step ahead. Centralizing this functionality makes it so much easier for people to implement to ensure that rich internet applications are accessibile.

+++ b/core/misc/drupal.jsundefined
@@ -259,6 +259,76 @@ Drupal.t = function (str, args, options) {
+      liveElement.id = "drupal-live-announce";

Any particular reason this is the only line with double quotes?

+++ b/core/misc/drupal.jsundefined
@@ -259,6 +259,76 @@ Drupal.t = function (str, args, options) {
+      liveElement.classList.add('element-invisible');

Does this depend on #1754166: Add classList polyfill for IE?

I don't have any more concerns with it, but it is still marked "Needs Documentation" and "Needs tests".

regarding #1754166: Add classList polyfill for IE, the way #1252178: Add Modernizr to core went in, it's not blocked anymore, but would need to depend on Modernizr and carry its own feature-test, I guess

The patch in #14 looks good. As long as the tests pass, this is RTBC.

Great work Jesse and seutje!

just want to know why we're not using setAttribute before this gets committed.

Re-uploading #19 so that the interdiff isn't the last bot-tested patch on this issue. Also, seems odd for a JS only patch to cause Simpletest failures, so let's see if bot comes back happier this time.

Oh, right, locale scans js files for translatable strings, and this issue adds another one, so the test's use of a magic number broke. This makes that test more robust to such incidental changes.

+1 rtbc

Committed to 8.x. Thanks!

This is really great! So happy that @Jesse was inspired to do this. Makes so much sense, but I hadn't heard anyone even discuss this before a week ago when this issue got going.

That's great. Thanks for coming to the sprint and following it up!

Hopefully we'll see this implemented here:
#1855036: ARIA and accessibility improvements in entity (content) translation settings page
#1272990: Make tabledrag warning message show when row weights are enabled, and add WAI-ARIA live region

Where else would this be useful?

A change record for this should be created; this is a new API that Drupal developers should be aware of!

Change notice created: https://drupal.org/node/2014521.

Change notice created: https://drupal.org/node/2014521.

Updated issue summary.

Just wondering, could this be backported to D7? Doesn't seem very complex.

We can try. We've got a few years of D7 left, so it could be good!

Adding tags.

This is what we ended up using in Backdrop: https://patch-diff.githubusercontent.com/raw/backdrop/backdrop/pull/3557...

I hope it helps with the D7 backport. Thanks for everyone that has worked on this issue ❤️

This was committed