fix(search): fix JSX onClick by removing double signal binding

Root cause: class={this.#overlayClass} in render reads a computed signal,
triggering full re-render when the overlay opens/closes. This recreates
the overlay DOM element, losing the JSX onClick handler (attached via
addEventListener in applyProps).

Fix: use static class='overlay' and rely solely on data-signal-attr='class'
for dynamic updates. This avoids full re-render and preserves the click
handler across open/close transitions.

Also reverts the connectedCallback addEventListener workaround — no longer
needed since the VNode onClick handler works correctly with this fix.

Verification:
- deno task test: 894 passed, 0 failed
- e2e chromium: 25/25 passed (search + theme + i18n)
- autoflow:push: 4/4 PASS

Author: DevBot

www/app/islands/open-search.tsx

@@ -215,11 +215,6 @@ export default class OpenSearch extends OpenElement {
   override connectedCallback(): void {
     super.connectedCallback();
     globalThis.addEventListener('keydown', this._onKeydown);
-    // ponytail: VNode onClick delegation does not reliably fire on shadow DOM
-    // children in all browsers. Attach a native click listener directly.
-    this.shadowRoot
-      ?.querySelector('.overlay')
-      ?.addEventListener('click', (e: Event) => this._closeOnBackdrop(e));
   }
 
   override disconnectedCallback(): void {
@@ -369,7 +364,7 @@ export default class OpenSearch extends OpenElement {
         </button>
 
         <div
-          class={this.#overlayClass}
+          class='overlay'
           data-signal='overlayClass'
           data-signal-attr='class'
           onClick={(e: Event) => this._closeOnBackdrop(e)}