Skip to content

Commit

Permalink
Accessibility toolbar btn #1560 (#1677)
Browse files Browse the repository at this point in the history
* fix(toolbar):toolbar button enclosed in a div which has unwanted role as button

* formatted file
  • Loading branch information
RiyaJethwa authored Nov 1, 2023
1 parent ff62ad6 commit fe0f8ca
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions packages/core/src/components/axes/toolbar.ts
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,9 @@ export class Toolbar extends Component {
})
}

// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore
// eslint-disable-next-line @typescript-eslint/no-unused-vars
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore
// eslint-disable-next-line @typescript-eslint/no-unused-vars
render(animate = true) {
const container = this.getComponentContainer()
.attr('role', 'toolbar')
Expand Down Expand Up @@ -77,7 +77,6 @@ export class Toolbar extends Component {
.enter()
.append('div')
.attr('class', 'toolbar-control cds--overflow-menu cds--overflow-menu')
.attr('role', 'button')

const self = this
enteringToolbarControls
Expand Down

0 comments on commit fe0f8ca

Please sign in to comment.