[matrix] Modal cleanup

* General modal cleanup

* Remove second scrollbar caused by modal
This commit is contained in:
Josh
2020-05-29 17:25:13 +10:00
committed by Rapptz
parent e7c6643730
commit 9653ec5188
3 changed files with 40 additions and 29 deletions

View File

@ -9,7 +9,7 @@ let sidebar;
function closeModal(modal) {
activeModal = null;
modal.style.display = 'none';
modal.hidden = true;
}
function openModal(modal) {
@ -18,7 +18,7 @@ function openModal(modal) {
}
activeModal = modal;
modal.style.removeProperty('display');
modal.hidden = false;
}
function updateSetting(element) {
@ -30,7 +30,11 @@ function updateSetting(element) {
function getRootAttributeToggle(attributeName, valueName) {
function toggleRootAttribute(set) {
document.documentElement.setAttribute(`data-${attributeName}`, set ? valueName : null);
if (set) {
document.documentElement.setAttribute(`data-${attributeName}`, valueName);
} else {
document.documentElement.removeAttribute(`data-${attributeName}`);
}
}
return toggleRootAttribute;
}