For #4707 (#4708)

Line 32 has to be `aria-current` instead of `ariaCurrent`, otherwise React will strip it out.
remix-run · Jul 5, 2017 · 89093a4 · 89093a4
1 parent 64f0aec
commit 89093a4
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/packages/react-router-dom/modules/NavLink.js b/packages/react-router-dom/modules/NavLink.js
@@ -16,6 +16,7 @@ const NavLink = ({
   activeStyle,
   style,
   isActive: getIsActive,
+  ariaCurrent,
   ...rest
 }) => (
   <Route
@@ -31,6 +32,7 @@ const NavLink = ({
           to={to}
           className={isActive ? [ className, activeClassName ].filter(i => i).join(' ') : className}
           style={isActive ? { ...style, ...activeStyle } : style}
+          aria-current={isActive && ariaCurrent}
           {...rest}
         />
       )
@@ -47,11 +49,13 @@ NavLink.propTypes = {
   className: PropTypes.string,
   activeStyle: PropTypes.object,
   style: PropTypes.object,
-  isActive: PropTypes.func
+  isActive: PropTypes.func,
+  ariaCurrent: PropTypes.oneOf(['page', 'step', 'location', 'true'])
 }
 
 NavLink.defaultProps = {
-  activeClassName: 'active'
+  activeClassName: 'active',
+  ariaCurrent: 'true'
 }
 
 export default NavLink