fix(deps): update dependency focus-trap to v8#2685
Open
renovate[bot] wants to merge 1 commit into
Open
Conversation
1fc9f0b to
bca54b7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
^7.8.0→^8.0.0Release Notes
focus-trap/focus-trap (focus-trap)
v8.2.1Compare Source
Patch Changes
7e0e518: Fixes lifecycle ordering bug introduced in v8.0.0 that always delays the order even ifdelayInitialFocus=false,delayReturnFocus=false,checkCanFocusTrap=undefined,checkCanReturnFocus=undefined(which would indicate a fully synchronous activation/pause/unpause/deactivation process) #18622cba31e: Improve shadow DOM focus handling for nested traps by ensuring a parent trap resumes using the innermost active element (#1885)v8.2.0Compare Source
Minor Changes
567dbe1: Add newdelayReturnFocusoption (default true) to control whether return focus and onPostDeactivate are deferred by one frame (#1689).Patch Changes
b70e8d9: Fix bug where removing the ancestor of a focused node within a trap would result in focus landing on the body instead of remaining in the trap (#1660).v8.1.0Compare Source
Minor Changes
642f7f2: Update lifecycle hooks to include the associated focus trap as a parameter.v8.0.1Compare Source
Patch Changes
7d5010e: Loosen checkCanFocusTrap Promise resolution type tounknownto make it easier to usePromise.all()orPromise.allSettled()as the returned Promise (Promise<void>was causing issues because those Promise APIs do not resolve with avoidvalue).v8.0.0Compare Source
Major Changes
b6ea4b5: Breaking: Fixed a long-standing bug whereonPostActivate()would be called before the initial focus node was focused and the trap was fully activated. (#1747)setTimeout(0)) but wasn't delaying callingonPostActivate()until after that delay.isolateSubtrees='aria-hidden'option, the currently-focused node's container (a non-subtree being "disabled") would get hidden before the delay was up, resulting in Chrome preventing the effect ofaria-hiddenon that subtree with a warning in the console due to the container being hidden still containing focus (e.g. the "activate trap" button).onPostActivate()await the initial focus delay (if there is one, which is default behavior; remove it withdelayInitialFocus=false) before being applied/called.await waitFor(() => expect(initialFocusNode).toBeFocused()).onPostActivate()handler (prior to this change, that node would not be focused yet; after this change, it will be focused).Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.