[BX-1431] ESC to dismiss Token Details fails #1521
Conversation
|
Here's the packed extension for this build: |
| setTimeout(() => { | ||
| app?.focus(); | ||
| }, 150); | ||
| }, []); |
There was a problem hiding this comment.
i guess this was as last resource?
There was a problem hiding this comment.
Sort of. This issue exists throughout the app, but I placed a hook inside AnimatedRoute to handle it.
This screen is not using animated route, so I had to put it in manually.
|
Here's the packed extension for this build: |
|
Here's the packed extension for this build: |
|
Here's the packed extension for this build: |
|
Here's the packed extension for this build: |
|
Here's the packed extension for this build: |
|
Here's the packed extension for this build: |
|
Here's the packed extension for this build: |
Co-authored-by: Daniel Sinclair <[email protected]>
What changed (plus any additional context for devs)
Added a few tweaks to the token details page to ensure that ESC behaves correctly
What to test
Make sure you can close the token details modal without closing the popup.