chore: remove unused imghdr import and dead CSS rules, fix attachment uploader DOM structure, add space after icon spans in templates, and increase test stderr capture limit

devplacepy/static/css/attachments.css

@@ -23,10 +23,6 @@
     background: var(--accent-light);
 }
 
-.attachment-upload-container button.attachment-upload-btn input[type="file"] {
-    display: none;
-}
-
 .attachment-upload-container button.attachment-upload-btn.uploading {
     pointer-events: none;
     opacity: 0.6;

devplacepy/static/css/base.css

@@ -667,9 +667,6 @@ img {
     padding-top: 0;
 }
 
-.page-messages .messages-layout {
-    margin-top: calc(var(--nav-height) + 1rem);
-}
 
 .modal-overlay {
     display: none;

devplacepy/static/js/Application.js

@@ -439,6 +439,8 @@ class Application {
             if (parent) {
                 const actions = parent.querySelector(".comment-form-actions");
                 if (actions) {
+                    actions.style.position = "relative";
+                    actions.appendChild(picker.wrapper);
                     actions.insertBefore(btn, actions.firstChild);
                 }
             }

devplacepy/static/js/AttachmentUploader.js

@@ -21,8 +21,8 @@ class AttachmentUploader {
         const input = document.createElement("input");
         input.type = "file";
         input.multiple = true;
+        input.style.display = "none";
         input.addEventListener("change", () => this.handleFiles(input.files));
-        this.button.appendChild(input);
 
         this.button.addEventListener("click", (e) => {
             e.preventDefault();
@@ -48,6 +48,7 @@ class AttachmentUploader {
         this.errorEl.className = "attachment-upload-error";
 
         this.container.appendChild(this.button);
+        this.container.appendChild(input);
 
         this.container.after(this.errorEl);
         this.container.after(this.previewList);