Removing some radix references

Author: SpicyPete

frontend/src/components/misc/user-preferences.tsx

@@ -56,7 +56,7 @@ export const UserPreferencesDialog: FC<{ isOpen: boolean; onClose: () => void }>
 
   return (
     <Dialog onOpenChange={(open) => !open && onClose()} open={isOpen}>
-      <DialogContent className="min-h-[50vh]" size="xl" >
+      <DialogContent className="min-h-[50vh]" size="xl">
         <DialogHeader>
           <DialogTitle>User Preferences</DialogTitle>
         </DialogHeader>

frontend/src/components/redpanda-ui/lib/use-controllable-state.ts

@@ -90,7 +90,7 @@ function useUncontrolledState<T>({
     onChangeRef.current = onChange;
   }, [onChange]);
 
-  // biome-ignore lint/correctness/useExhaustiveDependencies: because radix did it
+  // biome-ignore lint/correctness/useExhaustiveDependencies: onChangeRef is a stable ref, intentionally omitted to avoid re-firing on every onChange identity change
   React.useEffect(() => {
     if (prevValueRef.current !== value) {
       onChangeRef.current?.(value);

frontend/vitest.setup.ts

@@ -35,9 +35,8 @@ Object.defineProperty(HTMLCanvasElement.prototype, 'getContext', {
   }),
 });
 
-// Mock ResizeObserver - not available in jsdom but required by RadixUI components
-// (Switch, Tabs, etc.) that use @radix-ui/react-use-size internally
-// Mock the ResizeObserver
+// Mock ResizeObserver - not available in jsdom but used by registry components
+// (Tabs, Tags, etc.) and other size-aware UI code.
 const ResizeObserverMock = vi.fn(() => ({
   observe: vi.fn(),
   unobserve: vi.fn(),