Skip to content

feat(chat): match the edit chrome to the context menu - #715

Merged
bmc08gt merged 1 commit into
mainfrom
feat/chat-edit-mode-chrome
Sep 3, 2026
Merged

bmc08gt merged 1 commit into
mainfrom
feat/chat-edit-mode-chrome

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Sep 3, 2026 •

Copy link
Copy Markdown
Collaborator

An edit raised its blur over the chat screen alone, while the context menu that preceded it blurred the whole navigation stack. Two things followed from that: the transcript sharpened for a frame on the way into an edit, and the navigation bar never went soft at all.

The held blur now stays over the same host the menu covered, and changes only its z-order and its bottom edge:

  • Under the navigation bar, so the back button stays legible and tappable above it.
  • Stopping at the top of the composer, the one piece of chrome an edit needs sharp. The bar rises and falls with the keyboard, so the blur is re-measured against it on every layout pass.

Two pieces of chrome follow the same reading. The title and avatar fade out for the duration, leaving the bar reading as empty chrome over the blur — close to how it looks under the menu, which covers them outright. And the back button ends the edit instead of popping the chat, matching the tap-outside gesture that already dismisses it.

An edit raised its blur over the chat screen alone, while the menu that preceded
it blurred the whole navigation stack. The transcript sharpened for a frame on
the way in and the navigation bar never went soft at all.

The held blur now stays over the same host the menu covered and changes only its
z-order and its bottom edge: under the navigation bar, so the back button stays
legible and tappable, and stopping at the top of the composer, the one piece of
chrome an edit needs sharp.

The title and avatar fade out for the duration, so the bar reads as empty chrome
over the blur, and the back button ends the edit rather than popping the chat.
@bmc08gt bmc08gt self-assigned this Sep 3, 2026
@bmc08gt
bmc08gt merged commit 51cefde into main Sep 3, 2026
1 check passed
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.

1 participant