From 1be294b6c818f53906dc8ceba5873959b3d7bc1e Mon Sep 17 00:00:00 2001
From: Suyeon Woo <134579071+seaneez@users.noreply.github.com>
Date: Tue, 28 May 2024 19:33:39 +0900
Subject: [PATCH] fix(VsTabs): remove scrobar-width css (#184)

---
 .../src/components/vs-tabs/VsTabs.scss        | 76 ++++++++++---------
 .../vlossom/src/components/vs-tabs/VsTabs.vue | 38 +++++-----
 2 files changed, 60 insertions(+), 54 deletions(-)

diff --git a/packages/vlossom/src/components/vs-tabs/VsTabs.scss b/packages/vlossom/src/components/vs-tabs/VsTabs.scss
index 760bfb7b1..d35bd8daf 100644
--- a/packages/vlossom/src/components/vs-tabs/VsTabs.scss
+++ b/packages/vlossom/src/components/vs-tabs/VsTabs.scss
@@ -6,52 +6,56 @@ $fontWeight: var(--vs-tabs-fontWeight, 400);
     overflow-x: hidden;
     user-select: none;
 
-    ul {
-        align-items: center;
-        display: inline-flex;
-        list-style: none;
+    .tabs-container {
         overflow-x: auto;
-        scrollbar-width: thin;
+        padding-bottom: 0.2rem;
 
-        &.bottomLine {
-            border-bottom: 1px solid var(--vs-tabs-borderBottomColor, var(--vs-comp-border-color));
-        }
-
-        .tab {
+        ul {
             align-items: center;
-            background: transparent;
-            color: var(--vs-tabs-fontColor, var(--vs-comp-color));
-            cursor: pointer;
             display: inline-flex;
-            font-size: var(--vs-tabs-fontSize, 1rem);
-            font-weight: $fontWeight;
-            height: var(--vs-tabs-height, auto);
-            justify-content: center;
-            padding: var(--vs-tabs-padding, 0.2rem 1rem);
-            text-align: center;
-            transition: color 0.1s;
-            width: var(--vs-tabs-tabWidth, auto);
-            min-width: fit-content;
+            list-style: none;
+            flex-wrap: nowrap;
 
-            &:not(:last-child) {
-                margin-right: var(--vs-tabs-gap, 0.3rem);
+            &.bottomLine {
+                border-bottom: 1px solid var(--vs-tabs-borderBottomColor, var(--vs-comp-border-color));
             }
 
-            &:hover {
-                background-color: var(--vs-tabs-backgroundColor, var(--vs-light-backgroundColor));
-            }
+            .tab {
+                align-items: center;
+                background: transparent;
+                color: var(--vs-tabs-fontColor, var(--vs-comp-color));
+                cursor: pointer;
+                display: inline-flex;
+                font-size: var(--vs-tabs-fontSize, 1rem);
+                font-weight: $fontWeight;
+                height: var(--vs-tabs-height, auto);
+                justify-content: center;
+                padding: var(--vs-tabs-padding, 0.2rem 1rem);
+                text-align: center;
+                transition: color 0.1s;
+                width: var(--vs-tabs-tabWidth, auto);
+                min-width: fit-content;
 
-            &:focus {
-                outline: none;
-            }
+                &:not(:last-child) {
+                    margin-right: var(--vs-tabs-gap, 0.3rem);
+                }
 
-            &.primary {
-                border-bottom: 2px solid var(--vs-tabs-backgroundColor, var(--vs-comp-backgroundColor-primary));
-                font-weight: calc($fontWeight + 200);
-            }
+                &:hover {
+                    background-color: var(--vs-tabs-backgroundColor, var(--vs-light-backgroundColor));
+                }
+
+                &:focus {
+                    outline: none;
+                }
+
+                &.primary {
+                    border-bottom: 2px solid var(--vs-tabs-backgroundColor, var(--vs-comp-backgroundColor-primary));
+                    font-weight: calc($fontWeight + 200);
+                }
 
-            &.disabled {
-                @include disabled;
+                &.disabled {
+                    @include disabled;
+                }
             }
         }
     }
diff --git a/packages/vlossom/src/components/vs-tabs/VsTabs.vue b/packages/vlossom/src/components/vs-tabs/VsTabs.vue
index 9645b62f8..54d1ac6b2 100644
--- a/packages/vlossom/src/components/vs-tabs/VsTabs.vue
+++ b/packages/vlossom/src/components/vs-tabs/VsTabs.vue
@@ -11,24 +11,26 @@
             >
                 <vs-icon icon="goPrev" size="1.2rem" />
             </button>
-            <ul role="tablist" ref="tabsContainerRef" :class="{ bottomLine }">
-                <li
-                    v-for="(tab, index) in tabs"
-                    ref="tabRefs"
-                    :key="tab"
-                    :class="['tab', { primary: isSelected(index), disabled: isDisabled(index) }]"
-                    role="tab"
-                    :aria-selected="isSelected(index)"
-                    :aria-disabled="isDisabled(index)"
-                    :tabindex="isSelected(index) ? 0 : -1"
-                    @click.stop="selectTab(index)"
-                    @keydown.stop="handleKeydown"
-                >
-                    <slot :name="tab" :index="index">
-                        {{ tab }}
-                    </slot>
-                </li>
-            </ul>
+            <div class="tabs-container" ref="tabsContainerRef">
+                <ul role="tablist" :class="{ bottomLine }">
+                    <li
+                        v-for="(tab, index) in tabs"
+                        ref="tabRefs"
+                        :key="tab"
+                        :class="['tab', { primary: isSelected(index), disabled: isDisabled(index) }]"
+                        role="tab"
+                        :aria-selected="isSelected(index)"
+                        :aria-disabled="isDisabled(index)"
+                        :tabindex="isSelected(index) ? 0 : -1"
+                        @click.stop="selectTab(index)"
+                        @keydown.stop="handleKeydown"
+                    >
+                        <slot :name="tab" :index="index">
+                            {{ tab }}
+                        </slot>
+                    </li>
+                </ul>
+            </div>
             <button
                 v-if="showScrollButtons"
                 type="button"