Skip to content

Feat: Re-Save view button + animation UI cleanup#91

Merged
arossti merged 2 commits intomainfrom
animation-edits
Feb 13, 2026
Merged

Feat: Re-Save view button + animation UI cleanup#91
arossti merged 2 commits intomainfrom
animation-edits

Conversation

@arossti
Copy link
Owner

@arossti arossti commented Feb 13, 2026

Summary

  • Add Re-Save button to overwrite the active view camera + scene state in-place
  • Reorder bulk actions row to Import | Export | SVG | X
  • Rename Camera label to Camera Only for clarity

Changes

  • index.html: Re-Save button (hidden until view selected), button reorder, label rename
  • modules/rt-viewmanager.js: resaveActiveView() method, _updateResaveButton() helper, click handler wiring, show/hide on select/delete/clear
  • Geometry Documents/Animations.md: Phase 7 documentation

Generated with Claude Code

Co-Authored-By: Andy and Claude andy@openbuilding.ca

…rder

- Add Re-Save button beside Save (hidden until a view is selected via ▶)
- resaveActiveView() overwrites camera + scene state, preserves view identity
- _updateResaveButton() shows/hides on view select, delete, clear
- Reorder bulk actions: Import | Export | SVG | X
- Rename "Camera" label to "Camera Only"
- Document Phase 7 in Animations.md

🤖 Co-Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Andy🤦‍♂️ & Claude🤖 <andy@openbuilding.ca>
🤖 Co-Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Andy🤦‍♂️ & Claude🤖 <andy@openbuilding.ca>
@arossti arossti self-assigned this Feb 13, 2026
@github-actions
Copy link

🤖 Claude Code Review

Looking at this PR, I'll focus on checking the final code for the five types of issues specified.

Code Quality Review

No issues found.

The code is well-structured with:

  • Clear function separation and appropriate length
  • Proper variable usage without unused declarations
  • No debug console.log statements (only intentional logging with emoji prefixes)
  • No commented-out code blocks
  • No significant code duplication

The new Re-Save functionality is implemented cleanly with appropriate state management and UI updates.


ℹ️ About this review

This automated review checks for:

  • Debug statements (console.log)
  • Commented-out code
  • Code duplication
  • Large functions (>50 lines)
  • Unused variables

Human review required - these are suggestions only.
Diff size: 10529 bytes

@arossti arossti merged commit 3cca9c8 into main Feb 13, 2026
3 checks passed
@arossti arossti deleted the animation-edits branch February 13, 2026 06:32
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