Skip to content

Commit 3b8de1a

Browse files
fix(web): dim only the row content that has no disabled state of its own
The wrapper-level dim stacked with each control's own: the rename field and remove button composited to roughly 0.41 alpha while every other disabled control in the desktop-only block sits at 0.64. Only the built-in row's name and badge lack a disabled treatment, so the dim belongs there. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 7a118b4 commit 3b8de1a

1 file changed

Lines changed: 12 additions & 8 deletions

File tree

apps/web/src/components/settings/IntegrationsSettings.tsx

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -544,19 +544,23 @@ function BrowserProfilesSetting({ disabled }: { readonly disabled: boolean }) {
544544
</Button>
545545
}
546546
>
547-
{/*
548-
Dimmed as a whole when the section is unavailable. The built-in rows
549-
are a plain span and a badge rather than `h3`/`p` or disabled controls,
550-
so the block's own dimming does not reach them and they would be the
551-
only full-contrast content inside "only available in the desktop app".
552-
*/}
553-
<div className={cn("mt-1 space-y-1", disabled && "opacity-64")}>
547+
<div className="mt-1 space-y-1">
554548
{resolveBrowserProfiles(userProfiles).map((profile) => {
555549
const builtIn = isBuiltInBrowserProfileId(profile.id);
556550
return (
557551
<div key={profile.id} className={ITEM_ROW_INNER_CLASSNAME}>
558552
{builtIn ? (
559-
<span className="flex min-w-0 items-center gap-2 text-sm text-foreground">
553+
// Dimmed here rather than on the list, which is the only
554+
// content in the row without a disabled treatment of its own:
555+
// a wrapper-level dim would stack with the rename field's and
556+
// the remove button's, landing them near 0.41 while every
557+
// other disabled control in the block sits at 0.64.
558+
<span
559+
className={cn(
560+
"flex min-w-0 items-center gap-2 text-sm text-foreground",
561+
disabled && "opacity-64",
562+
)}
563+
>
560564
{profile.name}
561565
<Badge variant="outline">
562566
{profile.kind === "incognito" ? "Ephemeral" : "Built-in"}

0 commit comments

Comments
 (0)