Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

UXPLATFORM-10190 added aria-expanded #4095

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 13 additions & 2 deletions packages/terra-form-select/src/multiple/Frame.jsx
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
/* eslint-disable jsx-a11y/role-supports-aria-props */
import React from 'react';
import PropTypes from 'prop-types';
import classNames from 'classnames';
Expand Down Expand Up @@ -440,7 +441,13 @@ class Frame extends React.Component {
</li>
) : null}
<li className={cx('search-wrapper')}>
<input {...inputAttrs} value={searchValue} />
<input
{...inputAttrs}
value={searchValue}
aria-expanded={this.state.isOpen}
role="combobox"
aria-controls="terra-select-dropdown"
/>
</li>
</ul>
);
Expand Down Expand Up @@ -653,7 +660,11 @@ class Frame extends React.Component {
* Key press events will be caught and evaluated by the frame.
*/
// eslint-disable-next-line jsx-a11y/click-events-have-key-events
<div data-terra-form-select-toggle className={toggleClasses} onClick={this.toggleDropdown}>
<div
data-terra-form-select-toggle
className={toggleClasses}
onClick={this.toggleDropdown}
>
<span className={cx('arrow-icon')} />
</div>
);
Expand Down
Loading