Remove offscreen clicks closing asset/subasset modals

This commit is contained in:
gitmotion 2025-06-08 15:07:24 -07:00
parent 1209fab88e
commit 73017e40cc
2 changed files with 3 additions and 26 deletions

View File

@ -101,24 +101,7 @@ export class ModalManager {
}
initializeEventListeners() {
// Add escape key listener for all modals
document.addEventListener('keydown', (e) => {
if (e.key === 'Escape') {
this.closeAssetModal();
this.closeSubAssetModal();
}
});
// Add click-off-to-close for modals
[this.assetModal, this.subAssetModal].forEach(modal => {
if (modal) {
modal.addEventListener('mousedown', (e) => {
if (e.target === modal) {
modal.style.display = 'none';
}
});
}
});
// Add any event listeners that are needed for the modals
}
openAssetModal(asset = null) {

View File

@ -1409,17 +1409,11 @@ document.addEventListener('DOMContentLoaded', () => {
}
});
// Add click-off-to-close for all modals on overlay click
[assetModal, subAssetModal, importModal, settingsModal].forEach(modal => {
[importModal, settingsModal].forEach(modal => {
if (modal) {
modal.addEventListener('mousedown', function(e) {
if (e.target !== modal) return;
if (modal === assetModal) {
modalManager.closeAssetModal();
}
else if (modal === subAssetModal) {
modalManager.closeSubAssetModal();
}
else if (modal === settingsModal) {
if (modal === settingsModal) {
settingsManager.closeSettingsModal();
}
else {