Skip to content

Respect returnFocusOnDeactivate when deactivating trap#169

Merged
stefcameron merged 1 commit intomasterfrom
return-focus-103
Oct 3, 2020
Merged

Respect returnFocusOnDeactivate when deactivating trap#169
stefcameron merged 1 commit intomasterfrom
return-focus-103

Conversation

@stefcameron
Copy link
Copy Markdown
Member

Fixes #103

The flag wasn't being respected when the trap is deactivated as
a result of clickOutsideDeactivates=true option.

Features and Bug Fixes

  • Issue being fixed is referenced.
  • Test coverage added/updated.
  • Typings added/updated.
  • README updated (API changes, instructions, etc.).
  • Changes to dependencies explained.
  • Changeset added (run yarn changeset locally to add one, follow prompts).

Fixes #103

The flag wasn't being respected when the trap is deactivated as
a result of `clickOutsideDeactivates=true` option.
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Oct 3, 2020

🦋 Changeset detected

Latest commit: 2eb0f0d

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@stefcameron stefcameron merged commit fe2b0ad into master Oct 3, 2020
@stefcameron stefcameron deleted the return-focus-103 branch October 3, 2020 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Return focus on outside click does not respect "returnFocusOnDeactivate" value

1 participant