serv/mox
1
0
Fork 0
mirror of https://github.com/mjl-/mox.git synced 2024-12-26 16:33:47 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

webmail: if we don't have loaded account settings yet, abort loading the popup after showing an error that the settings aren't available yet

Browse source 
missing returning/throwing error.

based on screenshot with unhandled js error in issue #218 by mgkirs
This commit is contained in:
Mechiel Lukkien 2024-10-10 14:29:52 +02:00
parent 0fbf24160c
commit 04305722a7
No known key found for this signature in database
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
webmail/webmail.js
View file

@ -2391,7 +2391,7 @@ const cmdSettings = async () => {
let showAddressSecurity; let showAddressSecurity;
let showHTML; let showHTML;
if (!accountSettings) { if (!accountSettings) {
window.alert('No account settings fetched yet.'); throw new Error('No account settings fetched yet.');
} }
const remove = popup(css('popupSettings', { padding: '1em 1em 2em 1em', minWidth: '30em' }), dom.h1('Settings'), dom.form(async function submit(e) { const remove = popup(css('popupSettings', { padding: '1em 1em 2em 1em', minWidth: '30em' }), dom.h1('Settings'), dom.form(async function submit(e) {
e.preventDefault(); e.preventDefault();

2
webmail/webmail.ts
View file

@ -1117,7 +1117,7 @@ const cmdSettings = async () => {
let showHTML: HTMLInputElement let showHTML: HTMLInputElement
if (!accountSettings) { if (!accountSettings) {
window.alert('No account settings fetched yet.') throw new Error('No account settings fetched yet.')
} }
const remove = popup( const remove = popup(