Skip to content

new rule: svelte/no-add-event-listner #1194

@baseballyama

Description

@baseballyama

Motivation

We need to use on instead.
https://svelte.dev/docs/svelte/svelte-events#on

Description

Since Svelte 5 uses event delegation, using addEventListener may cause events to fire in unexpected order.
Therefore, we should always use on instead.

Examples

<script>
  import { on } from 'svelte/events';

  onMount(() => {
    // ✓ GOOD
    on(parent, 'click',() => {});
    // ✗ BAD
    parent.addEventListener('click',() => {});
  })
</script>

<div bind:this={parent}>
  <button />
</div>

Additional comments

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions