Skip to content

Commit 54c31cf

Browse files
committed
Merge remote-tracking branch 'refs/remotes/origin/main' into dohooo/team-cloud-sandbox
2 parents 03dbe6c + b23e5d9 commit 54c31cf

2 files changed

Lines changed: 14 additions & 3 deletions

File tree

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"helmor": patch
3+
---
4+
5+
Restore the accent colors on the composer's mode controls — the animated Extra High / Max effort label, the Plan toggle, and the Terminal-Mode toggle.

src/features/composer/index.tsx

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1251,7 +1251,13 @@ export const WorkspaceComposer = memo(function WorkspaceComposer({
12511251
disabled={toolbarDisabled}
12521252
className={cn(
12531253
`flex items-center gap-0.5 ${composerToolbarTriggerClassName}`,
1254-
composerToolbarActiveClassName,
1254+
// `effort-max-text` paints via `-webkit-text-fill-color:
1255+
// transparent` without a `color`, so it needs a muted
1256+
// baseline to avoid a flash when the gradient class drops.
1257+
"text-muted-foreground hover:text-muted-foreground",
1258+
(effectiveEffort === "max" ||
1259+
effectiveEffort === "xhigh") &&
1260+
"effort-max-text",
12551261
toolbarDisabled
12561262
? "cursor-not-allowed opacity-45 hover:bg-transparent"
12571263
: null,
@@ -1305,7 +1311,7 @@ export const WorkspaceComposer = memo(function WorkspaceComposer({
13051311
className={cn(
13061312
`size-7 justify-center px-0 ${composerToolbarTriggerClassName}`,
13071313
permissionMode === "plan"
1308-
? composerToolbarActiveClassName
1314+
? "text-plan hover:text-plan"
13091315
: "text-muted-foreground/70 hover:text-muted-foreground/70",
13101316
)}
13111317
onToggle={() =>
@@ -1326,7 +1332,7 @@ export const WorkspaceComposer = memo(function WorkspaceComposer({
13261332
className={cn(
13271333
`size-7 justify-center px-0 ${composerToolbarTriggerClassName}`,
13281334
terminalMode
1329-
? composerToolbarActiveClassName
1335+
? "text-emerald-500 hover:bg-emerald-500/10 hover:text-emerald-500"
13301336
: // Pin the hover text color (like the Plan toggle) so the
13311337
// toolbar base `hover:text-foreground` can't flash the icon
13321338
// white for a frame while `transition-colors` runs on toggle.

0 commit comments

Comments
 (0)