Skip to content

Commit 57fccc1

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 e3998e1 commit 57fccc1

1 file changed

Lines changed: 8 additions & 15 deletions

File tree

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

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -504,28 +504,21 @@ function DesktopOnlyBrowserDefaults({ children }: { readonly children: ReactNode
504504
);
505505
}
506506

507-
/**
508-
* Create, rename, and remove browser profiles.
509-
*
510-
* Built-ins render without controls: they are synthesized rather than stored,
511-
* so there is nothing to rename and removing them would strand every tab that
512-
* opened under them.
513-
*/
514-
515-
/**
516-
* Per-profile cookie import from a browser installed on this machine.
517-
*
518-
* Sources are listed lazily on open: detection touches the other browser's
519-
* files, and the answer changes while the app is running (quitting the browser
520-
* clears `browserRunning`), so a value cached at mount would go stale.
521-
*/
522507
/**
523508
* Profile list, its header menu, and the import flow.
524509
*
525510
* One menu creates profiles and imports into them, because the two are the
526511
* same decision from the user's side: "I want a profile that has my Helium
527512
* logins in it". Import targets include "New profile" so that case does not
528513
* 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.
529522
*/
530523
function BrowserProfilesSetting({ disabled }: { readonly disabled: boolean }) {
531524
const userProfiles = useClientSettings((settings) => settings.browserProfiles);

0 commit comments

Comments
 (0)