Skip to content

Commit

Permalink
fixing a11y errors so we can add a11y rules for tslint (#26902) (#26921)
Browse files Browse the repository at this point in the history
  • Loading branch information
bhavyarm committed Dec 10, 2018
1 parent 6232b19 commit dd8e16a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ class BasicLoginFormUI extends Component<Props, State> {
onChange={this.onUsernameChange}
disabled={this.state.isLoading}
isInvalid={false}
aria-required
aria-required={true}
inputRef={this.setUsernameInputRef}
/>
</EuiFormRow>
Expand All @@ -79,7 +79,7 @@ class BasicLoginFormUI extends Component<Props, State> {
onChange={this.onPasswordChange}
disabled={this.state.isLoading}
isInvalid={false}
aria-required
aria-required={true}
/>
</EuiFormRow>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ class EditRolePageUI extends Component<Props, State> {
<Fragment>
<EuiSpacer size="s" />
<EuiText size="s" color="subdued">
<p id="reservedRoleDescription" tabIndex={1}>
<p id="reservedRoleDescription" tabIndex={0}>
<FormattedMessage
id="xpack.security.management.editRole.modifyingReversedRolesDescription"
defaultMessage="Reserved roles are built-in and cannot be removed or modified."
Expand Down

0 comments on commit dd8e16a

Please sign in to comment.