Skip to content

ui: fix removed reasoning menu in single model mode on desktop - #27985

Open
benbenbenbenbenben wants to merge 2 commits into
ggml-org:masterfrom
benbenbenbenbenben:fix/restore-reasoning-submenu-desktop
Open

ui: fix removed reasoning menu in single model mode on desktop#27985
benbenbenbenbenben wants to merge 2 commits into
ggml-org:masterfrom
benbenbenbenbenben:fix/restore-reasoning-submenu-desktop

Conversation

@benbenbenbenbenben

@benbenbenbenbenben benbenbenbenbenben commented Aug 29, 2026

Copy link
Copy Markdown

Overview

This PR restores the missing reasoning menu in the web UI on large/desktop layouts when running in single model mode. The regression appears to have been introduced in #27744 and fixes #27981

Additional information

N/A

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: Qwen 3.8 27B was used to find the offending commit, I accepted a single line change suggestion and ran all git and gh operations manually.

@allozaur allozaur left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@benbenbenbenbenben please run npm run format in tools/ui and update the failing Storybook test

@ServeurpersoCom

Copy link
Copy Markdown
Contributor

Thanks for the fix. Could you also test it in router mode? The models selector already mounts the reasoning submenu there, so this needs to not end up showing it twice before it can be merged. Also you need to launch "npm run format" before commit.

@Remowylliams

Copy link
Copy Markdown

Very looking forward to this PR making it through. Reasoning menu was super useful.

@benbenbenbenbenben

Copy link
Copy Markdown
Author

Thanks for the fix. Could you also test it in router mode? The models selector already mounts the reasoning submenu there, so this needs to not end up showing it twice before it can be merged. Also you need to launch "npm run format" before commit.

I've tested it. In router mode on large screens the reasoning menu is in the correct location and I confirmed it doesn't duplicated when the screen is narrow.

Model selector on large screens:

image

Model selector on small screens:

image

Reasoning in "+" menu on small screens:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

llama-ui : unable to open reasoning level selection menu on desktop

4 participants