Skip to content
This repository was archived by the owner on Dec 19, 2024. It is now read-only.

feat: add [iron-focusable] to mark as focusable#309

Merged
e111077 merged 1 commit intomasterfrom
focusable-escape-hatch
May 5, 2021
Merged

feat: add [iron-focusable] to mark as focusable#309
e111077 merged 1 commit intomasterfrom
focusable-escape-hatch

Conversation

@e111077
Copy link
Contributor

@e111077 e111077 commented May 5, 2021

addresses but not f1x #282 and PolymerElements/paper-dialog#125

@e111077 e111077 requested a review from rictic May 5, 2021 21:51
@e111077 e111077 requested a review from bicknellr as a code owner May 5, 2021 21:51
@google-cla google-cla bot added the cla: yes label May 5, 2021
@bicknellr
Copy link
Contributor

Would this issue also be fixed by #292? (which I've neglected to review forever...)

@e111077
Copy link
Contributor Author

e111077 commented May 5, 2021

That would indeed also work but I'd be weary of performance

@e111077
Copy link
Contributor Author

e111077 commented May 5, 2021

Also I think we'd need both in case the host has a custom .focus rather than the focus-within inside

@e111077 e111077 merged commit 97af5e9 into master May 5, 2021
@e111077 e111077 deleted the focusable-escape-hatch branch May 5, 2021 23:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants