Comprehensive architectural fix for master trainer announcements modal that was invisible despite JavaScript executing correctly. Root Causes Fixed: 1. Duplicate CSS modal definitions causing cascade conflicts 2. JavaScript using fadeIn() but CSS expecting .active class 3. Inline style="display:none" overriding all CSS rules 4. Browser cache preventing JavaScript updates Changes: - Remove duplicate .hvac-modal CSS definition (lines 794-835) - Remove unused @keyframes fadeIn and slideIn animations - Update openModal() to use .active class + body scroll prevention - Update closeModal() to remove .active class - Remove inline display:none from modal HTML templates - Increment HVAC_VERSION to 2.0.1 for cache busting Testing: - Validated with MCP Playwright browser automation - Visual confirmation of working modal - Code review with Zen GLM-4.6 expert analysis Files Modified: - assets/css/hvac-announcements.css - assets/js/hvac-announcements-admin.js - includes/class-hvac-plugin.php - includes/class-hvac-announcements-admin.php - includes/class-hvac-announcements-display.php Status: Modal now fully functional on staging Next: Fix wp.editor.setContent error, investigate remaining page errors 🤖 Generated with Claude Code (https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| agents | ||
| settings.local.json | ||