From 9dca44283a460cdf1ab9e187fe16eb78f2b1c74e Mon Sep 17 00:00:00 2001
From: Neko Ayaka <neko@ayaka.moe>
Date: Tue, 31 Mar 2026 15:54:06 +0800
Subject: [PATCH] Revert "fix(stage-ui): keep onboarding save button visible
 (#1280)"

This reverts commit 660765b1aafc6da3ff0e338b73a8b106be422ecb.
---
 .../onboarding/step-model-selection.vue       | 91 +++++++------------
 1 file changed, 34 insertions(+), 57 deletions(-)

diff --git a/packages/stage-ui/src/components/scenarios/dialogs/onboarding/step-model-selection.vue b/packages/stage-ui/src/components/scenarios/dialogs/onboarding/step-model-selection.vue
index ee299ff15..96c1b4242 100644
--- a/packages/stage-ui/src/components/scenarios/dialogs/onboarding/step-model-selection.vue
+++ b/packages/stage-ui/src/components/scenarios/dialogs/onboarding/step-model-selection.vue
@@ -3,7 +3,6 @@ import type { OnboardingStepNextHandler, OnboardingStepPrevHandler } from './typ
 
 import { Button } from '@proj-airi/ui'
 import { storeToRefs } from 'pinia'
-import { computed } from 'vue'
 import { useI18n } from 'vue-i18n'
 
 import Alert from '../../../misc/alert.vue'
@@ -25,22 +24,10 @@ const {
   isLoadingActiveProviderModels,
   activeProviderModelError,
 } = storeToRefs(consciousnessStore)
-
-const sortedProviderModels = computed(() => {
-  return providerModels.value.toSorted((a, b) => {
-    if (a.id === activeModel.value)
-      return -1
-
-    if (b.id === activeModel.value)
-      return 1
-
-    return a.id.localeCompare(b.id)
-  })
-})
 </script>
 
 <template>
-  <div h-full min-h-0 flex flex-col gap-4>
+  <div h-full flex flex-col gap-4>
     <div sticky top-0 z-100 flex flex-shrink-0 items-center gap-2>
       <button outline-none @click="props.onPrevious">
         <div i-solar:alt-arrow-left-line-duotone h-5 w-5 />
@@ -52,7 +39,7 @@ const sortedProviderModels = computed(() => {
     </div>
 
     <!-- Using the new RadioCardManySelect component -->
-    <div min-h-0 flex flex-1 flex-col gap-4>
+    <div flex flex-1 flex-col gap-4>
       <Alert
         v-if="providerModels.length === 0 && !isLoadingActiveProviderModels"
         type="error"
@@ -67,51 +54,41 @@ const sortedProviderModels = computed(() => {
         </template>
       </Alert>
 
-      <div min-h-0 flex-1>
-        <RadioCardManySelect
-          v-model="activeModel"
-          v-model:search-query="modelSearchQuery"
-          :items="sortedProviderModels"
-          :searchable="true"
-          :allow-custom="true"
-          :search-placeholder="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.search_placeholder')"
-          :search-no-results-title="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.no_search_results')"
-          :search-no-results-description="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.no_search_results_description', { query: modelSearchQuery })"
-          :search-results-text="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.search_results', { count: '{count}', total: '{total}' })"
-          :custom-input-placeholder="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.custom_model_placeholder')"
-          :expand-button-text="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.expand')"
-          :collapse-button-text="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.collapse')"
-          list-class="max-h-[calc(100dvh-20rem)] sm:max-h-100 overflow-y-auto"
-        />
+      <RadioCardManySelect
+        v-model="activeModel"
+        v-model:search-query="modelSearchQuery"
+        :items="providerModels.toSorted((a, b) => a.id === activeModel ? -1 : b.id === activeModel ? 1 : 0)"
+        :searchable="true"
+        :allow-custom="true"
+        :search-placeholder="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.search_placeholder')"
+        :search-no-results-title="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.no_search_results')"
+        :search-no-results-description="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.no_search_results_description', { query: modelSearchQuery })"
+        :search-results-text="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.search_results', { count: '{count}', total: '{total}' })"
+        :custom-input-placeholder="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.custom_model_placeholder')"
+        :expand-button-text="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.expand')"
+        :collapse-button-text="t('settings.pages.modules.consciousness.sections.section.provider-model-selection.collapse')"
+        list-class="max-h-[calc(100dvh-17rem)] sm:max-h-120 overflow-y-auto"
+      />
 
-        <Alert v-if="activeProviderModelError" type="error">
-          <template #title>
-            {{ t('settings.dialogs.onboarding.validationFailed') }}
-          </template>
-          <template #content>
-            <div class="whitespace-pre-wrap break-all">
-              {{ activeProviderModelError }}
-            </div>
-          </template>
-        </Alert>
-      </div>
+      <Alert v-if="activeProviderModelError" type="error">
+        <template #title>
+          {{ t('settings.dialogs.onboarding.validationFailed') }}
+        </template>
+        <template #content>
+          <div class="whitespace-pre-wrap break-all">
+            {{ activeProviderModelError }}
+          </div>
+        </template>
+      </Alert>
     </div>
 
     <!-- Action Buttons -->
-    <div
-      :class="[
-        'sticky bottom-0 z-20 -mx-3 px-3 pt-3 pb-2',
-        'bg-gradient-to-t from-white via-white to-white/85',
-        'dark:from-[#0f0f0f] dark:via-[#0f0f0f] dark:to-[#0f0f0f]/85',
-      ]"
-    >
-      <Button
-        variant="primary"
-        :disabled="!activeModel"
-        :loading="isLoadingActiveProviderModels"
-        :label="t('settings.dialogs.onboarding.saveAndContinue')"
-        @click="props.onNext"
-      />
-    </div>
+    <Button
+      variant="primary"
+      :disabled="!activeModel"
+      :loading="isLoadingActiveProviderModels"
+      :label="t('settings.dialogs.onboarding.saveAndContinue')"
+      @click="props.onNext"
+    />
   </div>
 </template>