fix(GlassSurface): drop the duplicated resize effect

[noron12234]

The problem

GlassSurface registers the same effect twice. Not similar — byte-identical:

useEffect(() => {
  if (!containerRef.current) return;
  const resizeObserver = new ResizeObserver(() => {
    setTimeout(updateDisplacementMap, 0);
  });
  resizeObserver.observe(containerRef.current);
  return () => { resizeObserver.disconnect(); };
}, []);

useEffect(() => {                       // <- the same block again, verbatim
  if (!containerRef.current) return;
  const resizeObserver = new ResizeObserver(() => {
    setTimeout(updateDisplacementMap, 0);
  });
  resizeObserver.observe(containerRef.current);
  return () => { resizeObserver.disconnect(); };
}, []);

So every instance runs two ResizeObservers on the same element, and each resize regenerates the SVG displacement map twice — that's a feImage data-URI rebuild plus feDisplacementMap attribute writes, done twice for nothing.

Measurement

One <GlassSurface> in a stock Vite React app, with ResizeObserver wrapped to count constructions:

	ResizeObservers constructed
current main	2
this PR	1

Still renders identically after the change — same three feDisplacementMap nodes, no console errors, and resizes still update the filter (verified by resizing twice via a width prop).

Scope

All four variants carry the same duplication, so all four are fixed: content, tailwind, ts-default, ts-tailwind. The diff is deletion only — 56 lines removed, nothing added.

Verification

• npx tsc --noEmit: no new errors (zero mentioning GlassSurface).
• npx prettier --check clean on all four files.
• npx vite build passes.
• Browser check above, before and after.

Found while scanning for effects that create a resource without releasing it; this one turned up because the duplicate made the same file report the same pattern twice.

[Copilot]

Pull request overview

This pull request removes a byte-identical, duplicated ResizeObserver useEffect from the GlassSurface component implementations so each instance registers only one observer and performs a single displacement-map regeneration per resize.

Changes:

• Removed the duplicated ResizeObserver effect in the TS + Tailwind variant.
• Removed the duplicated ResizeObserver effect in the TS + CSS variant.
• Removed the duplicated ResizeObserver effect in the JS + Tailwind and JS + CSS variants.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
src/ts-tailwind/Components/GlassSurface/GlassSurface.tsx	Removes the duplicated ResizeObserver useEffect block.
src/ts-default/Components/GlassSurface/GlassSurface.tsx	Removes the duplicated ResizeObserver useEffect block.
src/tailwind/Components/GlassSurface/GlassSurface.jsx	Removes the duplicated ResizeObserver useEffect block.
src/content/Components/GlassSurface/GlassSurface.jsx	Removes the duplicated ResizeObserver useEffect block.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[noron12234]

Good catch, the review is right. public/r/*.json embeds the component source verbatim and that is what the jsrepo CLI actually writes into a user's project, so fixing only the four source files still shipped the duplicated effect.

Pushed afa0dba: regenerated with npm run registry:build. Only the four GlassSurface entries change, one line each, and each now carries a single new ResizeObserver instead of two.

Same gap applied to my three other open PRs, so I have pushed the rebuilt artifacts there too: #1018, #1020, #1022.