Conversation
When viewing a note that has an associated chat session, show a MessageSquare button in the dialog header that dismisses the note and opens the corresponding session dialog. When viewing a chat session that has an associated note, show a FileText button in the dialog header that dismisses the session and opens the corresponding note dialog. Both buttons only appear when the association exists. Wired up in BranchCard (timeline notes ↔ branch sessions) and ProjectSection (project notes ↔ project sessions).
Update NoteModal .modal dimensions from 640px/max-width:90vw/max-height:80vh to 700px/height:80vh/max-height:900px to match SessionModal's layout, ensuring both dialogs have a consistent size when opened.
Add visible text labels alongside the icons for the cross-navigation buttons between note and session dialogs: - NoteModal: MessageSquare icon now shows 'Chat session' label - SessionModal: FileText icon now shows 'Note' label Update header-btn styles in both modals with gap, horizontal padding, and font-size to accommodate the text labels.
Replace icon-only cross-navigation buttons with text-only bordered
buttons in NoteModal ("View chat") and SessionModal ("View note").
- Remove MessageSquare and FileText icon imports
- Add 1px solid border using --border-primary with hover transition
- Use text labels instead of icons for clearer affordance
--border-primary doesn't exist in the theme. Use --border-muted which is defined in theme.ts and resolves to a visible border color.
NoteModal header was taller than SessionModal due to differences in padding, font size, and markup structure: - Reduce header padding from 16px 20px to 12px 16px - Change title font-size from --size-base to --size-sm - Replace <h2 class="modal-title"> with <span class="header-title"> wrapped in <div class="header-content"> to match SessionModal's markup structure Both modal headers now render at the same height.
Move the copy button before the view chat button in the header actions so the order is now: Copy → View chat → Close.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add buttons to navigate between related note and session modals: