Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

A11y: calcite-panel - set focus on dismiss button #740

Closed
AdelheidF opened this issue Jan 7, 2020 · 1 comment
Closed

A11y: calcite-panel - set focus on dismiss button #740

AdelheidF opened this issue Jan 7, 2020 · 1 comment
Assignees
@kat10140
Labels
4 - verified a11y bug Something isn't working p - high
Milestone
Grob Gob Glob Grod

Comments

@AdelheidF
Copy link

Summary

The smartMapping Gallery component info option needs a way to set the focus on the close button of a calcite-panel.

cc @driskull

Actual Behavior

Expected Behavior

Reproduction Steps

Relevant Info

Working W3C Example/Tutorial

@AdelheidF AdelheidF added 0 - new a11y bug Something isn't working p - high labels Jan 7, 2020
@driskull driskull self-assigned this Jan 7, 2020
@driskull driskull added this to the Grob Gob Glob Grod milestone Jan 7, 2020
driskull added a commit that referenced this issue Jan 7, 2020
@driskull
#740 setFocus on dismiss button method
6c920bf
@driskull driskull mentioned this issue Jan 7, 2020
Merged
driskull added a commit that referenced this issue Jan 15, 2020
@driskull
feat: setFocus on dismiss button method #740 (#742)
ec3cfff 
* #740 setFocus on dismiss button method

* review fixes

* review fixes

* rename type

* fix test for review

* add ability to focus on container as well.
@driskull driskull assigned kat10140 and AdelheidF and unassigned driskull Jan 15, 2020
@AdelheidF
Copy link
Author

Focus box seems big, but the focus works,

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kat10140 kat10140

Labels
4 - verified a11y bug Something isn't working p - high
Projects
None yet
Milestone
Grob Gob Glob Grod
Development

No branches or pull requests
3 participants
@driskull @AdelheidF @kat10140