swarm-ui: don't break at a narrow viewport
Closes #3447. - Shell.css: .shell-header and .shell-nav wrap onto a second line below a breakpoint instead of forcing horizontal overflow — no hamburger/collapse menu, per mara's own 'don't break, don't over-invest' scoping. - LinksMenu.css: bumped .links-menu-button and .links-menu-item to the same 2.75em (~44px, WCAG 2.5.5) touch-target floor the shared ui/ kit's form controls and buttons already carry — this trigger is bespoke, not built from that kit, but the floor applies regardless. - Table.tsx/.css: wrapped the table in a .ui-table-scroll container (overflow-x: auto) so a wide table scrolls within its own box on a narrow viewport instead of forcing the whole page to overflow horizontally — not named in the issue's two concrete spots, but the same 'should not break' failure mode, and fixing it at the shared Table primitive covers every future caller. .create-agent-input's fixed width -> max-width was already fixed as part of #3448's form kit. Verified: tsc clean, build succeeds, screenshotted at 320px (hives nav wraps, table scrolls internally, page itself doesn't overflow) and 480px (create-agent form fields fit).
This commit is contained in:
parent
5e6f26f209
commit
7c15cd46d2
4 changed files with 45 additions and 19 deletions
|
|
@ -1,6 +1,10 @@
|
|||
/* <LinksMenu> — a single header icon-button + popover. Kept visually
|
||||
quiet (no border/fill until interacted with) so it reads as chrome,
|
||||
not another nav item. */
|
||||
not another nav item. Button is sized to the same touch-target floor
|
||||
as the shared `ui/` kit (2.75em ≈ 44px, WCAG 2.5.5) even though it
|
||||
isn't built from that kit — a bespoke icon trigger, not a form
|
||||
control, but the floor applies regardless of which component drew
|
||||
it. */
|
||||
.links-menu {
|
||||
position: relative;
|
||||
margin-left: auto;
|
||||
|
|
@ -9,8 +13,8 @@
|
|||
display: flex;
|
||||
align-items: center;
|
||||
justify-content: center;
|
||||
width: 2em;
|
||||
height: 2em;
|
||||
width: 2.75em;
|
||||
height: 2.75em;
|
||||
padding: 0;
|
||||
border: 1px solid transparent;
|
||||
border-radius: 0.4em;
|
||||
|
|
@ -43,6 +47,7 @@
|
|||
display: flex;
|
||||
align-items: center;
|
||||
gap: 0.5em;
|
||||
min-height: 2.75em;
|
||||
padding: 0.4em 0.6em;
|
||||
border-radius: 0.35em;
|
||||
color: var(--fg);
|
||||
|
|
|
|||
|
|
@ -2,10 +2,17 @@
|
|||
renders into. Colour vars come from ../theme.css (same base16 slots
|
||||
the per-hive dashboard uses, so this reads as recognizably
|
||||
hyperhive) — the layout itself is this package's own, not
|
||||
@hive/shared's chrome.css. */
|
||||
@hive/shared's chrome.css.
|
||||
|
||||
`flex-wrap` on the header and nav rather than a hamburger/collapse
|
||||
menu: a narrow viewport (phone or a tiled desktop window) wraps onto
|
||||
a second line instead of overflowing — "don't break, don't
|
||||
over-invest" is the explicit scope here, not full mobile navigation
|
||||
redesign. */
|
||||
.shell-header {
|
||||
display: flex;
|
||||
align-items: center;
|
||||
flex-wrap: wrap;
|
||||
gap: 1.5em;
|
||||
padding: 0.75em 1.25em;
|
||||
border-bottom: 1px solid var(--border);
|
||||
|
|
@ -18,6 +25,7 @@
|
|||
}
|
||||
.shell-nav {
|
||||
display: flex;
|
||||
flex-wrap: wrap;
|
||||
gap: 1em;
|
||||
}
|
||||
.shell-nav-link {
|
||||
|
|
|
|||
|
|
@ -1,3 +1,8 @@
|
|||
/* Scrolls horizontally within its own box on a narrow viewport instead
|
||||
of forcing the whole page to overflow — see Table.tsx's comment. */
|
||||
.ui-table-scroll {
|
||||
overflow-x: auto;
|
||||
}
|
||||
.ui-table {
|
||||
width: 100%;
|
||||
border-collapse: collapse;
|
||||
|
|
|
|||
|
|
@ -3,6 +3,12 @@
|
|||
// their own cell rendering rather than this component knowing about
|
||||
// any particular row shape, so it stays reusable for the swarm-wide
|
||||
// agent roster later without a rewrite.
|
||||
//
|
||||
// Wrapped in a scrollable container rather than letting a wide table
|
||||
// force the whole page to overflow horizontally on a narrow viewport —
|
||||
// a `<table>` doesn't shrink below its content's natural width on its
|
||||
// own, so without this the page itself would break, not just look
|
||||
// cramped.
|
||||
import type { ComponentChildren } from 'preact';
|
||||
import './Table.css';
|
||||
|
||||
|
|
@ -22,23 +28,25 @@ export function Table<T>({
|
|||
rowKey: (row: T) => string;
|
||||
}) {
|
||||
return (
|
||||
<table class="ui-table">
|
||||
<thead>
|
||||
<tr>
|
||||
{columns.map((c) => (
|
||||
<th key={c.key}>{c.header}</th>
|
||||
))}
|
||||
</tr>
|
||||
</thead>
|
||||
<tbody>
|
||||
{rows.map((row) => (
|
||||
<tr key={rowKey(row)}>
|
||||
<div class="ui-table-scroll">
|
||||
<table class="ui-table">
|
||||
<thead>
|
||||
<tr>
|
||||
{columns.map((c) => (
|
||||
<td key={c.key}>{c.render(row)}</td>
|
||||
<th key={c.key}>{c.header}</th>
|
||||
))}
|
||||
</tr>
|
||||
))}
|
||||
</tbody>
|
||||
</table>
|
||||
</thead>
|
||||
<tbody>
|
||||
{rows.map((row) => (
|
||||
<tr key={rowKey(row)}>
|
||||
{columns.map((c) => (
|
||||
<td key={c.key}>{c.render(row)}</td>
|
||||
))}
|
||||
</tr>
|
||||
))}
|
||||
</tbody>
|
||||
</table>
|
||||
</div>
|
||||
);
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in a new issue