Skip to content

Commit 6053673

Browse files
committed
fix(client): restore feed-tabs CSS + MD paragraph separator
Two regressions flagged in code review: C1 — .feed-tabs/.feed-tab/.feed-tab.active rules were accidentally deleted alongside the selection-styling cleanup in 170b455. The HEAD markup in ActivityFeedHeader.tsx still emits those classes for the All/By Agent toggle, so the tabs were rendering unstyled. Restored the rules from 62e4b50 as-is. C2 — The ReactMarkdown `p` renderer emitted a bare fragment, which caused multi-paragraph messages ("foo\n\nbar") to render as "foobar" with no separator. Now emits a trailing space so adjacent paragraphs collapse via white-space:normal into one space between them, preserving the inline flow.
1 parent 11b435f commit 6053673

2 files changed

Lines changed: 29 additions & 3 deletions

File tree

client/src/components/ActivityFeed.css

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,31 @@
8080

8181
.feed-spacer { flex: 1; }
8282

83+
.feed-tabs {
84+
display: flex;
85+
gap: 2px;
86+
background: rgba(0, 0, 0, 0.25);
87+
border-radius: 4px;
88+
padding: 2px;
89+
}
90+
.feed-tab {
91+
padding: 3px 10px;
92+
font-size: 12px;
93+
color: var(--feed-pergamena);
94+
opacity: 0.6;
95+
border: none;
96+
background: transparent;
97+
border-radius: 3px;
98+
cursor: pointer;
99+
font-family: inherit;
100+
}
101+
.feed-tab.active {
102+
background: rgba(196, 163, 90, 0.3);
103+
color: #fff;
104+
opacity: 1;
105+
font-weight: bold;
106+
}
107+
83108
.feed-fold-buttons { display: flex; gap: 4px; }
84109
.feed-fold-btn {
85110
width: 26px; height: 24px;

client/src/components/ActivityRow.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,11 @@ import { isError, isPath, resolvePath, categorizeEntry } from './activityFeedUti
66
import type { ActionFilter } from './activityFeedUtils';
77

88
// Message cells render detail as markdown (bold, italic, code, links). The
9-
// custom `p` renderer emits a fragment so multiple paragraphs flow inline —
10-
// no extra vertical space, no visible paragraph breaks.
9+
// custom `p` renderer emits the children followed by a trailing space so
10+
// multiple paragraphs flow inline with a visible word-break between them —
11+
// no extra vertical space, no paragraphs glued together ("fooBar").
1112
const MD_COMPONENTS = {
12-
p: ({ children }: { children?: ReactNode }) => <>{children}</>,
13+
p: ({ children }: { children?: ReactNode }) => <>{children}{' '}</>,
1314
};
1415

1516
interface ActivityRowProps {

0 commit comments

Comments
 (0)