From 48d101bf36d2f0d7d90babc4d262c6ccf7bc2666 Mon Sep 17 00:00:00 2001
From: Rich Harris <rich.harris@vercel.com>
Date: Tue, 24 Feb 2026 21:14:26 -0500
Subject: [PATCH] Update
 documentation/docs/97-best-practices/10-best-practices.md

Co-authored-by: ComputerGuy <63362464+Ocean-OS@users.noreply.github.com>
---
 documentation/docs/97-best-practices/10-best-practices.md | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/documentation/docs/97-best-practices/10-best-practices.md b/documentation/docs/97-best-practices/10-best-practices.md
index 8b593c9953..d4d6c50045 100644
--- a/documentation/docs/97-best-practices/10-best-practices.md
+++ b/documentation/docs/97-best-practices/10-best-practices.md
@@ -54,7 +54,11 @@ to allow `<MyComponent bind:value />`. This can get hairy when the value of the
 
 ## Events on elements
 
-Every prop that starts with `on` is treated as an event listener for the element. To register a `click` listener on an element you can do `<button onclick={()=>{}} />` (NOTE: in svelte 5 there's no colon between `on` and `click`). Since elements are just attributes you can spread them, use the `{onclick}` shorthand etc.
+Every prop that starts with `on` is treated as an event listener for the element. To register a `click` listener on an element you can do `<button onclick={() => {}} />` 
+
+> [!NOTE] In Svelte 5, `on` is no longer a directive, so you can't use `on:event`, you have to use `onevent`. 
+
+Since elements are just attributes you can spread them, use the `{onclick}` shorthand, etc.
 
 If you need to attach listeners to `window` or `document` use `<svelte:window onclick>` or `<svelte:document onclick>` instead of using `onMount`/`$effect`