diff --git a/apps/dotcom/client/e2e/fixtures/Sidebar.ts b/apps/dotcom/client/e2e/fixtures/Sidebar.ts index 2d23d35406b2..a5e294b6688c 100644 --- a/apps/dotcom/client/e2e/fixtures/Sidebar.ts +++ b/apps/dotcom/client/e2e/fixtures/Sidebar.ts @@ -24,7 +24,7 @@ export class Sidebar { this.helpMenu = this.page.getByTestId('tla-sidebar-help-menu-trigger') this.themeButton = this.page.getByTestId('dialog-sub.help menu color-scheme-button') this.darkModeButton = this.page.getByText('Dark') - this.signOutButton = this.page.getByText('Sign out') + this.signOutButton = this.page.getByTestId('dialog.sign-out') } async isVisible() { @@ -62,6 +62,8 @@ export class Sidebar { async openUserSettingsMenu() { await this.userSettingsMenu.hover() await this.userSettingsMenu.click() + // Wait for the dropdown content to mount before child-item clicks race the open animation. + await expect(this.page.getByRole('menu')).toBeVisible() } @step diff --git a/apps/dotcom/client/src/main.tsx b/apps/dotcom/client/src/main.tsx index 850a6f239e81..2afdfd1a5962 100644 --- a/apps/dotcom/client/src/main.tsx +++ b/apps/dotcom/client/src/main.tsx @@ -44,7 +44,6 @@ createRoot(document.getElementById('root')!).render( ) showConsoleBranding() - try { // we have a dummy service worker that unregisters itself immediately // this was needed to remove the service worker we used to have from the cache diff --git a/apps/dotcom/client/src/tla/components/TlaEditor/TlaEditorTopLeftPanel.tsx b/apps/dotcom/client/src/tla/components/TlaEditor/TlaEditorTopLeftPanel.tsx index 8cc6ea316431..328c6e454e34 100644 --- a/apps/dotcom/client/src/tla/components/TlaEditor/TlaEditorTopLeftPanel.tsx +++ b/apps/dotcom/client/src/tla/components/TlaEditor/TlaEditorTopLeftPanel.tsx @@ -44,6 +44,7 @@ import { TLAppUiEventSource, useTldrawAppUiEvents } from '../../utils/app-ui-eve import { getIsCoarsePointer } from '../../utils/getIsCoarsePointer' import { defineMessages, useIntl, useMsg } from '../../utils/i18n' import { TlaSignInDialog } from '../dialogs/TlaSignInDialog' +import { ExternalLink } from '../ExternalLink/ExternalLink' import { CookieConsentMenuItem, GiveUsFeedbackMenuItem, @@ -106,15 +107,14 @@ export function TlaEditorTopLeftPanelAnonymous() { return ( <> - - + {anonFileName && ( <> + - + ) } diff --git a/apps/dotcom/client/src/tla/components/TlaSidebar/sidebar.module.css b/apps/dotcom/client/src/tla/components/TlaSidebar/sidebar.module.css index 5cb70f8daa1a..367a20fc5ef0 100644 --- a/apps/dotcom/client/src/tla/components/TlaSidebar/sidebar.module.css +++ b/apps/dotcom/client/src/tla/components/TlaSidebar/sidebar.module.css @@ -236,7 +236,7 @@ flex: 0 0 auto; } -/* Workspace link (sort of) */ +/* Workspace link */ .sidebarWorkspaceButton { background: none;