Skip to content

refactor(staged): move chat info into note modal footer#755

Merged
matt2e merged 2 commits into
mainfrom
x-messages-in-chat-after-note
May 28, 2026
Merged

refactor(staged): move chat info into note modal footer#755
matt2e merged 2 commits into
mainfrom
x-messages-in-chat-after-note

Conversation

@matt2e
Copy link
Copy Markdown
Contributor

@matt2e matt2e commented May 28, 2026

Summary

Move the "N messages after note" indicator from a floating overlay on top of the note content into the note modal's footer (next-steps) section, styled as a centered capsule.

  • Replaces the absolute-positioned floating-chat-info button with a chat-info-capsule row rendered inside .next-steps.
  • Drops the has-floating-chat-info padding hack on .modal-content since the capsule no longer overlays the content.
  • Adjusts the next-steps render condition so the footer shows when either the chat info or any next-step prompt is present.

Test plan

  • Open a note with assistant messages after it and confirm the capsule appears centered in the footer above any next-step rows.
  • Open a note with no assistant messages after it and confirm the capsule is hidden.
  • Confirm clicking the capsule opens the chat session.
  • Verify layout at narrow widths (<=700px).

matt2e added 2 commits May 28, 2026 13:51
Replace the absolutely-positioned floating chat-info button inside
`.modal-body` with a row at the top of the existing `.next-steps`
footer, alongside note/commit next-step rows. Removes the shadow
and overlay padding hacks; the footer now renders whenever there's
chat info or next steps to show.

Signed-off-by: Matt Toohey <contact@matttoohey.com>
Render the "N messages after note" entry as a horizontally-centered
capsule button with the MessageCircle icon, sitting above the note
and commit next-step rows. Visually delineates the chat-info shortcut
from the action rows below it.

Signed-off-by: Matt Toohey <contact@matttoohey.com>
@matt2e matt2e requested review from baxen and wesbillman as code owners May 28, 2026 04:16
@matt2e matt2e changed the title refactor(staged): move chat-info into note modal footer refactor(staged): move chat info into note modal footer May 28, 2026
@matt2e matt2e merged commit 46da533 into main May 28, 2026
5 checks passed
@matt2e matt2e deleted the x-messages-in-chat-after-note branch May 28, 2026 04:26
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