fix: Bidirectional hover sync + theme-aware CV text colors
**Bug 1: Bidirectional Hover Sync** - Fixed hover sync to work in BOTH directions - Hovering left buttons → highlights menu buttons (was working) - Hovering menu buttons → highlights left buttons (NOW WORKING) - Added .download-btn and .print-friendly-btn to syncPdfHover/syncPrintHover selectors **Bug 2: Theme-Aware CV Text Colors** - CV content text now adapts to light/dark themes - Light theme: Dark text (#1a1a1a, #333333) for readability - Dark theme: Light text (#e0e0e0, #d0d0d0) for readability - Added theme-specific overrides for --text-dark and --text-gray variables - Solution uses CSS variables (not hardcoded) for future flexibility Changes: - static/hyperscript/hover-sync._hs: Added left-side button selectors - static/css/color-theme.css: Added legacy variable overrides for all themes
This commit is contained in:
@@ -9,8 +9,8 @@
|
||||
-- ==============================================================================
|
||||
|
||||
def syncPdfHover(show)
|
||||
-- Select all PDF buttons (action bar + menu)
|
||||
set pdfButtons to <.pdf-btn, .menu-pdf-btn/>
|
||||
-- Select all PDF buttons (left side + action bar + menu)
|
||||
set pdfButtons to <.download-btn, .pdf-btn, .menu-pdf-btn/>
|
||||
|
||||
if show is true
|
||||
for btn in pdfButtons
|
||||
@@ -28,8 +28,8 @@ end
|
||||
-- ==============================================================================
|
||||
|
||||
def syncPrintHover(show)
|
||||
-- Select all Print buttons (action bar + menu)
|
||||
set printButtons to <.print-btn, .menu-print-btn/>
|
||||
-- Select all Print buttons (left side + action bar + menu)
|
||||
set printButtons to <.print-friendly-btn, .print-btn, .menu-print-btn/>
|
||||
|
||||
if show is true
|
||||
for btn in printButtons
|
||||
|
||||
Reference in New Issue
Block a user