Skip to content

Commit

Permalink
docs(test-utils): add tip about using trigger('focus') (#1848)
Browse files Browse the repository at this point in the history
When using `trigger('focus')` with jsdom v16.4.0 and above you must
use `attachTo` when mounting the component.

Co-authored-by: peterhegman <phegman@gitlab.com>
  • Loading branch information
phegman and peterhegman committed May 26, 2021
1 parent 7e78174 commit 83aa823
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions docs/api/wrapper/trigger.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,10 @@ test('trigger demo', async () => {
})
```

::: tip
When using `trigger('focus')` with [jsdom v16.4.0](https://github.com/jsdom/jsdom/releases/tag/16.4.0) and above you must use the [attachTo](../options.md#attachto) option when mounting the component. This is because a bug fix in [jsdom v16.4.0](https://github.com/jsdom/jsdom/releases/tag/16.4.0) changed `el.focus()` to do nothing on elements that are disconnected from the DOM.
:::

- **Setting the event target:**

Under the hood, `trigger` creates an `Event` object and dispatches the event on the Wrapper element.
Expand Down

0 comments on commit 83aa823

Please sign in to comment.