From 08035e3447594232a21803c98929a63a16dd1ce7 Mon Sep 17 00:00:00 2001
From: Zichao Lin <earthjasonlin@163.com>
Date: Sun, 5 Jul 2026 20:19:47 +0800
Subject: [PATCH] feat(ui): arrival time only required when status is
 delivered, not sent

---
 app/templates/postcard_form.html | 45 ++++++++++++++++++--------------
 1 file changed, 26 insertions(+), 19 deletions(-)

diff --git a/app/templates/postcard_form.html b/app/templates/postcard_form.html
index 55f00c2..03f296c 100644
--- a/app/templates/postcard_form.html
+++ b/app/templates/postcard_form.html
@@ -17,20 +17,20 @@
 
     <div id="fields-sent-delivered">
         <label>收件人 *</label>
-        <input type="text" name="recipient_name" value="{{ postcard.recipient_name if postcard else '' }}" required data-group="sent-delivered">
+        <input type="text" name="recipient_name" value="{{ postcard.recipient_name if postcard else '' }}" required>
         <label>国家/地区 *</label>
-        <input type="text" name="country" value="{{ postcard.country if postcard else '' }}" required maxlength="2" placeholder="JP" style="width:80px;text-transform:uppercase" data-group="sent-delivered">
-        <label>寄出时间</label>
-        <input type="date" name="send_time" value="{{ postcard.send_time.strftime('%Y-%m-%d') if postcard and postcard.send_time else '' }}" data-group="sent-delivered">
-        <label>到达时间</label>
-        <input type="date" name="arrival_time" value="{{ postcard.arrival_time.strftime('%Y-%m-%d') if postcard and postcard.arrival_time else '' }}" data-group="sent-delivered">
+        <input type="text" name="country" value="{{ postcard.country if postcard else '' }}" required maxlength="2" placeholder="JP" style="width:80px;text-transform:uppercase">
+        <label>寄出时间 *</label>
+        <input type="date" name="send_time" value="{{ postcard.send_time.strftime('%Y-%m-%d') if postcard and postcard.send_time else '' }}" required>
+        <label id="label-arrival">到达时间</label>
+        <input type="date" name="arrival_time" id="input-arrival" value="{{ postcard.arrival_time.strftime('%Y-%m-%d') if postcard and postcard.arrival_time else '' }}">
     </div>
 
     <div id="fields-received" style="display:none">
         <label>发件人 *</label>
-        <input type="text" name="sender_name" value="{{ postcard.sender_name if postcard else '' }}" required data-group="received">
-        <label>收到时间</label>
-        <input type="date" name="receive_time" value="{{ postcard.receive_time.strftime('%Y-%m-%d') if postcard and postcard.receive_time else '' }}" data-group="received">
+        <input type="text" name="sender_name" value="{{ postcard.sender_name if postcard else '' }}" required>
+        <label>收到时间 *</label>
+        <input type="date" name="receive_time" value="{{ postcard.receive_time.strftime('%Y-%m-%d') if postcard and postcard.receive_time else '' }}" required>
     </div>
 
     <div class="form-actions">
@@ -43,16 +43,23 @@ function toggleFields() {
     var s = document.getElementById('status-select').value;
     var sd = document.getElementById('fields-sent-delivered');
     var rc = document.getElementById('fields-received');
-    var isSent = (s !== 'received');
-    sd.style.display = isSent ? '' : 'none';
-    rc.style.display = isSent ? 'none' : '';
-    // toggle required on hidden fields
-    sd.querySelectorAll('[data-group="sent-delivered"]').forEach(function(el) {
-        el.required = isSent;
-    });
-    rc.querySelectorAll('[data-group="received"]').forEach(function(el) {
-        el.required = !isSent;
-    });
+    var arrival = document.getElementById('input-arrival');
+    var labelArrival = document.getElementById('label-arrival');
+    if (s === 'received') {
+        sd.style.display = 'none';
+        rc.style.display = '';
+        arrival.required = false;
+    } else {
+        sd.style.display = '';
+        rc.style.display = 'none';
+        if (s === 'delivered') {
+            arrival.required = true;
+            labelArrival.textContent = '到达时间 *';
+        } else {
+            arrival.required = false;
+            labelArrival.textContent = '到达时间';
+        }
+    }
 }
 toggleFields();
 </script>