Fix bug in look and feel where compact mode 'both' would never be selected on page load

This commit is contained in:
brahim
2024-09-22 18:19:29 +02:00
parent 960d733c76
commit b0755d3834

View File

@@ -500,23 +500,21 @@ var gZenLooksAndFeel = {
setCompactModeStyle() { setCompactModeStyle() {
const chooser = document.getElementById('zen-compact-mode-styles-form'); const chooser = document.getElementById('zen-compact-mode-styles-form');
const radios = [...chooser.querySelectorAll('input')]; const radios = [...chooser.querySelectorAll('input')];
for (let radio of radios) {
if (radio.value === 'left' && Services.prefs.getBoolPref('zen.view.compact.hide-tabbar')) { let value = '';
radio.checked = true; if (Services.prefs.getBoolPref('zen.view.compact.hide-tabbar')
} else if (radio.value === 'top' && Services.prefs.getBoolPref('zen.view.compact.hide-toolbar')) { && Services.prefs.getBoolPref('zen.view.compact.hide-toolbar')) {
radio.checked = true; value = 'both';
} else if ( } else {
radio.value === 'both' && value = Services.prefs.getBoolPref('zen.view.compact.hide-tabbar') ? 'left' : 'top';
!Services.prefs.getBoolPref('zen.view.compact.hide-tabbar') &&
!Services.prefs.getBoolPref('zen.view.compact.hide-toolbar')
) {
radio.checked = true;
} }
chooser.querySelector(`[value='${value}']`).checked = true;
const disableExpandTabsOnHover = () => { const disableExpandTabsOnHover = () => {
if (Services.prefs.getBoolPref('zen.view.sidebar-expanded.on-hover')) { if (Services.prefs.getBoolPref('zen.view.sidebar-expanded.on-hover')) {
document.querySelector(`#zen-expand-tabbar-strat [value='expand']`).click(); document.querySelector(`#zen-expand-tabbar-strat input[value='expand']`).click();
} }
} }
for (let radio of radios) {
radio.addEventListener('change', (e) => { radio.addEventListener('change', (e) => {
let value = e.target.value; let value = e.target.value;
switch (value) { switch (value) {