Fixed critical issue where Accept and Cancel buttons in the floating editor
were not properly clearing the currentFloatingMenu reference after hiding.
PROBLEM IDENTIFIED:
- Accept/Cancel buttons called floatingMenu.hide() but left stale reference
- DOMRenderer.currentFloatingMenu remained pointing to hidden menu object
- This caused incorrect state tracking and prevented proper menu lifecycle
SOLUTION IMPLEMENTED:
- Added this.currentFloatingMenu = null after floatingMenu.hide() calls
- Applied fix to both text editor and image editor accept/cancel buttons
- Ensures clean menu state management and proper reference cleanup
TESTING:
- Added comprehensive test for Accept button functionality
- Added comprehensive test for Cancel button functionality
- Both tests verify menu is properly hidden and references cleared
- All 12 integration tests now pass with button functionality validated
This fix ensures users can properly save or discard changes when editing
sections, restoring the expected click-to-edit workflow behavior.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
tegwick
0cedcaf5c8