fix: remove label update code from toggle functions
- Removed references to non-existent lengthLabel, logoLabel, and themeLabel elements - Fixed "Cannot set properties of null" JavaScript errors - Toggles now work correctly without trying to update removed label text - Cleaned up unnecessary language variable declarations
This commit is contained in:
@@ -231,15 +231,11 @@
|
||||
function toggleCVLength() {
|
||||
const toggle = document.getElementById('lengthToggle');
|
||||
const paper = document.querySelector('.cv-paper');
|
||||
const lang = document.documentElement.lang;
|
||||
const label = document.getElementById('lengthLabel');
|
||||
|
||||
if (toggle.checked) {
|
||||
label.textContent = lang === 'es' ? 'Largo' : 'Long';
|
||||
paper.classList.add('cv-long');
|
||||
paper.classList.remove('cv-short');
|
||||
} else {
|
||||
label.textContent = lang === 'es' ? 'Corto' : 'Short';
|
||||
paper.classList.add('cv-short');
|
||||
paper.classList.remove('cv-long');
|
||||
}
|
||||
@@ -248,14 +244,10 @@
|
||||
function toggleLogos() {
|
||||
const toggle = document.getElementById('logoToggle');
|
||||
const paper = document.querySelector('.cv-paper');
|
||||
const lang = document.documentElement.lang;
|
||||
const label = document.getElementById('logoLabel');
|
||||
|
||||
if (toggle.checked) {
|
||||
label.textContent = lang === 'es' ? 'Con' : 'On';
|
||||
paper.classList.add('show-logos');
|
||||
} else {
|
||||
label.textContent = lang === 'es' ? 'Sin' : 'Off';
|
||||
paper.classList.remove('show-logos');
|
||||
}
|
||||
}
|
||||
@@ -263,15 +255,11 @@
|
||||
function toggleTheme() {
|
||||
const toggle = document.getElementById('themeToggle');
|
||||
const container = document.querySelector('.cv-container');
|
||||
const lang = document.documentElement.lang;
|
||||
const label = document.getElementById('themeLabel');
|
||||
|
||||
if (toggle.checked) {
|
||||
label.textContent = lang === 'es' ? 'Limpia' : 'Clean';
|
||||
container.classList.add('theme-clean');
|
||||
localStorage.setItem('cv-theme', 'clean');
|
||||
} else {
|
||||
label.textContent = lang === 'es' ? 'Completa' : 'Full';
|
||||
container.classList.remove('theme-clean');
|
||||
localStorage.setItem('cv-theme', 'default');
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user