Add copy affordances, collapsible long content, and follow-the-tail s… - #488
Merged
Merged
Conversation
…crolling Chat and log panes: - Copy button on each content log entry; timestamp moves to its own row beneath the agent name and the button. - Copy button on fenced code blocks in agent messages. The button is emitted by a marked renderer override so it survives streaming re-renders, and the code is read back out of the DOM on click so the clipboard matches the screen and no message content passes through generated markup. Off by default elsewhere via a new `copyableCode` prop. - Both the message thread and the log panes now follow new content whenever they are already at the bottom, instead of only after the jump button was pressed. Scrolling back down resumes following on its own. Collapsible long content: - New shared CollapsibleText component: clips past ~10 measured lines, with a fade mask and click-anywhere or keyboard toggling. - Applied to the chat bubbles (held off while a reply streams), and it now backs the content log rows and the conversation page's dialog entries, replacing three separate hand-rolled implementations. Removes the dialog page's hardcoded-white '...' indicator, which was wrong in dark mode, and the double clamp it needed to work around Markdown's inner BFC. Class naming: - Rename the agent detail page's `ad-` prefix to `agd-`; ad blockers ship element-hiding rules that match a leading `ad-` and were hiding the page's layout. `cta-popup` renamed to `cta-overlay` as a precaution. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
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
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.
…crolling
Chat and log panes:
copyableCodeprop.Collapsible long content:
Class naming:
ad-prefix toagd-; ad blockers ship element-hiding rules that match a leadingad-and were hiding the page's layout.cta-popuprenamed tocta-overlayas a precaution.