lion
lion copied to clipboard
fix(form-core) sync autofocus to focusable node (fixes #1775)
What I did
- In FocusMixin synced the autofocus attribute from the host to the focusable node.
- Added tests to ensure the sync happens if autofocus was set on the host during render or when autofocus is added or removed from the host.
⚠️ No Changeset found
Latest commit: 5a4b794ceea040c7bc0c09ebb88d5e7cdc6b9c4d
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Hi @Sciurus7, thanks for the contribution! Somehow i missed this one completely, my excuses.
Could you resolve the conflicts and add a changeset (npm run changeset)?
fix: https://github.com/ing-bank/lion/issues/1775
Hi Gerjan, I finally managed to get my personal git account in a working state again. While trying to bring this pull request up to date I messed up a merge, so instead of trying to fix that particular mess, I branched of master again added the changes to an up to date branch and created a new pull request. This one should be up to date and has a change set. I will close this pull request accordingly.