From b2b1fd3ba1a908824969c8787cac95b9d628e50d Mon Sep 17 00:00:00 2001
From: mpge <matthew@matthewpg.com>
Date: Wed, 13 May 2026 07:19:49 -0400
Subject: [PATCH] fix(skills): default SkillTagManager buttons to type="button"

The remove-chip, add-toggle, and option-list buttons sit inside Admin/Skills/Form.vue's <form>. Without an explicit type they default to type="submit", so clicking any of them would submit the skill form before the user finished editing routing tags or departments.
---
 src/components/SkillTagManager.vue | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/components/SkillTagManager.vue b/src/components/SkillTagManager.vue
index 344b236..0e7ba2a 100644
--- a/src/components/SkillTagManager.vue
+++ b/src/components/SkillTagManager.vue
@@ -46,6 +46,7 @@ function removeItem(value) {
             >
                 {{ item[itemLabelKey] }}
                 <button
+                    type="button"
                     :aria-label="`Remove ${item[itemLabelKey]}`"
                     class="ml-0.5 text-cyan-400 hover:text-cyan-200"
                     @click="removeItem(item[itemValueKey])"
@@ -60,6 +61,7 @@ function removeItem(value) {
         <div class="relative">
             <button
                 v-if="unselectedItems.length"
+                type="button"
                 class="rounded-lg border border-dashed border-white/[0.08] px-2 py-1 text-xs text-neutral-400 transition-colors hover:border-white/[0.15] hover:text-neutral-300"
                 @click="showDropdown = !showDropdown"
             >
@@ -72,6 +74,7 @@ function removeItem(value) {
                 <button
                     v-for="item in unselectedItems"
                     :key="item[itemValueKey]"
+                    type="button"
                     class="w-full px-3 py-1.5 text-left text-xs text-neutral-300 transition-colors hover:bg-white/[0.04]"
                     @click="addItem(item[itemValueKey])"
                 >