Skip to content

apprt: show title override in command palette jump commands#10458

Open
bernsno wants to merge 3 commits intoghostty-org:mainfrom
bernsno:command-palette-title-override
Open

apprt: show title override in command palette jump commands#10458
bernsno wants to merge 3 commits intoghostty-org:mainfrom
bernsno:command-palette-title-override

Conversation

@bernsno
Copy link
Contributor

@bernsno bernsno commented Jan 27, 2026

Relates to #3709

When a surface has a title override set (via prompt_surface_title), the command palette's "Focus:" jump commands now display the override instead of the terminal title.

No title override:
Screenshot 2026-01-26 at 6 11 56 PM

With title override
Screenshot 2026-01-26 at 6 11 30 PM

AI DISCLAIMER

I leveraged Claude Code to understand the codebase, make a plan and write the first draft of the code. I reviewed and edited the code written by claude and manually tested the change on iOS.

@bernsno bernsno requested review from a team as code owners January 27, 2026 02:23
Copy link
Member

@bo2themax bo2themax left a comment

Choose a reason for hiding this comment

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

Small suggestion for macOS

…swift

Co-authored-by: Lukas <134181853+bo2themax@users.noreply.github.com>
@bernsno bernsno requested a review from bo2themax January 27, 2026 15:01
Copy link
Member

@bo2themax bo2themax left a comment

Choose a reason for hiding this comment

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

macOS part is good

Copy link
Member

@pluiedev pluiedev left a comment

Choose a reason for hiding this comment

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

Ack GTK changes. Sorry for the delay!

@pluiedev
Copy link
Member

Looks like you got some compile errors on macOS - need to fix that before we can hit merge

@bernsno
Copy link
Contributor Author

bernsno commented Feb 14, 2026

@pluiedev looks like a merge got a little borked. pushed a fix. appreciate the review!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments