vinzenz/hyperhive
vinzenz/hyperhive
Watch
0
0
Fork
You've already forked hyperhive
 0
Code Activity

Compare commits

base: vinzenz:3e4ffa9312e190095aeb580e89f60212c478fa03
Branches Tags
vinzenz:main
...
compare: vinzenz:302778573dc416f9e4678070d3f1be8f27a87023
Branches Tags
vinzenz:main
Author SHA1 Message Date
iris
302778573d agent: HeaderPill wraps Badge instead of a hand-styled pill 
Badge already covers icon+label+value+onClick -- exactly HeaderPill's
shape. mara, reviewing the first cut (a bespoke .header-pill matched to
Badge's own CSS values): "cant we reuse the badge component". HeaderPill
now renders <Badge> directly and only owns hiding at count 0 + the
inbox/todos tone (amber/green on the count, same as before). Drops the
now-unused .header-pill* CSS from agent.css entirely.
 2026-08-29 10:12:19 +02:00
iris
ad58894c25 agent: fill the inbox/todos header pill to match the rest of the cluster 
HeaderPill used the bordered/transparent .hive-pill shape while every
other chip in .agent-header-pills (StatusChips, MetaNav's trigger) is
@hive/shared's filled Badge (.ui-badge) -- the one visibly hollow pill
in a row of filled ones. Restyled .header-pill with .ui-badge's own
values (filled --purple-dim background, no border, matching radius/
padding/hover) instead of a new shape, and dropped hive-pill from the
markup since it no longer describes what this component looks like.
 2026-08-29 10:12:19 +02:00
2 changed files with 28 additions and 53 deletions
Download patch file Download diff file Expand all files Collapse all files

45
frontend/packages/agent/src/agent.css
View file

@ -319,47 +319,10 @@ h2, h3 {
background: color-mix(in srgb, var(--purple) 6%, transparent); background: color-mix(in srgb, var(--purple) 6%, transparent);
} }
/* Header pill — inbox / loose-ends triggers. Compact, count-prominent. /* Header pill (inbox / loose-ends triggers) — no CSS of its own any
Markup carries `hive-pill` alongside `header-pill` (shared pill.css more. `HeaderPill.tsx` renders `@hive/shared`'s `Badge` directly now
supplies border/radius/padding/font-size/letter-spacing; `display` (mara: "cant we reuse the badge component") instead of a hand-styled
is the one thing overridden here, to `inline-flex`, for the icon+ pill matched to Badge's own values; nothing left to override here. */
label+count row layout — see pill.css's own note on that). */
.header-pill {
background: transparent;
border-color: var(--purple-dim);
color: var(--fg);
font-family: inherit;
display: inline-flex;
align-items: center;
gap: 0.4em;
cursor: pointer;
transition: border-color 0.15s ease, box-shadow 0.15s ease, color 0.15s ease;
}
.header-pill:hover {
border-color: var(--purple);
color: var(--purple);
box-shadow: 0 0 10px -2px var(--purple);
}
.header-pill-icon { font-size: 1.05em; line-height: 1; }
.header-pill-label { color: var(--muted); }
.header-pill-count {
background: var(--purple-dim);
color: var(--purple);
border-radius: 999px;
padding: 0 0.5em;
min-width: 1.6em;
text-align: center;
font-weight: bold;
font-variant-numeric: tabular-nums;
}
.header-pill-inbox .header-pill-count {
background: color-mix(in srgb, var(--amber) 18%, transparent);
color: var(--amber);
}
.header-pill-todos .header-pill-count {
background: color-mix(in srgb, var(--green) 18%, transparent);
color: var(--green);
}
.agent-main { .agent-main {
position: absolute; position: absolute;

36
frontend/packages/agent/src/components/HeaderPill.tsx
View file

@ -1,9 +1,15 @@
// <HeaderPill> — the inbox/todos flyout triggers in the header's right // <HeaderPill> — the inbox/todos flyout triggers in the header's right
// cluster. Reuses agent.css's existing `.hive-pill.header-pill…` rules // cluster. A thin wrapper over `@hive/shared`'s `Badge` (icon+label+
// (loaded globally) — same "layout is a like-for-like port, the // value+onClick already covers this shape exactly) rather than a
// component model is the change" approach as Header.tsx. Hidden // bespoke pill — mara, reviewing the first cut of this (a hand-styled
// (renders nothing) at count 0, matching the old page's `pill.hidden = // `.header-pill` matched to `Badge`'s own CSS values rather than the
// count === 0`. // component itself): "cant we reuse the badge component". Only two
// things this wrapper still owns: hiding at count 0 (`pill.hidden =
// count === 0` in the old page) and picking the inbox/todos tone so
// the count reads amber/green like it always has (`Badge`'s `tone`
// colours the value, which here is the count).
import { Badge, type BadgeTone } from '@hive/shared/badge.js';
export interface HeaderPillProps { export interface HeaderPillProps {
kind: 'inbox' | 'todos'; kind: 'inbox' | 'todos';
icon: string; icon: string;
@ -12,15 +18,21 @@ export interface HeaderPillProps {
onClick: () => void; onClick: () => void;
} }
const TONE: Record<HeaderPillProps['kind'], BadgeTone> = {
inbox: 'warning',
todos: 'positive',
};
export function HeaderPill({ kind, icon, label, count, onClick }: HeaderPillProps) { export function HeaderPill({ kind, icon, label, count, onClick }: HeaderPillProps) {
if (count === 0) return null; if (count === 0) return null;
return ( return (
<button type="button" class={`hive-pill header-pill header-pill-${kind}`} onClick={onClick} title={`open ${label} flyout`}> <Badge
<span class="header-pill-icon" aria-hidden="true"> icon={icon}
{icon} label={label}
</span> value={count}
<span class="header-pill-label">{label}</span> tone={TONE[kind]}
<span class="header-pill-count">{count}</span> onClick={onClick}
</button> title={`open ${label} flyout`}
/>
); );
} }