Skip to content

Hide queued prompts panel while an inline menu is open#11848

Merged
harryalbert merged 1 commit into
masterfrom
oz/hide-queued-prompts-when-inline-menu-open
May 29, 2026
Merged

Hide queued prompts panel while an inline menu is open#11848
harryalbert merged 1 commit into
masterfrom
oz/hide-queued-prompts-when-inline-menu-open

Conversation

@harryalbert
Copy link
Copy Markdown
Contributor

@harryalbert harryalbert commented May 28, 2026

Description

@peterrajani pointed out that it looks kind of funky when both an inline menu and queued prompts menu are open at the same time. When an inline menu is open we can just auto-hide the queued prompts menu.

Addresses APP-4623

Testing

  • I have manually tested my changes locally with ./script/run

Demo

https://www.loom.com/share/0035bef74b29400eb33f7bbdebf4a98e

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

Conversation: https://staging.warp.dev/conversation/7f318ff5-49ae-48d3-b6bc-0d03af4020b3
Run: https://oz.staging.warp.dev/runs/019e6fe9-33a7-7d27-be90-e51cd07f68f9

This PR was generated with Oz.

@cla-bot cla-bot Bot added the cla-signed label May 28, 2026
@harryalbert harryalbert force-pushed the oz/hide-queued-prompts-when-inline-menu-open branch from 979e024 to 54ee355 Compare May 28, 2026 19:36
The queued prompts panel and the inline input menus (slash commands,
model/profile selector, prompts, conversation, skill, user query,
rewind, history, repos, plan) were both rendered into the same agent
input column, so they could appear simultaneously.

Guard the panel's render in `render_agent_input` with
`!suggestions_mode_model.is_inline_menu_open()` so the panel is not
rendered at all (rather than collapsed) whenever any inline menu is
open.

Co-Authored-By: Oz <oz-agent@warp.dev>
Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@harryalbert harryalbert force-pushed the oz/hide-queued-prompts-when-inline-menu-open branch from 54ee355 to c705495 Compare May 28, 2026 19:37
@harryalbert harryalbert requested a review from zachbai May 28, 2026 19:46
@harryalbert harryalbert marked this pull request as ready for review May 28, 2026 19:46
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 28, 2026

@harryalbert

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This PR passes the input suggestions-mode model to QueuedPromptsPanelView and hides the queued prompts panel while an inline menu is open. I reviewed the changed render path, model subscriptions, queue event handling, and security surface.

Concerns

  • No blocking correctness, security, or spec-alignment concerns found in the changed lines.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@harryalbert harryalbert requested a review from advait-m May 29, 2026 18:35
@harryalbert harryalbert merged commit eadc05e into master May 29, 2026
58 of 60 checks passed
@harryalbert harryalbert deleted the oz/hide-queued-prompts-when-inline-menu-open branch May 29, 2026 19:57
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.

2 participants