Skip to content

Commit ea2efcb

Browse files
docs(web): fold the stale profile-setting comments into one
Renaming the component left three consecutive doc blocks above it, two of them describing controls that no longer exist separately. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent a923ab0 commit ea2efcb

1 file changed

Lines changed: 8 additions & 25 deletions

File tree

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

Lines changed: 8 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -42,15 +42,6 @@ import { usePrimaryEnvironment } from "~/state/environments";
4242
import { isElectron } from "../../env";
4343

4444
import { Badge } from "../ui/badge";
45-
import {
46-
AlertDialog,
47-
AlertDialogClose,
48-
AlertDialogDescription,
49-
AlertDialogFooter,
50-
AlertDialogHeader,
51-
AlertDialogPopup,
52-
AlertDialogTitle,
53-
} from "../ui/alert-dialog";
5445
import {
5546
Menu,
5647
MenuGroup,
@@ -513,36 +504,28 @@ function DesktopOnlyBrowserDefaults({ children }: { readonly children: ReactNode
513504
);
514505
}
515506

516-
/**
517-
* Create, rename, and remove browser profiles.
518-
*
519-
* Built-ins render without controls: they are synthesized rather than stored,
520-
* so there is nothing to rename and removing them would strand every tab that
521-
* opened under them.
522-
*/
523-
524-
/**
525-
* Per-profile cookie import from a browser installed on this machine.
526-
*
527-
* Sources are listed lazily on open: detection touches the other browser's
528-
* files, and the answer changes while the app is running (quitting the browser
529-
* clears `browserRunning`), so a value cached at mount would go stale.
530-
*/
531507
/**
532508
* Profile list, its header menu, and the import flow.
533509
*
534510
* One menu creates profiles and imports into them, because the two are the
535511
* same decision from the user's side: "I want a profile that has my Helium
536512
* logins in it". Import targets include "New profile" so that case does not
537513
* require creating one first and then finding a second control.
514+
*
515+
* Built-ins render without a rename field: they are synthesized rather than
516+
* stored, so there is nothing to rename and removing them would strand every
517+
* tab that opened under them.
518+
*
519+
* Sources are listed lazily on open: detection touches the other browser's
520+
* files, and the answer changes while the app is running (quitting the browser
521+
* clears `browserRunning`), so a value cached at mount would go stale.
538522
*/
539523
function BrowserProfilesSetting({ disabled }: { readonly disabled: boolean }) {
540524
const userProfiles = useClientSettings((settings) => settings.browserProfiles);
541525
const defaultProfileId = useClientSettings((settings) => settings.browserDefaultProfileId);
542526
const updateSettings = useUpdatePrimarySettings();
543527
const environmentId = usePrimaryEnvironment()?.environmentId;
544528
const [sources, setSources] = useState<ReadonlyArray<BrowserImportSource> | null>(null);
545-
const [profilePendingRemoval, setProfilePendingRemoval] = useState<BrowserProfile | null>(null);
546529
const [busy, setBusy] = useState(false);
547530
const [profilePendingRemoval, setProfilePendingRemoval] = useState<BrowserProfile | null>(null);
548531

0 commit comments

Comments
 (0)