CSS: - Remove duplicate li::before block in app-search.css - Fix typo: search-reuslt → search-result in app-search.css - Remove commented-out CSS rules in app-flash.css - Add descriptive header comment to all 27 CSS component files JS: - Complete WAI-ARIA Tabs pattern: generate IDs, add aria-controls on tab buttons and aria-labelledby on tab panels (app-tabs.js) - Add JSDoc header comments to ~33 JS files (core + components) - Add explanatory block comment to app-boot.js (why classic IIFE) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
75 lines
2.0 KiB
JavaScript
75 lines
2.0 KiB
JavaScript
/**
|
|
* Toggles visibility of the detail page aside column.
|
|
*/
|
|
import { optionalEl, warnOnce } from '../core/app-dom.js';
|
|
|
|
export function initDetailsAsideToggle(options = {}) {
|
|
const {
|
|
buttonSelector = '#toggle-main-content-aside',
|
|
asideSelector = '#app-details-aside-section',
|
|
containerSelector = '.app-details-container',
|
|
storageKey = 'app-details-aside-collapsed'
|
|
} = options;
|
|
|
|
const button = optionalEl(buttonSelector);
|
|
const aside = optionalEl(asideSelector);
|
|
if (!button || !aside) {return;}
|
|
|
|
const container = aside.closest(containerSelector) || document.querySelector(containerSelector);
|
|
if (!container) {
|
|
warnOnce('UI_EL_MISSING', `Missing details container: ${containerSelector}`, { module: 'details-aside' });
|
|
return;
|
|
}
|
|
|
|
if (!button.getAttribute('aria-controls')) {
|
|
button.setAttribute('aria-controls', aside.id || 'app-details-aside-section');
|
|
}
|
|
|
|
const readStored = () => {
|
|
try {
|
|
return window.localStorage.getItem(storageKey);
|
|
} catch (error) {
|
|
return null;
|
|
}
|
|
};
|
|
|
|
const writeStored = (collapsed) => {
|
|
try {
|
|
window.localStorage.setItem(storageKey, collapsed ? '1' : '0');
|
|
} catch (error) {
|
|
// ignore storage errors
|
|
}
|
|
};
|
|
|
|
const applyState = (collapsed, persist = false) => {
|
|
container.classList.toggle('is-aside-collapsed', collapsed);
|
|
aside.hidden = collapsed;
|
|
button.setAttribute('aria-expanded', collapsed ? 'false' : 'true');
|
|
if (persist) {
|
|
writeStored(collapsed);
|
|
}
|
|
};
|
|
|
|
const getInitialState = () => {
|
|
const stored = readStored();
|
|
if (stored === '1' || stored === '0') {
|
|
return stored === '1';
|
|
}
|
|
return false;
|
|
};
|
|
|
|
applyState(getInitialState(), false);
|
|
|
|
button.addEventListener('click', (event) => {
|
|
event.preventDefault();
|
|
const collapsed = !container.classList.contains('is-aside-collapsed');
|
|
applyState(collapsed, true);
|
|
});
|
|
}
|
|
|
|
if (document.readyState === 'loading') {
|
|
document.addEventListener('DOMContentLoaded', () => initDetailsAsideToggle());
|
|
} else {
|
|
initDetailsAsideToggle();
|
|
}
|