diff --git a/.Jules/palette.md b/.Jules/palette.md new file mode 100644 index 0000000..43c5f2e --- /dev/null +++ b/.Jules/palette.md @@ -0,0 +1,3 @@ +## 2026-06-19 - Added loading states to auth forms +**Learning:** Missing loading states for async operations like login/register can lead to user confusion and potentially multiple form submissions. +**Action:** Always add visual feedback by disabling the submit button and updating its text to indicate a loading state during async API calls, and reliably restore state using a finally block. diff --git a/web-demo/js/app.js b/web-demo/js/app.js index 11508de..68b8c2b 100644 --- a/web-demo/js/app.js +++ b/web-demo/js/app.js @@ -144,6 +144,14 @@ class ClimaAI { e.preventDefault(); const email = document.getElementById('loginEmail').value; const password = document.getElementById('loginPassword').value; + const submitBtn = e.submitter; + let originalContent = ''; + + if (submitBtn) { + originalContent = submitBtn.innerHTML; + submitBtn.disabled = true; + submitBtn.innerHTML = 'Logging in...'; + } try { this.showToast('Logging in...', 'info'); @@ -155,6 +163,11 @@ class ClimaAI { this.checkSubscription(); } catch (error) { this.showToast(error.message || 'Login failed', 'error'); + } finally { + if (submitBtn) { + submitBtn.disabled = false; + submitBtn.innerHTML = originalContent; + } } } @@ -163,6 +176,14 @@ class ClimaAI { const name = document.getElementById('registerName').value; const email = document.getElementById('registerEmail').value; const password = document.getElementById('registerPassword').value; + const submitBtn = e.submitter; + let originalContent = ''; + + if (submitBtn) { + originalContent = submitBtn.innerHTML; + submitBtn.disabled = true; + submitBtn.innerHTML = 'Creating account...'; + } try { this.showToast('Creating account...', 'info'); @@ -174,6 +195,11 @@ class ClimaAI { this.checkSubscription(); } catch (error) { this.showToast(error.message || 'Registration failed', 'error'); + } finally { + if (submitBtn) { + submitBtn.disabled = false; + submitBtn.innerHTML = originalContent; + } } }