Merge branch 'ui-improvements-backup' into 'main'

UI improvements: collapsible schedule menu and fix double extensions

See merge request engineering/unturf/upload.unturf.com!9

CLAUDE.md

@@ -0,0 +1,18 @@
+# Project Instructions for Claude
+
+## Commit Message Format
+
+Never include Claude attribution in commit messages. Always end commit messages with:
+
+russell@unturf.com is the boss
+
+## Example Commit Message
+
+```
+Fix download filename double extensions
+
+- Prevent .zip.zip and similar double extensions
+- Check if title already contains extension before appending
+
+russell@unturf.com is the boss
+```

app.py

@@ -1816,9 +1816,13 @@ def view_media_view(request):
     if media.title:
         file_extension = os.path.splitext(media.filename)[1]
         raw_title = media.title
-        download_filename = sanitize_filename_for_http_header(
-            f"{raw_title}{file_extension}"
-        )
+        # Check if title already ends with the file extension to avoid doubling
+        if file_extension and not raw_title.lower().endswith(file_extension.lower()):
+            download_filename = sanitize_filename_for_http_header(
+                f"{raw_title}{file_extension}"
+            )
+        else:
+            download_filename = sanitize_filename_for_http_header(raw_title)
     else:
         download_filename = sanitize_filename_for_http_header(media.filename)
 

templates/upload_media.html.j2

@@ -30,20 +30,22 @@
         </label>
     </fieldset><br>
 
-    <fieldset>
-        <legend>Scheduled Actions (Optional)</legend>
-        <label for="scheduled_public">Schedule to become Public:</label>
-        <input type="datetime-local" name="scheduled_public"><br><br>
+    <details>
+        <summary>Scheduled Actions (Optional)</summary>
+        <fieldset>
+            <label for="scheduled_public">Schedule to become Public:</label>
+            <input type="datetime-local" name="scheduled_public"><br><br>
 
-        <label for="scheduled_private">Schedule to become Private:</label>
-        <input type="datetime-local" name="scheduled_private"><br><br>
+            <label for="scheduled_private">Schedule to become Private:</label>
+            <input type="datetime-local" name="scheduled_private"><br><br>
 
-        <label for="scheduled_unlisted">Schedule to become Unlisted:</label>
-        <input type="datetime-local" name="scheduled_unlisted"><br><br>
+            <label for="scheduled_unlisted">Schedule to become Unlisted:</label>
+            <input type="datetime-local" name="scheduled_unlisted"><br><br>
 
-        <label for="scheduled_delete">Schedule for Deletion:</label>
-        <input type="datetime-local" name="scheduled_delete"><br><br>
-    </fieldset>
+            <label for="scheduled_delete">Schedule for Deletion:</label>
+            <input type="datetime-local" name="scheduled_delete"><br><br>
+        </fieldset>
+    </details>
 
     <button type="submit">Upload</button>
 </form>