juanatsap
3f77fedeaf
CRITICAL FIX: Icon toggle now works without page refresh - Changed class name from 'show-logos' to 'show-icons' (CSS mismatch bug) - Updated localStorage key from 'cv-logos' to 'cv-icons' - Fixed toggleIcons() function in cv-functions.js HYPERSCRIPT ARCHITECTURE: - Moved 6 toggle functions from hyperscript to JavaScript (cv-functions.js) - Solves hyperscript 0.9.14 parser limitation (max 3 def statements total) - Upgraded hyperscript from 0.9.12 to 0.9.14 - Fixed operator precedence in keyboard shortcuts - Cleaned view-controls.html templates (inline → function calls) NEW FILES: - static/js/cv-functions.js - Global toggle functions (6 functions) - HYPERSCRIPT-RULES.md - Permanent architecture documentation - tests/mjs/0-zoom.test.mjs - Zoom functionality test - tests/mjs/1-toggles.test.mjs - Comprehensive toggle test with real-time verification - tests/TEST-SUMMARY.md - Test suite documentation TESTS: - Real-time DOM update verification (no refresh required) - Screenshot capture for visual regression - localStorage persistence validation - Toggle synchronization between action bar and menu BREAKING CHANGE: localStorage key changed from 'cv-logos' to 'cv-icons' Users may need to re-toggle icons preference on first load after update.
134 lines
3.9 KiB
Plaintext
134 lines
3.9 KiB
Plaintext
-- ==============================================================================
|
|
-- CV Site - Hyperscript Functions
|
|
-- ==============================================================================
|
|
-- Global hyperscript functions for CV interactive features
|
|
-- Keeps HTML templates clean and behavior organized
|
|
|
|
-- ==============================================================================
|
|
-- PRINT FUNCTIONS
|
|
-- ==============================================================================
|
|
|
|
def printFriendly()
|
|
-- Store current state
|
|
set container to the first .cv-container
|
|
set paper to the first .cv-paper
|
|
set wasClean to container.classList.contains('theme-clean')
|
|
set wasLong to paper.classList.contains('cv-long')
|
|
set currentZoom to localStorage.getItem('cv-zoom') or '100'
|
|
|
|
-- Apply print-friendly settings
|
|
if wasClean is false
|
|
add .theme-clean to container
|
|
end
|
|
remove .cv-long from paper
|
|
add .cv-short to paper
|
|
|
|
-- Reset zoom for consistent printing
|
|
set #zoom-wrapper's *zoom to 1
|
|
|
|
-- Let CSS apply, then print
|
|
wait 50ms
|
|
call window.print()
|
|
|
|
-- Wait for print dialog to close, then restore
|
|
wait 100ms
|
|
|
|
-- Restore original theme
|
|
if wasClean is false
|
|
remove .theme-clean from container
|
|
end
|
|
|
|
-- Restore original length
|
|
if wasLong is true
|
|
remove .cv-short from paper
|
|
add .cv-long to paper
|
|
end
|
|
|
|
-- Restore original zoom by triggering slider input event
|
|
if currentZoom is not '100'
|
|
set #zoom-slider's value to currentZoom
|
|
send input to #zoom-slider
|
|
end
|
|
end
|
|
|
|
-- ==============================================================================
|
|
-- SCROLL BEHAVIOR
|
|
-- ==============================================================================
|
|
|
|
def initScrollBehavior()
|
|
set :lastScroll to 0
|
|
set :scrollThreshold to 100
|
|
set :keepHeaderVisible to false
|
|
end
|
|
|
|
def handleScroll()
|
|
set currentScroll to window.pageYOffset
|
|
set menu to the first .navigation-menu
|
|
set isMenuOpen to menu.classList.contains('menu-open')
|
|
|
|
-- Calculate if at bottom (within 50px)
|
|
set scrollHeight to document.documentElement.scrollHeight
|
|
set clientHeight to document.documentElement.clientHeight
|
|
set isAtBottom to (scrollHeight - currentScroll - clientHeight) < 50
|
|
|
|
-- Reset keepHeaderVisible when scrolling up
|
|
if currentScroll < :lastScroll
|
|
set :keepHeaderVisible to false
|
|
end
|
|
|
|
-- Header visibility: Scrolling down past threshold
|
|
if currentScroll > :scrollThreshold and currentScroll > :lastScroll and :keepHeaderVisible is false
|
|
add .header-hidden to .action-bar
|
|
if isMenuOpen is true
|
|
add .header-hidden to menu
|
|
end
|
|
end
|
|
|
|
-- Header visibility: Scrolling up past threshold
|
|
if currentScroll > :scrollThreshold and (currentScroll <= :lastScroll or :keepHeaderVisible is true)
|
|
remove .header-hidden from .action-bar
|
|
if isMenuOpen is true
|
|
remove .header-hidden from menu
|
|
end
|
|
end
|
|
|
|
-- Header visibility: At top
|
|
if currentScroll <= :scrollThreshold
|
|
remove .header-hidden from .action-bar
|
|
if isMenuOpen is true
|
|
remove .header-hidden from menu
|
|
end
|
|
end
|
|
|
|
-- Back to top button visibility
|
|
if currentScroll > 300
|
|
set #back-to-top's *display to 'flex'
|
|
end
|
|
|
|
if currentScroll <= 300
|
|
set #back-to-top's *display to 'none'
|
|
end
|
|
|
|
-- At-bottom class for fixed buttons
|
|
if isAtBottom
|
|
add .at-bottom to #back-to-top
|
|
add .at-bottom to #info-button
|
|
add .at-bottom to #shortcuts-button
|
|
end
|
|
|
|
if not isAtBottom
|
|
remove .at-bottom from #back-to-top
|
|
remove .at-bottom from #info-button
|
|
remove .at-bottom from #shortcuts-button
|
|
end
|
|
|
|
-- Update last scroll position
|
|
set :lastScroll to currentScroll
|
|
end
|
|
|
|
-- ==============================================================================
|
|
-- KEYBOARD SHORTCUTS
|
|
-- ==============================================================================
|
|
-- Note: Keyboard event handlers are now defined inline in the body tag
|
|
-- because hyperscript 0.9.12 doesn't support nested event handlers (on ... inside def)
|