Skip to content

Perf/lightweight modal state#228

Open
binayyub4211 wants to merge 4 commits into
StellarFlow-Network:mainfrom
binayyub4211:perf/lightweight-modal-state
Open

Perf/lightweight modal state#228
binayyub4211 wants to merge 4 commits into
StellarFlow-Network:mainfrom
binayyub4211:perf/lightweight-modal-state

Conversation

@binayyub4211
Copy link
Copy Markdown

Closes #170

fix: optimize modal memory footprint with short-circuit rendering expressions

Description
What was done:

Migrated heavily mounted structural modal overlays to state-driven on-demand rendering.
Modified

src/app/components/nav.jsx
to mount the wallet connection and anomaly alert dialogs only when their triggers are active.
Updated

src/app/governance/page.tsx
to render the proposal submission form conditionally.
Edited

src/app/relayers/page.tsx
to wire the registration modal to on-demand rendering.
Cleaned up

src/app/consumers/page.tsx
by importing missing style variants from classNameVariants.ts and configuring state-driven provision access forms.
Extended

src/app/staking/page.tsx
to handle dynamic APY adjustment and manual slashing forms via conditional rendering hooks.
Styled all newly added interactive overlays using uniform glassmorphic backdrops, premium gradients, HSL tailored variables, and micro-animations.
Why it was done:

Closed modals were retaining heavy node structures in the active DOM tree, leading to layout memory bloat and reduced UI responsiveness during heavy rendering updates. Utilizing short-circuit rendering expressions ensures closed modals have a zero DOM footprint.
How it was verified:

Performed static type analysis on all new lifecycle structures.
Verified layout alignment, prop definitions, and custom state hooks statically.
Updated the local progress tracking checklist inside the task list.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 29, 2026

@binayyub4211 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🔌 Perf | Lightweight State Containers for Modals

1 participant