feat: demote discussion speaker hints

This commit is contained in:
lda
2026-07-08 08:10:11 +07:00 Verified
parent 35a7b5cf2e
commit 9364712dd9
3 changed files with 20 additions and 10 deletions
@@ -71,6 +71,14 @@ describe("DiscussionPanel", () => {
expect(screen.getByText(/Answer directly first/i)).toBeDefined();
});
it("renders speaker hints as presenter notes instead of answer content", () => {
render(<DiscussionPanel branchId="where-is-ai-agent" onClose={onClose} />);
const note = screen.getByLabelText("presenter note");
expect(note).toHaveTextContent(/Answer directly first/i);
expect(note).toHaveTextContent(/Presenter note/i);
});
it("shows mcp-agent-scale links to Anthropic and Cloudflare", () => {
render(<DiscussionPanel branchId="mcp-agent-scale" onClose={onClose} />);
const anthropic = screen.getByText("Anthropic MCP");
@@ -71,8 +71,8 @@ export const DiscussionPanel = ({ branchId, onClose }: DiscussionPanelProps) =>
<p className="discussion-panel__expanded-answer">{branch.expandedAnswer}</p>
)}
{branch.speakerHint && (
<p className="discussion-panel__speaker-hint">
<span>Speaker hint</span>
<p className="discussion-panel__presenter-note" aria-label="presenter note">
<span>Presenter note</span>
{branch.speakerHint}
</p>
)}
@@ -563,20 +563,22 @@
color: var(--text-primary);
}
.discussion-panel__speaker-hint {
.discussion-panel__presenter-note {
margin: 0;
padding-top: 0.5rem;
border-top: 1px solid color-mix(in srgb, var(--stage-line) 70%, transparent);
padding: 0.55rem 0.65rem;
border: 1px dashed color-mix(in oklch, var(--stage-line) 70%, transparent);
border-radius: 0.55rem;
background: color-mix(in oklch, var(--stage-surface) 72%, transparent);
color: var(--text-muted);
font-size: 0.88rem;
font-size: 0.82rem;
line-height: 1.35;
}
.discussion-panel__speaker-hint span {
.discussion-panel__presenter-note span {
display: block;
margin-bottom: 0.2rem;
color: var(--accent-cyan);
font-size: 0.72rem;
font-weight: 800;
color: var(--accent-amber);
font: 750 0.64rem/1 var(--font-mono, monospace);
letter-spacing: 0.08em;
text-transform: uppercase;
}