Fix adding aircraft in new routes

va-net · Dec 16, 2024 · dcacb06 · dcacb06
1 parent bf69dc2
commit dcacb06
Showing 1 changed file with 16 additions and 12 deletions.
diff --git a/themes/tailwind/views/admin/routes_new.php b/themes/tailwind/views/admin/routes_new.php
@@ -37,26 +37,30 @@
                 </div>
                 <div class="space-y-1">
                     <label for="notes">Notes</label>
-                    <input id="notes" name="notes" type="text" class="form-control" value="<?= Page::$pageData->route->notes ?>" />
+                    <input id="notes" name="notes" type="text" class="form-control" />
                 </div>
                 <input type="hidden" name="aircraft" :value="aircraft.map(a => a.id).join(',')" />
             </div>
             <div class="flex-1 p-3 dark:bg-white/10 bg-gray-100 rounded">
                 <h2 class="text-2xl font-bold mb-2">Route Aircraft</h2>
-                <ul class="list-disc mb-3 ml-5">
+                <ul class="mb-3 space-y-0.5">
                     <template x-for="a in aircraft" :key="a.id">
-                        <li x-text="`${a.name} (${a.liveryname})`"></li>
+                        <li class="flex items-center group">
+                            <span class="flex-1" x-text="`${a.name} (${a.liveryname})`"></span>
+                            <span class="invisible group-hover:visible flex-none text-gray-400 cursor-pointer" title="Remove Aircraft" @click="aircraft = aircraft.filter(ac => ac.id != a.id)">
+                                <svg xmlns="http://www.w3.org/2000/svg" class="h-5 w-5" viewBox="0 0 20 20" fill="currentColor">
+                                    <path fill-rule="evenodd" d="M4.293 4.293a1 1 0 011.414 0L10 8.586l4.293-4.293a1 1 0 111.414 1.414L11.414 10l4.293 4.293a1 1 0 01-1.414 1.414L10 11.414l-4.293 4.293a1 1 0 01-1.414-1.414L8.586 10 4.293 5.707a1 1 0 010-1.414z" clip-rule="evenodd" />
+                                </svg>
+                            </span>
+                        </li>
                     </template>
                 </ul>
-                <div class="space-y-1">
-                    <label for="newaircraft">Add Aircraft</label>
-                    <select id="newaircraft" x-ref="newaircraft" class="form-control" @change.prevent="if ($event.target.value) { aircraft.push(allaircraft.find(a => a.ifliveryid == $event.target.value)); $refs.newaircraft.value = ''; }">
-                        <option value>Select</option>
-                        <template x-for="a in allaircraft.filter(ac => !aircraft.includes(ac))">
-                            <option :value="a.ifliveryid" x-text="`${a.name} (${a.liveryname})${a.notes ? ' - ' + a.notes : ''}`"></option>
-                        </template>
-                    </select>
-                </div>
+                <select class="form-control" @change.prevent="if ($event.target.value) { aircraft.push(allaircraft.find(a => a.ifliveryid == $event.target.value)); $el.value = ''; }">
+                    <option value>Add Aircraft</option>
+                    <template x-for="a in allaircraft.filter(ac => !aircraft.some(a => a.id == ac.id))">
+                        <option :value="a.ifliveryid" x-text="`${a.name} (${a.liveryname})${a.notes ? ' - ' + a.notes : ''}`"></option>
+                    </template>
+                </select>
             </div>
         </div>
         <button type="submit" :disabled="aircraft.length < 1" class="button-primary">