Merge pull request #10 from Yoast/stories/af/9-strip-html

Add simple replace to strip HTML tags.
WordPress · Oct 28, 2016 · a968fff · a968fff
2 parents 20c58f4 + 6f5752e
commit a968fff
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/a11y-speak.js b/a11y-speak.js
@@ -76,6 +76,15 @@ var A11ySpeak = function( message, ariaLive ) {
 	// Clear previous messages to allow repeated strings being read out.
 	clear();
 
+	/*
+	 * Strip HTML tags (if any) from the message string. Ideally, messages should
+	 * be simple strings, carefully crafted for specific use with A11ySpeak.
+	 * When re-using already existing strings this will ensure simple HTML to be
+	 * stripped out and replaced with a space. Browsers will collapse multiple
+	 * spaces natively.
+	 */
+	message = message.replace( /<[^<>]+>/g, " " );
+
 	if ( containerAssertive && "assertive" === ariaLive ) {
 		containerAssertive.textContent = message;
 	} else if ( containerPolite ) {

diff --git a/a11y-speak.min.js b/a11y-speak.min.js