From bd68354e70c800f96e2ed44e1f67affd8642282d Mon Sep 17 00:00:00 2001
From: Felix Ruf <felix.ruf@aoe.com>
Date: Fri, 2 Aug 2024 14:38:30 +0200
Subject: [PATCH] fixed broken button coloring

---
 src/Resources/src/components/balance/BalanceHeader.vue | 2 +-
 src/Resources/src/components/guest/GuestForm.vue       | 2 +-
 src/Resources/src/components/misc/CreateButton.vue     | 2 +-
 src/Resources/src/components/misc/SubmitButton.vue     | 2 +-
 src/Resources/tests/unit/misc/CreateButton.spec.ts     | 3 ---
 src/Resources/tests/unit/misc/SubmitButton.spec.ts     | 3 ---
 6 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/src/Resources/src/components/balance/BalanceHeader.vue b/src/Resources/src/components/balance/BalanceHeader.vue
index 4dba2ef82..5b5bd83cd 100644
--- a/src/Resources/src/components/balance/BalanceHeader.vue
+++ b/src/Resources/src/components/balance/BalanceHeader.vue
@@ -12,7 +12,7 @@
       >
         <template #button>
           <button
-            class="hover:bg-highlight-2 btn-highlight-shadow mx-2 mb-6 mt-4 h-9 items-center rounded-btn bg-highlight px-[34px] text-center text-btn font-bold text-white shadow-btn drop-shadow-btn transition-all duration-300 ease-out active:translate-y-0.5 active:shadow-btn-active"
+            class="hover:bg-highlight-2 btn-highlight-shadow mx-2 mb-6 mt-4 h-9 items-center rounded-btn bg-highlight px-[34px] text-center text-btn font-bold text-white drop-shadow-btn transition-all duration-300 ease-out active:translate-y-0.5 active:shadow-btn-active"
           >
             <span class="align-middle leading-[10px]">+ {{ t('balance.add') }}</span>
           </button>
diff --git a/src/Resources/src/components/guest/GuestForm.vue b/src/Resources/src/components/guest/GuestForm.vue
index 73aaa7c84..42421c264 100644
--- a/src/Resources/src/components/guest/GuestForm.vue
+++ b/src/Resources/src/components/guest/GuestForm.vue
@@ -44,7 +44,7 @@
     </div>
     <div class="mt-10 text-right">
       <button
-        class="hover:bg-highlight-2 btn-highlight-shadow mx-2 mb-6 mt-4 h-9 items-center rounded-btn bg-highlight px-[34px] text-center text-btn font-bold text-white shadow-btn drop-shadow-btn transition-all duration-300 ease-out active:translate-y-0.5 active:shadow-btn-active"
+        class="hover:bg-highlight-2 btn-highlight-shadow mx-2 mb-6 mt-4 h-9 items-center rounded-btn bg-highlight px-[34px] text-center text-btn font-bold text-white drop-shadow-btn transition-all duration-300 ease-out active:translate-y-0.5 active:shadow-btn-active"
         @click="emit('submitForm')"
       >
         {{ t('guest.submit') }}
diff --git a/src/Resources/src/components/misc/CreateButton.vue b/src/Resources/src/components/misc/CreateButton.vue
index 2352fe244..cad3567f2 100644
--- a/src/Resources/src/components/misc/CreateButton.vue
+++ b/src/Resources/src/components/misc/CreateButton.vue
@@ -1,6 +1,6 @@
 <template>
   <div
-    :class="[open || pressed ? 'translate-y-0.5' : 'btn-highlight-shadow shadow-btn']"
+    :class="[open || pressed ? 'translate-y-0.5' : 'btn-highlight-shadow']"
     class="mx-2 mb-6 mt-4 flex h-9 items-center justify-center rounded-btn bg-highlight px-[34px] text-center text-btn font-bold leading-[10px] text-white drop-shadow-btn transition-all duration-300 ease-out hover:bg-[#f7a043] focus:outline-none"
     @mousedown="handleMouseDown"
     @mouseup="handleMouseUp"
diff --git a/src/Resources/src/components/misc/SubmitButton.vue b/src/Resources/src/components/misc/SubmitButton.vue
index b89383f63..c3beb4f1e 100644
--- a/src/Resources/src/components/misc/SubmitButton.vue
+++ b/src/Resources/src/components/misc/SubmitButton.vue
@@ -3,7 +3,7 @@
     type="submit"
     :value="btnText ?? t('slot.save')"
     class="col-span-6 col-start-1 mx-auto mb-6 mt-4 flex h-9 cursor-pointer items-center rounded-btn bg-highlight px-[34px] text-btn font-bold leading-[10px] text-white drop-shadow-btn transition-all duration-300 ease-out hover:bg-[#f7a043] focus:outline-none"
-    :class="[isMouseDown ? 'translate-y-0.5' : 'btn-highlight-shadow shadow-btn']"
+    :class="[isMouseDown ? 'translate-y-0.5' : 'btn-highlight-shadow']"
     @mousedown="isMouseDown = true"
     @mouseup="isMouseDown = false"
   />
diff --git a/src/Resources/tests/unit/misc/CreateButton.spec.ts b/src/Resources/tests/unit/misc/CreateButton.spec.ts
index acabfa019..8ea23e21d 100644
--- a/src/Resources/tests/unit/misc/CreateButton.spec.ts
+++ b/src/Resources/tests/unit/misc/CreateButton.spec.ts
@@ -13,7 +13,6 @@ describe('Test CreateButton', () => {
 
         expect(wrapper.find('div').text()).toEqual('TestText123');
         expect(wrapper.classes()).toContain('btn-highlight-shadow');
-        expect(wrapper.classes()).toContain('shadow-btn');
     });
 
     it('should change classes when the open prop changes', async () => {
@@ -25,13 +24,11 @@ describe('Test CreateButton', () => {
         });
 
         expect(wrapper.classes()).toContain('btn-highlight-shadow');
-        expect(wrapper.classes()).toContain('shadow-btn');
 
         await wrapper.setProps({ open: true });
 
         expect(wrapper.vm.open).toBeTruthy();
         expect(wrapper.classes()).toContain('translate-y-0.5');
         expect(wrapper.classes()).not.toContain('btn-highlight-shadow');
-        expect(wrapper.classes()).not.toContain('shadow-btn');
     });
 });
diff --git a/src/Resources/tests/unit/misc/SubmitButton.spec.ts b/src/Resources/tests/unit/misc/SubmitButton.spec.ts
index 8714ee697..a31a629db 100644
--- a/src/Resources/tests/unit/misc/SubmitButton.spec.ts
+++ b/src/Resources/tests/unit/misc/SubmitButton.spec.ts
@@ -13,18 +13,15 @@ describe('Test SubmitButton', () => {
         const wrapper = mount(SubmitButton);
 
         expect(wrapper.classes()).not.toContain('translate-y-0.5');
-        expect(wrapper.classes()).toContain('shadow-btn');
         expect(wrapper.classes()).toContain('btn-highlight-shadow');
 
         await wrapper.trigger('mousedown');
 
         expect(wrapper.classes()).toContain('translate-y-0.5');
-        expect(wrapper.classes()).not.toContain('shadow-btn');
         expect(wrapper.classes()).not.toContain('btn-highlight-shadow');
 
         await wrapper.trigger('mouseup');
         expect(wrapper.classes()).not.toContain('translate-y-0.5');
-        expect(wrapper.classes()).toContain('shadow-btn');
         expect(wrapper.classes()).toContain('btn-highlight-shadow');
     });
 });