fix: aria-live bug and elements update

Pearson-Higher-Ed · Feb 20, 2017 · 4823c86 · 4823c86
1 parent ab11ad0
commit 4823c86
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -78,7 +78,7 @@
     "json-loader": "~0.5.4",
     "mocha": "~2.4.5",
     "node-sass": "~3.7.0",
-    "pearson-elements": "^1.1.0",
+    "pearson-elements": "^1.5.0",
     "react": "^15.0.0",
     "react-addons-test-utils": "^15.0.0",
     "react-dom": "^15.0.0",

diff --git a/src/js/Alert.js b/src/js/Alert.js
@@ -1,7 +1,6 @@
 import React, { Component } from 'react';
 import Icon                 from './Icon';
 
-
 class Alert extends Component {
 
   render () {

diff --git a/src/js/AlertList.js b/src/js/AlertList.js
@@ -33,6 +33,7 @@ class AlertList extends Component {
       <div role="alert">
         <ReactCSSTransitionGroup
           component="ul"
+          aria-live="polite"
           transitionName="transition"
           transitionEnterTimeout={300}
           transitionLeaveTimeout={200}

diff --git a/src/js/Icon.js b/src/js/Icon.js
@@ -4,10 +4,8 @@ const Icon = (props) => {
 
   return (
     <svg version = "1.1"
-       xmlns = "http://www.w3.org/2000/svg"
-       xmlnsXlink = "http://www.w3.org/1999/xlink"
        aria-hidden = "true"
-       focusable = "no"
+       focusable = "false"
        className = {icon_class}>
       <use xlinkHref = {'#' + props.name}></use>
     </svg>