Skip to content

Commit af89a28

Browse files
committed
sync(bfmono): fix(simulator-ui): add tab button border suppression (+19 more) (bfmono@59a2b99fd)
This PR is an automated gambitmono sync of bfmono Gambit packages. - Source: `packages/gambit/` - Core: `packages/gambit-core/` - bfmono rev: 59a2b99fd Changes: - 59a2b99fd fix(simulator-ui): add tab button border suppression - b7df84782 fix(simulator-ui): open sessions in test from docs - 6e199ea1e chore(simulator-ui): remove unused styles - 84274be56 feat(gambit): refine simulator nav and surface model params - 342bb4d8d feat(gambit): publish rc prereleases in CI - 8bae64370 feat(gambit): harden session persistence logs - 9739cf8ce feat(simulator-ui): add scrolling text for overflow labels - 0aca193f8 feat(gambit): add deck tools tab in simulator - 9959c81f0 feat(simulator-ui): add guidance to test bot start overlay - ecbe06eb7 fix(simulator-ui): align focus ring with squircle inputs - 336e4c0eb fix(simulator): allow assistant start without test decks - a7bcc23be fix(simulator-ui): stabilize test bot session navigation - 2ffc619a2 test(simulator-ui): add e2e session url checklist - c9a52f844 feat(simulator-ui): polish sessions drawer list - 7a2b83a3b feat(gambit-bot): add recipe selection test personas - 27ceb80b1 fix(gambit-bot): require valid gambit deck output - 9184563de fix(gambit): block symlink escapes in bot tools - af4ec9485 feat(simulator-ui): add squircle corner scaling - bb5695ad5 docs(reorg): prune legacy archives - 7fcb795ca docs: drop aibff CLI mention Do not edit this repo directly; make changes in bfmono and re-run the sync.
1 parent 4a1475d commit af89a28

4 files changed

Lines changed: 34 additions & 4 deletions

File tree

simulator-ui/src/TestBotPage.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1332,6 +1332,7 @@ export default function TestBotPage(props: {
13321332
<div className="panel-tabs" style={{ marginTop: 6 }}>
13331333
<Button
13341334
className="flex-1"
1335+
tab
13351336
variant={assistantDeckTab === "input"
13361337
? "primary-deemph"
13371338
: "ghost"}
@@ -1342,6 +1343,7 @@ export default function TestBotPage(props: {
13421343
</Button>
13431344
<Button
13441345
className="flex-1"
1346+
tab
13451347
variant={assistantDeckTab === "tools"
13461348
? "primary-deemph"
13471349
: "ghost"}
@@ -1352,6 +1354,7 @@ export default function TestBotPage(props: {
13521354
</Button>
13531355
<Button
13541356
className="flex-1"
1357+
tab
13551358
variant={assistantDeckTab === "schema"
13561359
? "primary-deemph"
13571360
: "ghost"}

simulator-ui/src/gds/Button.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ type ButtonProps =
1515
variant?: ButtonVariant;
1616
size?: ButtonSize;
1717
href?: string;
18+
tab?: boolean;
1819
}
1920
& React.ButtonHTMLAttributes<HTMLButtonElement>
2021
& React.AnchorHTMLAttributes<HTMLAnchorElement>;
@@ -26,13 +27,15 @@ export default function Button(props: ButtonProps) {
2627
className,
2728
type,
2829
href,
30+
tab = false,
2931
...rest
3032
} = props;
3133

3234
const classes = classNames(
3335
"gds-button",
3436
`gds-button--${variant}`,
3537
`gds-button--size-${size}`,
38+
tab && "gds-button--tab",
3639
className,
3740
);
3841

simulator-ui/src/main.tsx

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1123,7 +1123,7 @@ function App() {
11231123
: "/grade";
11241124
const handleSelectSession = useCallback(
11251125
(sessionId: string) => {
1126-
const nextPath = currentPage === "test"
1126+
const nextPath = currentPage === "test" || currentPage === "docs"
11271127
? `${SESSIONS_BASE_PATH}/${encodeURIComponent(sessionId)}/test`
11281128
: currentPage === "grade"
11291129
? buildGradePath(sessionId)
@@ -1203,27 +1203,31 @@ function App() {
12031203
</div>
12041204
<div className="top-nav-buttons">
12051205
<Button
1206-
variant={currentPage === "docs" ? "primary-deemph" : "secondary"}
1206+
tab
1207+
variant={currentPage === "docs" ? "primary-deemph" : "ghost"}
12071208
onClick={() => navigate(DOCS_PATH)}
12081209
data-testid="nav-docs"
12091210
>
12101211
Docs
12111212
</Button>
12121213
<Button
1213-
variant={currentPage === "test" ? "primary-deemph" : "secondary"}
1214+
tab
1215+
variant={currentPage === "test" ? "primary-deemph" : "ghost"}
12141216
onClick={() => navigate(testBotPath)}
12151217
data-testid="nav-test"
12161218
>
12171219
Test
12181220
</Button>
12191221
<Button
1220-
variant={currentPage === "grade" ? "primary-deemph" : "secondary"}
1222+
tab
1223+
variant={currentPage === "grade" ? "primary-deemph" : "ghost"}
12211224
onClick={() => navigate(gradePath)}
12221225
data-testid="nav-grade"
12231226
>
12241227
Grade
12251228
</Button>
12261229
<Button
1230+
tab
12271231
variant={currentPage === "debug" ? "primary-deemph" : "ghost"}
12281232
onClick={() => navigate(debugPath)}
12291233
data-testid="nav-debug"

simulator-ui/src/styles.ts

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1559,6 +1559,26 @@ code:not(pre *) {
15591559
justify-content: center;
15601560
gap: 8px
15611561
}
1562+
.gds-button--tab {
1563+
border-color: transparent;
1564+
}
1565+
.gds-button--tab:hover {
1566+
border-color: transparent;
1567+
}
1568+
.gds-button--tab.gds-button--primary,
1569+
.gds-button--tab.gds-button--primary:hover,
1570+
.gds-button--tab.gds-button--primary-deemph,
1571+
.gds-button--tab.gds-button--primary-deemph:hover,
1572+
.gds-button--tab.gds-button--secondary,
1573+
.gds-button--tab.gds-button--secondary:hover,
1574+
.gds-button--tab.gds-button--ghost,
1575+
.gds-button--tab.gds-button--ghost:hover,
1576+
.gds-button--tab.gds-button--ghost-danger,
1577+
.gds-button--tab.gds-button--ghost-danger:hover,
1578+
.gds-button--tab.gds-button--danger,
1579+
.gds-button--tab.gds-button--danger:hover {
1580+
border-color: transparent;
1581+
}
15621582
.gds-button--size-medium {
15631583
padding: 8px 14px;
15641584
}

0 commit comments

Comments
 (0)