Skip to content

fix(library): hide Ready Templates reset chrome; remove Updates footer modal - #686

Merged
CaYatur merged 2 commits into
mainfrom
fix/templates-reset-and-remove-updates-footer
Oct 1, 2026
Merged

CaYatur merged 2 commits into
mainfrom
fix/templates-reset-and-remove-updates-footer

Conversation

@CaYatur

@CaYatur CaYatur commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Ready Templates: Temporarily hide the card-level modified badge and circular reset (hideCardReset: true + sectionShowsResetChrome). Re-enable later by flipping the flag; SCENE_KEYS roots and resetScene() remain ready.
  • Why: Card reset only covered a partial root set vs apply(), so the badge could rebound (e.g. with layer stack / syncStackState) without further user action. Not chasing the full bounce-fix in this PR.
  • Updates: Remove footer download button + Updates modal; keep Updates only as the Settings full-width card.

Test plan

  • npm test — 2547 pass locally
  • CI green
  • Ready Templates card has no badge/reset after applying a template
  • Settings → Updates card still works; footer has no Updates button

…tes footer modal

Ready Templates card badge/reset only tracked visualizer/background/postfx,
while apply() resets the full SCENE_KEYS set. Partial reset left layers/
modulation dirty and, with layerStack on, syncStackState re-cleared
visualizer to none so the modified badge bounced back without user action.

- Align card roots with SVTemplates.SCENE_KEYS + rootOmit for apply preserves
- Add resetScene() and use it from resetSection for permanent factory restore
- Remove footer download button and Updates modal; keep Settings card only
- Tests + README counts 2547
Stop exposing the modified badge and circular reset on Hazır Şablonlar.
Users were hitting a misleading partial reset that could bounce the badge
back. Gate with hideCardReset: true (+ sectionShowsResetChrome) so chrome
can be re-enabled later; SCENE_KEYS roots and resetScene stay in place.
Updates footer modal removal unchanged.
@CaYatur CaYatur changed the title fix(library): Ready Templates reset + remove Updates footer modal fix(library): hide Ready Templates reset chrome; remove Updates footer modal Oct 1, 2026
@CaYatur
CaYatur merged commit 262da8b into main Oct 1, 2026
5 checks passed
@CaYatur
CaYatur deleted the fix/templates-reset-and-remove-updates-footer branch October 1, 2026 17:50
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.

1 participant