New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(value-list): add back instructions for screen reader when drag handle is activated #6402
Conversation
…ndle is activated
This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from a few comments, this LGTM!
* @internal | ||
*/ | ||
@Event({ cancelable: true }) | ||
calciteValueListItemDragHandleBlur: EventEmitter<ListItemAndHandle>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to use the getHandleAndItemElement
util to avoid passing the item and handle here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i tried doing so but the focusEvent
doesn't show up the handle element in either composedPath
or event.target
@@ -457,6 +458,14 @@ export class ValueList< | |||
} | |||
}; | |||
|
|||
@Listen("calciteValueListItemDragHandleBlur") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this can be achieved by using focusout
, but let's proceed with this for now and please open a follow-up issue to revisit this approach.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a follow-up issue #6526
Related Issue: #6401
Summary
This PR will add back instructions for screen reader when drag handle is activated via
Space
key.Sidebar: This will also cover #5739