Skip to content

fix(ui): move the fast mode indicator to settings - #879

Merged
alphaxiv-bot[bot] merged 2 commits into
alphaXiv:mainfrom
exhyy:fix/fast-mode-settings-indicator
Nov 4, 2026
Merged

alphaxiv-bot[bot] merged 2 commits into
alphaXiv:mainfrom
exhyy:fix/fast-mode-settings-indicator

Conversation

@exhyy

@exhyy exhyy commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Closes #878

Summary

When Fast Mode is enabled, its lightning indicator appears on the model picker even though the toggle lives in the adjacent settings menu. Move the indicator and accessible Fast Mode description to the settings trigger, so users can follow the indicator to the control. Keep the harness logo on the model picker.

Fast enabled:
Before: [⚡ GPT-6.1 Sol ▾] [XHigh ▾]
After:  [OpenAI logo · GPT-6.1 Sol ▾] [⚡ XHigh ▾]

This is a presentation-only change in ModelPicker.tsx using existing localized messages and styling.

Test plan

Verified on commit e4e7fab:

  • pnpm build (includes localization checks, typecheck, and production build).
  • pnpm lint:styles.
  • pnpm test: all 590 existing UI tests passed.
  • git diff --check.
  • Browser verification in the isolated node dev.ts environment: GPT-6.1 Sol / XHigh; enabling Fast shows the lightning icon on settings, disabling it removes the icon, and the model logo remains.
  • Native macOS development app on macOS 15.8.2 / arm64, launched with node dev.ts desktop using isolated data: repeated Fast on/off, settings-trigger indicator, retained model logo, and model-menu opening verified. Accessibility inspection confirms the Fast Mode description belongs to the settings trigger and disappears when disabled.
  • Windows/Linux native application verification.

Native verification used the development app with the Vite-served UI, not a packaged release build. No model request was submitted; validation covers the UI behavior.

Assisted-by: gpt-6.1-sol

Keep the harness logo on the model picker and show the lightning indicator on the settings trigger that contains the Fast Mode toggle. Move the accessible fast-mode description to the same trigger.

Assisted-by: gpt-6.1-sol
@greptile-apps

greptile-apps Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge.

Summary

Moves the Fast Mode lightning icon and screen-reader label to the settings button in ModelPicker.tsx.

  • The settings control shows the Fast Mode cue when priority is active.

No actionable issues found.

Reviews (1) · Last reviewed commit: "fix(ui): move the fast mode indicator to..." · Reviewed by Greptile

@paperclover paperclover added the auto-merge Queue for automatic merge once all checks pass label Nov 4, 2026
@paperclover

Copy link
Copy Markdown
Contributor

thank you, perfect first contribution

@alphaxiv-bot
alphaxiv-bot Bot merged commit c40e96e into alphaXiv:main Nov 4, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Queue for automatic merge once all checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fast Mode indicator points to the model menu instead of its settings toggle

2 participants