fix(multiple): wrap ngDevMode check around afterRenderEffect in aria primitives

Moving the ngDevMode check outside of afterRenderEffect ensures the
effect is not registered at all in production, rather than registering
an effect that does nothing.

Affected packages: aria/listbox, aria/toolbar

Author: OmerGronich

src/aria/listbox/listbox.ts

@@ -159,14 +159,14 @@ export class Listbox<V> {
       this._popup._controls.set(this._pattern as ComboboxListboxPattern<V>);
     }
 
-    afterRenderEffect(() => {
-      if (typeof ngDevMode === 'undefined' || ngDevMode) {
+    if (typeof ngDevMode === 'undefined' || ngDevMode) {
+      afterRenderEffect(() => {
         const violations = this._pattern.validate();
         for (const violation of violations) {
           console.error(violation);
         }
-      }
-    });
+      });
+    }
 
     afterRenderEffect(() => {
       if (!this._hasFocused()) {

src/aria/toolbar/toolbar.ts

@@ -109,14 +109,14 @@ export class Toolbar<V> {
   private _hasBeenFocused = signal(false);
 
   constructor() {
-    afterRenderEffect(() => {
-      if (typeof ngDevMode === 'undefined' || ngDevMode) {
+    if (typeof ngDevMode === 'undefined' || ngDevMode) {
+      afterRenderEffect(() => {
         const violations = this._pattern.validate();
         for (const violation of violations) {
           console.error(violation);
         }
-      }
-    });
+      });
+    }
 
     afterRenderEffect(() => {
       if (!this._hasBeenFocused()) {