diff --git a/TODO b/TODO
index 46494f1..f674c8c 100644
--- a/TODO
+++ b/TODO
@@ -1,7 +1,8 @@
 ## GENERAL
     * move all unsorted photos/* -> import/
         TEST: moving over the top of same dir name (what does this do to FS content)
-        FIX: BUG-69
+            -- I think the AddDir()'s should be finding existing ones, but maybe are adding?
+            FIX: BUG-69
         TEST: what if we try to move a path in settings, should not allow this?
 
     * when we do a legitimate move of a file, update last_scanned_dt otherwise FS change is newer, and it will 're-scan'
@@ -11,8 +12,6 @@
 
     * put a delete option on viewer page
 
-    * close button on invalid password should look like danger/alert/close for jobs
-
     * metadata at folder level with file level to add more richness
 
     * why .xcf is seen as a video???
diff --git a/templates/login.html b/templates/login.html
index 8eefb3c..9f7eaa7 100644
--- a/templates/login.html
+++ b/templates/login.html
@@ -27,7 +27,10 @@
 <div class="container">
 {% if form.errors|length > 0 %}
     <div class="row my-5">
-        <alert class="alert alert-danger alert-dismissible fade show">
+        <alert id="err" class="alert alert-danger alert-dismissible fade show">
+            <button type="button" class="close btn border-secondary me-3" data-dismiss="alert" aria-label="Close" onClick="$('#err').hide()">
+                <span aria-hidden="true">&times;</span>
+            </button>
             {% set last_err = namespace(txt="") %}
             {% for e in form.errors %}
                 {% if last_err.txt != form.errors[e] %}
@@ -37,9 +40,6 @@
                     {% set last_err.txt=form.errors[e] %}
                 {% endif %}
             {% endfor %}
-            <button type="button" class="close" data-dismiss="alert" aria-label="Close">
-    <span aria-hidden="true">&times;</span>
-  </button>
         </alert>
     </div class="row">
 {% endif %}