fix(modal): do autofocus on content change#2575
Conversation
|
UPDATE: as coded now it will amplify #2476 / #2477 (comment) issues and restore focus on any modal DOM change even if completely unrelated with inputs I am not sure if it can be fixed easily. IMHO the autoficus on observed change should be set only if: also the #2490 is about tab key trapping, but this PR does not fix it if there is no input in modal to focus |
# Conflicts: # src/definitions/modules/flyout.js # src/definitions/modules/modal.js
|
@lubber-de please see my updated comment, I do not think the current fix is good there are 2 separate problems:
|
I'll do that if
Yes, understood, i am on it |
|
I think i covered all mentioned situations now. |
|
I upgraded atk4/ui to Fomantic-UI 2.9.1 and the autofocus is still not fully working, steps to reproduce:
|
Should finally be fixed by #2674 |
Description
The autofocus feature does not work when the content of a modal or flyout is loaded/changed after the modal is shown (and contains an input after the change)
Testcase
The content of the modal changes after 3 seconds (and then has got an input
Broken
https://jsfiddle.net/lubber/Lzkw5dyq/4/
Fixed
https://jsfiddle.net/lubber/Lzkw5dyq/18/
Closes
#2490