Merge pull request #3 from jnachtigall/aria-owns-not-empty

Fix JedWatson#1667: Do not output invalid empty 'aria-owns' attr
jnachtigall · Feb 28, 2018 · 728e2a5 · 728e2a5
2 parents 8da7329 + 1be2865
commit 728e2a5
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Select.js b/src/Select.js
@@ -876,7 +876,7 @@ class Select extends React.Component {
 			'aria-haspopup': '' + isOpen,
 			'aria-label': this.props['aria-label'],
 			'aria-labelledby': this.props['aria-labelledby'],
-			'aria-owns': ariaOwns,
+			...(ariaOwns ? { 'aria-owns': ariaOwns } : {}),
 			className: className,
 			onBlur: this.handleInputBlur,
 			onChange: this.handleInputChange,
@@ -903,7 +903,7 @@ class Select extends React.Component {
 				<div
 					{...divProps}
 					aria-expanded={isOpen}
-					aria-owns={ariaOwns}
+					{...(ariaOwns ? { 'aria-owns': ariaOwns } : {})}
 					aria-activedescendant={isOpen ? this._instancePrefix + '-option-' + focusedOptionIndex : this._instancePrefix + '-value'}
 					aria-disabled={'' + this.props.disabled}
 					aria-label={this.props['aria-label']}