0.69.2

* Refactor attachment downloads during export to use promises instead of streams
Date attachments in zip file correctly

* tsc

.env.sample

@@ -30,7 +30,7 @@ REDIS_URL=redis://localhost:6379
 
 # URL should point to the fully qualified, publicly accessible URL. If using a
 # proxy the port in URL and PORT may be different.
-URL=https://app.outline.dev:3000
+URL=http://localhost:3000
 PORT=3000
 
 # See [documentation](docs/SERVICES.md) on running a separate collaboration
@@ -99,7 +99,7 @@ OIDC_USERINFO_URI=
 OIDC_USERNAME_CLAIM=preferred_username
 
 # Display name for OIDC authentication
-OIDC_DISPLAY_NAME=OpenID Connect
+OIDC_DISPLAY_NAME=OpenID
 
 # Space separated auth scopes.
 OIDC_SCOPES=openid profile email
@@ -181,7 +181,3 @@ RATE_LIMITER_ENABLED=true
 # Configure default throttling parameters for rate limiter
 RATE_LIMITER_REQUESTS=1000
 RATE_LIMITER_DURATION_WINDOW=60
-
-# Iframely API config
-IFRAMELY_URL=
-IFRAMELY_API_KEY=

.eslintrc

@@ -3,7 +3,6 @@
   "parserOptions": {
     "sourceType": "module",
     "extraFileExtensions": [".json"],
-    "project": "./tsconfig.json",
     "ecmaFeatures": {
       "jsx": true
     }
@@ -37,14 +36,6 @@
       "component": true,
       "html": true
     }],
-    "@typescript-eslint/no-floating-promises": "error",
-    "@typescript-eslint/await-thenable": "error",
-    "@typescript-eslint/no-misused-promises": [
-      "error",
-      {
-        "checksVoidReturn": false
-      }
-    ],
     "@typescript-eslint/no-unused-vars": [
       "error",
       {

.github/ISSUE_TEMPLATE/config.yml

@@ -1,8 +1,8 @@
 blank_issues_enabled: false
 contact_links:
   - name: Feature request
-    url: https://github.com/outline/outline/discussions/new?category=ideas
+    url: https://github.com/outline/outline/discussions/new
     about: Request a feature to be added to the project
   - name: Self hosting questions
-    url: https://github.com/outline/outline/discussions/new?category=self-hosting
+    url: https://github.com/outline/outline/discussions/new
     about: Ask questions and discuss running Outline with community members

Dockerfile

@@ -7,10 +7,6 @@ WORKDIR $APP_PATH
 # ---
 FROM node:18-alpine AS runner
 
-RUN apk update && apk add --no-cache curl && apk add --no-cache ca-certificates
-
-LABEL org.opencontainers.image.source="https://github.com/outline/outline"
-
 ARG APP_PATH
 WORKDIR $APP_PATH
 ENV NODE_ENV production

Dockerfile.base

@@ -1,5 +1,7 @@
 ARG APP_PATH=/opt/outline
-FROM node:18-alpine AS deps
+FROM node:16.14.2-alpine3.15 AS deps
+
+RUN apk --no-cache add curl
 
 ARG APP_PATH
 WORKDIR $APP_PATH

Makefile

@@ -1,6 +1,5 @@
 up:
 	docker-compose up -d redis postgres s3
-	yarn install-local-ssl
 	yarn install --pure-lockfile
 	yarn dev:watch
 

README.md

@@ -7,7 +7,7 @@
   <img width="1640" alt="screenshot" src="https://user-images.githubusercontent.com/380914/110356468-26374600-7fef-11eb-9f6a-f2cc2c8c6590.png">
 </p>
 <p align="center">
-  <a href="https://circleci.com/gh/outline/outline" rel="nofollow"><img src="https://circleci.com/gh/outline/outline.svg?style=shield"></a>
+  <a href="https://circleci.com/gh/outline/outline" rel="nofollow"><img src="https://circleci.com/gh/outline/outline.svg?style=shield&amp;circle-token=c0c4c2f39990e277385d5c1ae96169c409eb887a"></a>
   <a href="http://www.typescriptlang.org" rel="nofollow"><img src="https://img.shields.io/badge/%3C%2F%3E-TypeScript-%230074c1.svg" alt="TypeScript"></a>
   <a href="https://github.com/prettier/prettier"><img src="https://img.shields.io/badge/code_style-prettier-ff69b4.svg?style=flat" alt="Prettier"></a>
   <a href="https://github.com/styled-components/styled-components"><img src="https://img.shields.io/badge/style-%F0%9F%92%85%20styled--components-orange.svg" alt="Styled Components"></a>

app.json

@@ -3,7 +3,13 @@
   "description": "Open source wiki and knowledge base for growing teams",
   "website": "https://www.getoutline.com/",
   "repository": "https://github.com/outline/outline",
-  "keywords": ["wiki", "team", "node", "markdown", "slack"],
+  "keywords": [
+    "wiki",
+    "team",
+    "node",
+    "markdown",
+    "slack"
+  ],
   "success_url": "/",
   "formation": {
     "web": {
@@ -88,7 +94,7 @@
     },
     "OIDC_DISPLAY_NAME": {
       "description": "Display name for OIDC authentication",
-      "value": "OpenID Connect",
+      "value": "OpenID",
       "required": false
     },
     "OIDC_SCOPES": {
@@ -182,7 +188,7 @@
       "required": false
     },
     "GOOGLE_ANALYTICS_ID": {
-      "description": "G-xxxx (optional)",
+      "description": "UA-xxxx (optional)",
       "required": false
     },
     "SENTRY_DSN": {

app/.eslintrc

@@ -1,7 +1,6 @@
 {
   "extends": [
     "../.eslintrc",
-    "plugin:react/recommended",
     "plugin:react-hooks/recommended",
   ],
   "plugins": [

app/actions/definitions/collections.tsx

@@ -4,7 +4,6 @@ import {
   PadlockIcon,
   PlusIcon,
   StarredIcon,
-  TrashIcon,
   UnstarredIcon,
 } from "outline-icons";
 import * as React from "react";
@@ -13,7 +12,6 @@ import Collection from "~/models/Collection";
 import CollectionEdit from "~/scenes/CollectionEdit";
 import CollectionNew from "~/scenes/CollectionNew";
 import CollectionPermissions from "~/scenes/CollectionPermissions";
-import CollectionDeleteDialog from "~/components/CollectionDeleteDialog";
 import DynamicCollectionIcon from "~/components/Icons/CollectionIcon";
 import { createAction } from "~/actions";
 import { CollectionSection } from "~/actions/sections";
@@ -124,13 +122,13 @@ export const starCollection = createAction({
       stores.policies.abilities(activeCollectionId).star
     );
   },
-  perform: async ({ activeCollectionId, stores }) => {
+  perform: ({ activeCollectionId, stores }) => {
     if (!activeCollectionId) {
       return;
     }
 
     const collection = stores.collections.get(activeCollectionId);
-    await collection?.star();
+    collection?.star();
   },
 });
 
@@ -150,47 +148,13 @@ export const unstarCollection = createAction({
       stores.policies.abilities(activeCollectionId).unstar
     );
   },
-  perform: async ({ activeCollectionId, stores }) => {
+  perform: ({ activeCollectionId, stores }) => {
     if (!activeCollectionId) {
       return;
     }
 
     const collection = stores.collections.get(activeCollectionId);
-    await collection?.unstar();
-  },
-});
-
-export const deleteCollection = createAction({
-  name: ({ t }) => t("Delete"),
-  analyticsName: "Delete collection",
-  section: CollectionSection,
-  icon: <TrashIcon />,
-  visible: ({ activeCollectionId, stores }) => {
-    if (!activeCollectionId) {
-      return false;
-    }
-    return stores.policies.abilities(activeCollectionId).delete;
-  },
-  perform: ({ activeCollectionId, stores, t }) => {
-    if (!activeCollectionId) {
-      return;
-    }
-
-    const collection = stores.collections.get(activeCollectionId);
-    if (!collection) {
-      return;
-    }
-
-    stores.dialogs.openModal({
-      isCentered: true,
-      title: t("Delete collection"),
-      content: (
-        <CollectionDeleteDialog
-          collection={collection}
-          onSubmit={stores.dialogs.closeAllModals}
-        />
-      ),
-    });
+    collection?.unstar();
   },
 });
 
@@ -199,5 +163,4 @@ export const rootCollectionActions = [
   createCollection,
   starCollection,
   unstarCollection,
-  deleteCollection,
 ];

app/actions/definitions/developer.tsx

@@ -5,7 +5,6 @@ import { createAction } from "~/actions";
 import { DeveloperSection } from "~/actions/sections";
 import env from "~/env";
 import { client } from "~/utils/ApiClient";
-import Logger from "~/utils/Logger";
 import { deleteAllDatabases } from "~/utils/developer";
 
 export const clearIndexedDB = createAction({
@@ -36,27 +35,16 @@ export const createTestUsers = createAction({
   },
 });
 
-export const toggleDebugLogging = createAction({
-  name: ({ t }) => t("Toggle debug logging"),
-  icon: <ToolsIcon />,
-  section: DeveloperSection,
-  perform: async ({ t }) => {
-    Logger.debugLoggingEnabled = !Logger.debugLoggingEnabled;
-    stores.toasts.showToast(
-      Logger.debugLoggingEnabled
-        ? t("Debug logging enabled")
-        : t("Debug logging disabled")
-    );
-  },
-});
-
 export const developer = createAction({
-  name: ({ t }) => t("Development"),
+  name: ({ t }) => t("Developer"),
   keywords: "debug",
   icon: <ToolsIcon />,
   iconInContextMenu: false,
   section: DeveloperSection,
-  children: [clearIndexedDB, toggleDebugLogging, createTestUsers],
+  visible: ({ event }) =>
+    env.ENVIRONMENT === "development" ||
+    (event instanceof KeyboardEvent && event.altKey),
+  children: [clearIndexedDB, createTestUsers],
 });
 
 export const rootDeveloperActions = [developer];

app/actions/definitions/documents.tsx

@@ -61,11 +61,8 @@ export const openDocument = createAction({
         // cache if the document is renamed
         id: path.url,
         name: path.title,
-        icon: function _Icon() {
-          return stores.documents.get(path.id)?.isStarred ? (
-            <StarredIcon />
-          ) : null;
-        },
+        icon: () =>
+          stores.documents.get(path.id)?.isStarred ? <StarredIcon /> : null,
         section: DocumentSection,
         perform: () => history.push(path.url),
       }));
@@ -101,13 +98,13 @@ export const starDocument = createAction({
       !document?.isStarred && stores.policies.abilities(activeDocumentId).star
     );
   },
-  perform: async ({ activeDocumentId, stores }) => {
+  perform: ({ activeDocumentId, stores }) => {
     if (!activeDocumentId) {
       return;
     }
 
     const document = stores.documents.get(activeDocumentId);
-    await document?.star();
+    document?.star();
   },
 });
 
@@ -127,13 +124,13 @@ export const unstarDocument = createAction({
       stores.policies.abilities(activeDocumentId).unstar
     );
   },
-  perform: async ({ activeDocumentId, stores }) => {
+  perform: ({ activeDocumentId, stores }) => {
     if (!activeDocumentId) {
       return;
     }
 
     const document = stores.documents.get(activeDocumentId);
-    await document?.unstar();
+    document?.unstar();
   },
 });
 
@@ -162,7 +159,7 @@ export const publishDocument = createAction({
     }
 
     if (document?.collectionId) {
-      await document.save(undefined, {
+      await document.save({
         publish: true,
       });
       stores.toasts.showToast(t("Document published"), {
@@ -189,14 +186,14 @@ export const unpublishDocument = createAction({
     }
     return stores.policies.abilities(activeDocumentId).unpublish;
   },
-  perform: async ({ activeDocumentId, stores, t }) => {
+  perform: ({ activeDocumentId, stores, t }) => {
     if (!activeDocumentId) {
       return;
     }
 
     const document = stores.documents.get(activeDocumentId);
 
-    await document?.unpublish();
+    document?.unpublish();
 
     stores.toasts.showToast(t("Document unpublished"), {
       type: "success",
@@ -221,14 +218,14 @@ export const subscribeDocument = createAction({
       stores.policies.abilities(activeDocumentId).subscribe
     );
   },
-  perform: async ({ activeDocumentId, stores, t }) => {
+  perform: ({ activeDocumentId, stores, t }) => {
     if (!activeDocumentId) {
       return;
     }
 
     const document = stores.documents.get(activeDocumentId);
 
-    await document?.subscribe();
+    document?.subscribe();
 
     stores.toasts.showToast(t("Subscribed to document notifications"), {
       type: "success",
@@ -253,14 +250,14 @@ export const unsubscribeDocument = createAction({
       stores.policies.abilities(activeDocumentId).unsubscribe
     );
   },
-  perform: async ({ activeDocumentId, stores, currentUserId, t }) => {
+  perform: ({ activeDocumentId, stores, currentUserId, t }) => {
     if (!activeDocumentId || !currentUserId) {
       return;
     }
 
     const document = stores.documents.get(activeDocumentId);
 
-    await document?.unsubscribe(currentUserId);
+    document?.unsubscribe(currentUserId);
 
     stores.toasts.showToast(t("Unsubscribed from document notifications"), {
       type: "success",
@@ -277,13 +274,13 @@ export const downloadDocumentAsHTML = createAction({
   iconInContextMenu: false,
   visible: ({ activeDocumentId, stores }) =>
     !!activeDocumentId && stores.policies.abilities(activeDocumentId).download,
-  perform: async ({ activeDocumentId, stores }) => {
+  perform: ({ activeDocumentId, stores }) => {
     if (!activeDocumentId) {
       return;
     }
 
     const document = stores.documents.get(activeDocumentId);
-    await document?.download(ExportContentType.Html);
+    document?.download(ExportContentType.Html);
   },
 });
 
@@ -324,13 +321,13 @@ export const downloadDocumentAsMarkdown = createAction({
   iconInContextMenu: false,
   visible: ({ activeDocumentId, stores }) =>
     !!activeDocumentId && stores.policies.abilities(activeDocumentId).download,
-  perform: async ({ activeDocumentId, stores }) => {
+  perform: ({ activeDocumentId, stores }) => {
     if (!activeDocumentId) {
       return;
     }
 
     const document = stores.documents.get(activeDocumentId);
-    await document?.download(ExportContentType.Markdown);
+    document?.download(ExportContentType.Markdown);
   },
 });
 
@@ -407,19 +404,13 @@ export const pinDocumentToCollection = createAction({
       return;
     }
 
-    try {
-      const document = stores.documents.get(activeDocumentId);
-      await document?.pin(document.collectionId);
+    const document = stores.documents.get(activeDocumentId);
+    await document?.pin(document.collectionId);
 
-      const collection = stores.collections.get(activeCollectionId);
+    const collection = stores.collections.get(activeCollectionId);
 
-      if (!collection || !location.pathname.startsWith(collection?.url)) {
-        stores.toasts.showToast(t("Pinned to collection"));
-      }
-    } catch (err) {
-      stores.toasts.showToast(err.message, {
-        type: "error",
-      });
+    if (!collection || !location.pathname.startsWith(collection?.url)) {
+      stores.toasts.showToast(t("Pinned to collection"));
     }
   },
 });
@@ -452,16 +443,10 @@ export const pinDocumentToHome = createAction({
     }
     const document = stores.documents.get(activeDocumentId);
 
-    try {
-      await document?.pin();
+    await document?.pin();
 
-      if (location.pathname !== homePath()) {
-        stores.toasts.showToast(t("Pinned to team home"));
-      }
-    } catch (err) {
-      stores.toasts.showToast(err.message, {
-        type: "error",
-      });
+    if (location.pathname !== homePath()) {
+      stores.toasts.showToast(t("Pinned to team home"));
     }
   },
 });

app/actions/definitions/navigation.tsx

@@ -30,13 +30,15 @@ import { isMac } from "~/utils/browser";
 import history from "~/utils/history";
 import isCloudHosted from "~/utils/isCloudHosted";
 import {
+  organizationSettingsPath,
+  profileSettingsPath,
+  accountPreferencesPath,
   homePath,
   searchPath,
   draftsPath,
   templatesPath,
   archivePath,
   trashPath,
-  settingsPath,
 } from "~/utils/routeHelpers";
 
 export const navigateToHome = createAction({
@@ -103,7 +105,7 @@ export const navigateToSettings = createAction({
   icon: <SettingsIcon />,
   visible: ({ stores }) =>
     stores.policies.abilities(stores.auth.team?.id || "").update,
-  perform: () => history.push(settingsPath("details")),
+  perform: () => history.push(organizationSettingsPath()),
 });
 
 export const navigateToProfileSettings = createAction({
@@ -112,16 +114,7 @@ export const navigateToProfileSettings = createAction({
   section: NavigationSection,
   iconInContextMenu: false,
   icon: <ProfileIcon />,
-  perform: () => history.push(settingsPath()),
-});
-
-export const navigateToNotificationSettings = createAction({
-  name: ({ t }) => t("Notifications"),
-  analyticsName: "Navigate to notification settings",
-  section: NavigationSection,
-  iconInContextMenu: false,
-  icon: <EmailIcon />,
-  perform: () => history.push(settingsPath("notifications")),
+  perform: () => history.push(profileSettingsPath()),
 });
 
 export const navigateToAccountPreferences = createAction({
@@ -130,7 +123,7 @@ export const navigateToAccountPreferences = createAction({
   section: NavigationSection,
   iconInContextMenu: false,
   icon: <SettingsIcon />,
-  perform: () => history.push(settingsPath("preferences")),
+  perform: () => history.push(accountPreferencesPath()),
 });
 
 export const openAPIDocumentation = createAction({
@@ -142,14 +135,6 @@ export const openAPIDocumentation = createAction({
   perform: () => window.open(developersUrl()),
 });
 
-export const toggleSidebar = createAction({
-  name: ({ t }) => t("Toggle sidebar"),
-  analyticsName: "Toggle sidebar",
-  keywords: "hide show navigation",
-  section: NavigationSection,
-  perform: ({ stores }) => stores.ui.toggleCollapsedSidebar(),
-});
-
 export const openFeedbackUrl = createAction({
   name: ({ t }) => t("Send us feedback"),
   analyticsName: "Open feedback",
@@ -225,6 +210,5 @@ export const rootNavigationActions = [
   openBugReportUrl,
   openChangelog,
   openKeyboardShortcuts,
-  toggleSidebar,
   logout,
 ];

app/actions/definitions/notifications.tsx

@@ -1,16 +0,0 @@
-import { MarkAsReadIcon } from "outline-icons";
-import * as React from "react";
-import { createAction } from "..";
-import { NotificationSection } from "../sections";
-
-export const markNotificationsAsRead = createAction({
-  name: ({ t }) => t("Mark notifications as read"),
-  analyticsName: "Mark notifications as read",
-  section: NotificationSection,
-  icon: <MarkAsReadIcon />,
-  shortcut: ["Shift+Escape"],
-  perform: ({ stores }) => stores.notifications.markAllAsRead(),
-  visible: ({ stores }) => stores.notifications.approximateUnreadCount > 0,
-});
-
-export const rootNotificationActions = [markNotificationsAsRead];

app/actions/definitions/settings.tsx

@@ -43,9 +43,8 @@ export const changeTheme = createAction({
     isContextMenu ? t("Appearance") : t("Change theme"),
   analyticsName: "Change theme",
   placeholder: ({ t }) => t("Change theme to"),
-  icon: function _Icon() {
-    return stores.ui.resolvedTheme === "light" ? <SunIcon /> : <MoonIcon />;
-  },
+  icon: () =>
+    stores.ui.resolvedTheme === "light" ? <SunIcon /> : <MoonIcon />,
   keywords: "appearance display",
   section: SettingsSection,
   children: [changeToLightTheme, changeToDarkTheme, changeToSystemTheme],

app/actions/definitions/teams.tsx

@@ -16,20 +16,18 @@ export const createTeamsList = ({ stores }: { stores: RootStore }) =>
     analyticsName: "Switch workspace",
     section: TeamSection,
     keywords: "change switch workspace organization team",
-    icon: function _Icon() {
-      return (
-        <StyledTeamLogo
-          alt={session.name}
-          model={{
-            initial: session.name[0],
-            avatarUrl: session.avatarUrl,
-            id: session.id,
-            color: stringToColor(session.id),
-          }}
-          size={24}
-        />
-      );
-    },
+    icon: () => (
+      <StyledTeamLogo
+        alt={session.name}
+        model={{
+          initial: session.name[0],
+          avatarUrl: session.avatarUrl,
+          id: session.id,
+          color: stringToColor(session.id),
+        }}
+        size={24}
+      />
+    ),
     visible: ({ currentTeamId }: ActionContext) => currentTeamId !== session.id,
     perform: () => (window.location.href = session.url),
   })) ?? [];

app/actions/definitions/users.tsx

@@ -2,7 +2,6 @@ import { PlusIcon } from "outline-icons";
 import * as React from "react";
 import stores from "~/stores";
 import Invite from "~/scenes/Invite";
-import { UserDeleteDialog } from "~/components/UserDialogs";
 import { createAction } from "~/actions";
 import { UserSection } from "~/actions/sections";
 
@@ -22,31 +21,4 @@ export const inviteUser = createAction({
   },
 });
 
-export const deleteUserActionFactory = (userId: string) =>
-  createAction({
-    name: ({ t }) => `${t("Delete user")}…`,
-    analyticsName: "Delete user",
-    keywords: "leave",
-    dangerous: true,
-    section: UserSection,
-    visible: ({ stores }) => stores.policies.abilities(userId).delete,
-    perform: ({ t }) => {
-      const user = stores.users.get(userId);
-      if (!user) {
-        return;
-      }
-
-      stores.dialogs.openModal({
-        title: t("Delete user"),
-        isCentered: true,
-        content: (
-          <UserDeleteDialog
-            user={user}
-            onSubmit={stores.dialogs.closeAllModals}
-          />
-        ),
-      });
-    },
-  });
-
 export const rootUserActions = [inviteUser];

app/actions/index.ts

@@ -35,7 +35,7 @@ export function createAction(definition: Optional<Action, "id">): Action {
           return definition.perform?.(context);
         }
       : undefined,
-    id: definition.id ?? uuidv4(),
+    id: uuidv4(),
   };
 }
 

app/actions/root.ts

@@ -2,7 +2,6 @@ import { rootCollectionActions } from "./definitions/collections";
 import { rootDeveloperActions } from "./definitions/developer";
 import { rootDocumentActions } from "./definitions/documents";
 import { rootNavigationActions } from "./definitions/navigation";
-import { rootNotificationActions } from "./definitions/notifications";
 import { rootRevisionActions } from "./definitions/revisions";
 import { rootSettingsActions } from "./definitions/settings";
 import { rootTeamActions } from "./definitions/teams";
@@ -13,7 +12,6 @@ export default [
   ...rootDocumentActions,
   ...rootUserActions,
   ...rootNavigationActions,
-  ...rootNotificationActions,
   ...rootRevisionActions,
   ...rootSettingsActions,
   ...rootDeveloperActions,

app/actions/sections.ts

@@ -12,8 +12,6 @@ export const SettingsSection = ({ t }: ActionContext) => t("Settings");
 
 export const NavigationSection = ({ t }: ActionContext) => t("Navigation");
 
-export const NotificationSection = ({ t }: ActionContext) => t("Notification");
-
 export const UserSection = ({ t }: ActionContext) => t("People");
 
 export const TeamSection = ({ t }: ActionContext) => t("Workspace");

app/components/ActionButton.tsx

@@ -1,9 +1,8 @@
-/* eslint-disable react/prop-types */
 import * as React from "react";
 import Tooltip, { Props as TooltipProps } from "~/components/Tooltip";
 import { Action, ActionContext } from "~/types";
 
-export type Props = React.HTMLAttributes<HTMLButtonElement> & {
+export type Props = React.ComponentPropsWithoutRef<"button"> & {
   /** Show the button in a disabled state */
   disabled?: boolean;
   /** Hide the button entirely if action is not applicable */
@@ -19,17 +18,14 @@ export type Props = React.HTMLAttributes<HTMLButtonElement> & {
 /**
  * Button that can be used to trigger an action definition.
  */
-const ActionButton = React.forwardRef<HTMLButtonElement, Props>(
-  function _ActionButton(
+const ActionButton = React.forwardRef(
+  (
     { action, context, tooltip, hideOnActionDisabled, ...rest }: Props,
     ref: React.Ref<HTMLButtonElement>
-  ) {
+  ) => {
     const [executing, setExecuting] = React.useState(false);
     const disabled = rest.disabled;
 
-    if (action && !context) {
-      throw new Error("Context must be provided with action");
-    }
     if (!context || !action) {
       return <button {...rest} ref={ref} />;
     }

app/components/Analytics.tsx

@@ -5,14 +5,10 @@ import * as React from "react";
 import { IntegrationService } from "@shared/types";
 import env from "~/env";
 
-type Props = {
-  children?: React.ReactNode;
-};
-
-const Analytics: React.FC = ({ children }: Props) => {
+const Analytics: React.FC = ({ children }) => {
   // Google Analytics 3
   React.useEffect(() => {
-    if (!env.GOOGLE_ANALYTICS_ID?.startsWith("UA-")) {
+    if (!env.GOOGLE_ANALYTICS_ID) {
       return;
     }
 
@@ -41,36 +37,25 @@ const Analytics: React.FC = ({ children }: Props) => {
 
   // Google Analytics 4
   React.useEffect(() => {
-    const measurementIds = [];
-
-    if (env.analytics.service === IntegrationService.GoogleAnalytics) {
-      measurementIds.push(escape(env.analytics.settings?.measurementId));
-    }
-    if (env.GOOGLE_ANALYTICS_ID?.startsWith("G-")) {
-      measurementIds.push(env.GOOGLE_ANALYTICS_ID);
-    }
-    if (measurementIds.length === 0) {
+    if (env.analytics.service !== IntegrationService.GoogleAnalytics) {
       return;
     }
 
-    const params = {
-      allow_google_signals: false,
-      restricted_data_processing: true,
-    };
+    const measurementId = escape(env.analytics.settings?.measurementId);
 
     window.dataLayer = window.dataLayer || [];
     window.gtag = function () {
       window.dataLayer.push(arguments);
     };
     window.gtag("js", new Date());
-
-    for (const measurementId of measurementIds) {
-      window.gtag("config", measurementId, params);
-    }
+    window.gtag("config", measurementId, {
+      allow_google_signals: false,
+      restricted_data_processing: true,
+    });
 
     const script = document.createElement("script");
     script.type = "text/javascript";
-    script.src = `https://www.googletagmanager.com/gtag/js?id=${measurementIds[0]}`;
+    script.src = `https://www.googletagmanager.com/gtag/js?id=${measurementId}`;
     script.async = true;
     document.getElementsByTagName("head")[0]?.appendChild(script);
   }, []);

app/components/ArrowKeyNavigation.tsx

@@ -20,10 +20,6 @@ function ArrowKeyNavigation(
   const handleKeyDown = React.useCallback(
     (ev) => {
       if (onEscape) {
-        if (ev.nativeEvent.isComposing) {
-          return;
-        }
-
         if (ev.key === "Escape") {
           onEscape(ev);
         }

app/components/Authenticated.tsx

@@ -2,9 +2,9 @@ import { observer } from "mobx-react";
 import * as React from "react";
 import { useTranslation } from "react-i18next";
 import { Redirect } from "react-router-dom";
+import LoadingIndicator from "~/components/LoadingIndicator";
 import useStores from "~/hooks/useStores";
 import { changeLanguage } from "~/utils/language";
-import LoadingIndicator from "./LoadingIndicator";
 
 type Props = {
   children: JSX.Element;
@@ -18,18 +18,20 @@ const Authenticated = ({ children }: Props) => {
   // Watching for language changes here as this is the earliest point we have
   // the user available and means we can start loading translations faster
   React.useEffect(() => {
-    void changeLanguage(language, i18n);
+    changeLanguage(language, i18n);
   }, [i18n, language]);
 
   if (auth.authenticated) {
+    const { user, team } = auth;
+
+    if (!team || !user) {
+      return <LoadingIndicator />;
+    }
+
     return children;
   }
 
-  if (auth.isFetching) {
-    return <LoadingIndicator />;
-  }
-
-  void auth.logout(true);
+  auth.logout(true);
   return <Redirect to="/" />;
 };
 

app/components/AuthenticatedLayout.tsx

@@ -15,7 +15,6 @@ import type { Editor as TEditor } from "~/editor";
 import usePolicy from "~/hooks/usePolicy";
 import useStores from "~/hooks/useStores";
 import history from "~/utils/history";
-import lazyWithRetry from "~/utils/lazyWithRetry";
 import {
   searchPath,
   newDocumentPath,
@@ -26,22 +25,18 @@ import {
 } from "~/utils/routeHelpers";
 import Fade from "./Fade";
 
-const DocumentComments = lazyWithRetry(
+const DocumentComments = React.lazy(
   () => import("~/scenes/Document/components/Comments")
 );
-const DocumentHistory = lazyWithRetry(
+const DocumentHistory = React.lazy(
   () => import("~/scenes/Document/components/History")
 );
-const DocumentInsights = lazyWithRetry(
+const DocumentInsights = React.lazy(
   () => import("~/scenes/Document/components/Insights")
 );
-const CommandBar = lazyWithRetry(() => import("~/components/CommandBar"));
+const CommandBar = React.lazy(() => import("~/components/CommandBar"));
 
-type Props = {
-  children?: React.ReactNode;
-};
-
-const AuthenticatedLayout: React.FC = ({ children }: Props) => {
+const AuthenticatedLayout: React.FC = ({ children }) => {
   const { ui, auth } = useStores();
   const location = useLocation();
   const can = usePolicy(ui.activeCollectionId);
@@ -66,7 +61,7 @@ const AuthenticatedLayout: React.FC = ({ children }: Props) => {
       return;
     }
     const { activeCollectionId } = ui;
-    if (!activeCollectionId || !can.createDocument) {
+    if (!activeCollectionId || !can.update) {
       return;
     }
     history.push(newDocumentPath(activeCollectionId));
@@ -101,10 +96,7 @@ const AuthenticatedLayout: React.FC = ({ children }: Props) => {
     team?.getPreference(TeamPreference.Commenting);
 
   const sidebarRight = (
-    <AnimatePresence
-      initial={false}
-      key={ui.activeDocumentId ? "active" : "inactive"}
-    >
+    <AnimatePresence>
       {(showHistory || showInsights || showComments) && (
         <Route path={`/doc/${slug}`}>
           <SidebarRight>
@@ -126,9 +118,7 @@ const AuthenticatedLayout: React.FC = ({ children }: Props) => {
         <RegisterKeyDown trigger="t" handler={goToSearch} />
         <RegisterKeyDown trigger="/" handler={goToSearch} />
         {children}
-        <React.Suspense fallback={null}>
-          <CommandBar />
-        </React.Suspense>
+        <CommandBar />
       </Layout>
     </DocumentContext.Provider>
   );

app/components/Avatar/Avatar.tsx

@@ -1,16 +1,9 @@
 import * as React from "react";
 import styled from "styled-components";
+import { s } from "@shared/styles";
 import useBoolean from "~/hooks/useBoolean";
 import Initials from "./Initials";
 
-export enum AvatarSize {
-  Small = 16,
-  Medium = 24,
-  Large = 32,
-  XLarge = 48,
-  XXLarge = 64,
-}
-
 export interface IAvatar {
   avatarUrl: string | null;
   color?: string;
@@ -19,8 +12,9 @@ export interface IAvatar {
 }
 
 type Props = {
-  size: AvatarSize;
+  size: number;
   src?: string;
+  icon?: React.ReactNode;
   model?: IAvatar;
   alt?: string;
   showBorder?: boolean;
@@ -30,7 +24,7 @@ type Props = {
 };
 
 function Avatar(props: Props) {
-  const { showBorder, model, style, ...rest } = props;
+  const { icon, showBorder, model, style, ...rest } = props;
   const src = props.src || model?.avatarUrl;
   const [error, handleError] = useBoolean(false);
 
@@ -50,12 +44,13 @@ function Avatar(props: Props) {
       ) : (
         <Initials $showBorder={showBorder} {...rest} />
       )}
+      {icon && <IconWrapper>{icon}</IconWrapper>}
     </Relative>
   );
 }
 
 Avatar.defaultProps = {
-  size: AvatarSize.Medium,
+  size: 24,
 };
 
 const Relative = styled.div`
@@ -64,6 +59,18 @@ const Relative = styled.div`
   flex-shrink: 0;
 `;
 
+const IconWrapper = styled.div`
+  display: flex;
+  position: absolute;
+  bottom: -2px;
+  right: -2px;
+  background: ${s("accent")};
+  border: 2px solid ${s("background")};
+  border-radius: 100%;
+  width: 20px;
+  height: 20px;
+`;
+
 const CircleImg = styled.img<{ size: number; $showBorder?: boolean }>`
   display: block;
   width: ${(props) => props.size}px;

app/components/Badge.ts

@@ -7,11 +7,7 @@ const Badge = styled.span<{ yellow?: boolean; primary?: boolean }>`
   background-color: ${({ yellow, primary, theme }) =>
     yellow ? theme.yellow : primary ? theme.accent : "transparent"};
   color: ${({ primary, yellow, theme }) =>
-    primary
-      ? theme.accentText
-      : yellow
-      ? theme.almostBlack
-      : theme.textTertiary};
+    primary ? theme.white : yellow ? theme.almostBlack : theme.textTertiary};
   border: 1px solid
     ${({ primary, yellow, theme }) =>
       primary || yellow

app/components/Button.tsx

@@ -60,7 +60,7 @@ const RealButton = styled(ActionButton)<RealProps>`
   ${(props) =>
     props.$neutral &&
     `
-    background: inherit;
+    background: ${props.theme.buttonNeutralBackground};
     color: ${props.theme.buttonNeutralText};
     box-shadow: ${
       props.$borderOnHover

app/components/CenteredContent.tsx

@@ -3,7 +3,6 @@ import styled from "styled-components";
 import breakpoint from "styled-components-breakpoint";
 
 type Props = {
-  children?: React.ReactNode;
   withStickyHeader?: boolean;
 };
 
@@ -27,7 +26,7 @@ const Content = styled.div`
   `};
 `;
 
-const CenteredContent: React.FC<Props> = ({ children, ...rest }: Props) => (
+const CenteredContent: React.FC<Props> = ({ children, ...rest }) => (
   <Container {...rest}>
     <Content>{children}</Content>
   </Container>

app/components/Collaborators.tsx

@@ -57,7 +57,7 @@ function Collaborators(props: Props) {
 
     if (!isEqual(requestedUserIds, ids) && ids.length > 0) {
       setRequestedUserIds(ids);
-      void users.fetchPage({ ids, limit: 100 });
+      users.fetchPage({ ids, limit: 100 });
     }
   }, [document, users, presentIds, document.collaboratorIds, requestedUserIds]);
 

app/components/CollectionDescription.tsx

@@ -71,9 +71,9 @@ function CollectionDescription({ collection }: Props) {
   );
 
   const handleChange = React.useCallback(
-    async (getValue) => {
+    (getValue) => {
       setDirty(true);
-      await handleSave(getValue);
+      handleSave(getValue);
     },
     [handleSave]
   );
@@ -111,7 +111,7 @@ function CollectionDescription({ collection }: Props) {
                 onBlur={handleStopEditing}
                 maxLength={1000}
                 embedsDisabled
-                canUpdate
+                readOnlyWriteCheckboxes
               />
             </React.Suspense>
           ) : (

app/components/CommandBar.tsx

@@ -25,9 +25,9 @@ function CommandBar() {
 
   const { rootAction } = useKBar((state) => ({
     rootAction: state.currentRootActionId
-      ? (state.actions[
+      ? ((state.actions[
           state.currentRootActionId
-        ] as unknown as CommandBarAction)
+        ] as unknown) as CommandBarAction)
       : undefined,
   }));
 
@@ -52,11 +52,7 @@ function CommandBar() {
   );
 }
 
-type Props = {
-  children?: React.ReactNode;
-};
-
-const KBarPortal: React.FC = ({ children }: Props) => {
+const KBarPortal: React.FC = ({ children }) => {
   const { showing } = useKBar((state) => ({
     showing: state.visualState !== "hidden",
   }));

app/components/CommandBarItem.tsx

@@ -5,7 +5,6 @@ import styled, { css, useTheme } from "styled-components";
 import { s, ellipsis } from "@shared/styles";
 import Flex from "~/components/Flex";
 import Key from "~/components/Key";
-import Text from "./Text";
 
 type Props = {
   action: ActionImpl;
@@ -56,36 +55,22 @@ function CommandBarItem(
         {action.children?.length ? "…" : ""}
       </Content>
       {action.shortcut?.length ? (
-        <Shortcut>
-          {action.shortcut.map((sc: string, index) => (
-            <React.Fragment key={sc}>
-              {index > 0 ? (
-                <>
-                  {" "}
-                  <Text size="xsmall" as="span" type="secondary">
-                    then
-                  </Text>{" "}
-                </>
-              ) : (
-                ""
-              )}
-              {sc.split("+").map((s) => (
-                <Key key={s}>{s}</Key>
-              ))}
-            </React.Fragment>
+        <div
+          style={{
+            display: "grid",
+            gridAutoFlow: "column",
+            gap: "4px",
+          }}
+        >
+          {action.shortcut.map((sc: string) => (
+            <Key key={sc}>{sc}</Key>
           ))}
-        </Shortcut>
+        </div>
       ) : null}
     </Item>
   );
 }
 
-const Shortcut = styled.div`
-  display: grid;
-  grid-auto-flow: column;
-  gap: 4px;
-`;
-
 const Icon = styled(Flex)`
   align-items: center;
   justify-content: center;

app/components/CommandBarResults.tsx

@@ -8,33 +8,24 @@ export default function CommandBarResults() {
   const { results, rootActionId } = useMatches();
 
   return (
-    <Container>
-      <KBarResults
-        items={results}
-        maxHeight={400}
-        onRender={({ item, active }) =>
-          typeof item === "string" ? (
-            <Header>{item}</Header>
-          ) : (
-            <CommandBarItem
-              action={item}
-              active={active}
-              currentRootActionId={rootActionId}
-            />
-          )
-        }
-      />
-    </Container>
+    <KBarResults
+      items={results}
+      maxHeight={400}
+      onRender={({ item, active }) =>
+        typeof item === "string" ? (
+          <Header>{item}</Header>
+        ) : (
+          <CommandBarItem
+            action={item}
+            active={active}
+            currentRootActionId={rootActionId}
+          />
+        )
+      }
+    />
   );
 }
 
-// Cannot style KBarResults unfortunately, so we must wrap and target the inner
-const Container = styled.div`
-  > div {
-    padding-bottom: 8px;
-  }
-`;
-
 const Header = styled.h3`
   font-size: 13px;
   letter-spacing: 0.04em;

app/components/ConfirmationDialog.tsx

@@ -17,7 +17,6 @@ type Props = {
   danger?: boolean;
   /** Keep the submit button disabled */
   disabled?: boolean;
-  children?: React.ReactNode;
 };
 
 const ConfirmationDialog: React.FC<Props> = ({
@@ -27,7 +26,7 @@ const ConfirmationDialog: React.FC<Props> = ({
   savingText,
   danger,
   disabled = false,
-}: Props) => {
+}) => {
   const [isSaving, setIsSaving] = React.useState(false);
   const { dialogs } = useStores();
   const { showToast } = useToasts();

app/components/ContentEditable.tsx

@@ -30,74 +30,69 @@ export type RefHandle = {
  * Defines a content editable component with the same interface as a native
  * HTMLInputElement (or, as close as we can get).
  */
-const ContentEditable = React.forwardRef(function _ContentEditable(
-  {
-    disabled,
-    onChange,
-    onInput,
-    onBlur,
-    onKeyDown,
-    value,
-    children,
-    className,
-    maxLength,
-    autoFocus,
-    placeholder,
-    readOnly,
-    dir,
-    onClick,
-    ...rest
-  }: Props,
-  ref: React.RefObject<RefHandle>
-) {
-  const contentRef = React.useRef<HTMLSpanElement>(null);
-  const [innerValue, setInnerValue] = React.useState<string>(value);
-  const lastValue = React.useRef(value);
+const ContentEditable = React.forwardRef(
+  (
+    {
+      disabled,
+      onChange,
+      onInput,
+      onBlur,
+      onKeyDown,
+      value,
+      children,
+      className,
+      maxLength,
+      autoFocus,
+      placeholder,
+      readOnly,
+      dir,
+      onClick,
+      ...rest
+    }: Props,
+    ref: React.RefObject<RefHandle>
+  ) => {
+    const contentRef = React.useRef<HTMLSpanElement>(null);
+    const [innerValue, setInnerValue] = React.useState<string>(value);
+    const lastValue = React.useRef("");
 
-  React.useImperativeHandle(ref, () => ({
-    focus: () => {
-      if (contentRef.current) {
-        contentRef.current.focus();
-        // looks unnecessary but required because of https://github.com/outline/outline/issues/5198
-        if (!contentRef.current.innerText) {
+    React.useImperativeHandle(ref, () => ({
+      focus: () => {
+        if (contentRef.current) {
+          contentRef.current.focus();
+          // looks unnecessary but required because of https://github.com/outline/outline/issues/5198
+          if (!contentRef.current.innerText) {
+            placeCaret(contentRef.current, true);
+          }
+        }
+      },
+      focusAtStart: () => {
+        if (contentRef.current) {
+          contentRef.current.focus();
           placeCaret(contentRef.current, true);
         }
-      }
-    },
-    focusAtStart: () => {
-      if (contentRef.current) {
-        contentRef.current.focus();
-        placeCaret(contentRef.current, true);
-      }
-    },
-    focusAtEnd: () => {
-      if (contentRef.current) {
-        contentRef.current.focus();
-        placeCaret(contentRef.current, false);
-      }
-    },
-    getComputedDirection: () => {
-      if (contentRef.current) {
-        return window.getComputedStyle(contentRef.current).direction;
-      }
-      return "ltr";
-    },
-  }));
+      },
+      focusAtEnd: () => {
+        if (contentRef.current) {
+          contentRef.current.focus();
+          placeCaret(contentRef.current, false);
+        }
+      },
+      getComputedDirection: () => {
+        if (contentRef.current) {
+          return window.getComputedStyle(contentRef.current).direction;
+        }
+        return "ltr";
+      },
+    }));
 
-  const wrappedEvent =
-    (
+    const wrappedEvent = (
       callback:
         | React.FocusEventHandler<HTMLSpanElement>
         | React.FormEventHandler<HTMLSpanElement>
         | React.KeyboardEventHandler<HTMLSpanElement>
         | undefined
-    ) =>
-    (event: any) => {
-      if (readOnly) {
-        return;
-      }
-
-      const text = event.currentTarget.textContent || "";
+    ) => (event: any) => {
+      const text = contentRef.current?.innerText || "";
 
       if (maxLength && isPrintableKeyEvent(event) && text.length >= maxLength) {
         event?.preventDefault();
@@ -106,62 +101,62 @@ const ContentEditable = React.forwardRef(function _ContentEditable(
 
       if (text !== lastValue.current) {
         lastValue.current = text;
-        onChange?.(text);
+        onChange && onChange(text);
       }
 
       callback?.(event);
     };
 
-  // This is to account for being within a React.Suspense boundary, in this
-  // case the component may be rendered with display: none. React 18 may solve
-  // this in the future by delaying useEffect hooks:
-  // https://github.com/facebook/react/issues/14536#issuecomment-861980492
-  const isVisible = useOnScreen(contentRef);
+    // This is to account for being within a React.Suspense boundary, in this
+    // case the component may be rendered with display: none. React 18 may solve
+    // this in the future by delaying useEffect hooks:
+    // https://github.com/facebook/react/issues/14536#issuecomment-861980492
+    const isVisible = useOnScreen(contentRef);
 
-  React.useEffect(() => {
-    if (autoFocus && isVisible && !disabled && !readOnly) {
-      contentRef.current?.focus();
-    }
-  }, [autoFocus, disabled, isVisible, readOnly, contentRef]);
+    React.useEffect(() => {
+      if (autoFocus && isVisible && !disabled && !readOnly) {
+        contentRef.current?.focus();
+      }
+    }, [autoFocus, disabled, isVisible, readOnly, contentRef]);
 
-  React.useEffect(() => {
-    if (contentRef.current && value !== contentRef.current.textContent) {
-      setInnerValue(value);
-    }
-  }, [value, contentRef]);
+    React.useEffect(() => {
+      if (value !== contentRef.current?.innerText) {
+        setInnerValue(value);
+      }
+    }, [value, contentRef]);
 
-  // Ensure only plain text can be pasted into input when pasting from another
-  // rich text source. Note: If `onPaste` prop is passed then it takes
-  // priority over this behavior.
-  const handlePaste = React.useCallback(
-    (event: React.ClipboardEvent<HTMLSpanElement>) => {
-      event.preventDefault();
-      const text = event.clipboardData.getData("text/plain");
-      window.document.execCommand("insertText", false, text);
-    },
-    []
-  );
+    // Ensure only plain text can be pasted into input when pasting from another
+    // rich text source
+    const handlePaste = React.useCallback(
+      (event: React.ClipboardEvent<HTMLSpanElement>) => {
+        event.preventDefault();
+        const text = event.clipboardData.getData("text/plain");
+        window.document.execCommand("insertText", false, text);
+      },
+      []
+    );
 
-  return (
-    <div className={className} dir={dir} onClick={onClick}>
-      <Content
-        ref={contentRef}
-        contentEditable={!disabled && !readOnly}
-        onInput={wrappedEvent(onInput)}
-        onBlur={wrappedEvent(onBlur)}
-        onKeyDown={wrappedEvent(onKeyDown)}
-        onPaste={handlePaste}
-        data-placeholder={placeholder}
-        suppressContentEditableWarning
-        role="textbox"
-        {...rest}
-      >
-        {innerValue}
-      </Content>
-      {children}
-    </div>
-  );
-});
+    return (
+      <div className={className} dir={dir} onClick={onClick}>
+        <Content
+          ref={contentRef}
+          contentEditable={!disabled && !readOnly}
+          onInput={wrappedEvent(onInput)}
+          onBlur={wrappedEvent(onBlur)}
+          onKeyDown={wrappedEvent(onKeyDown)}
+          onPaste={handlePaste}
+          data-placeholder={placeholder}
+          suppressContentEditableWarning
+          role="textbox"
+          {...rest}
+        >
+          {innerValue}
+        </Content>
+        {children}
+      </div>
+    );
+  }
+);
 
 function placeCaret(element: HTMLElement, atStart: boolean) {
   if (
@@ -185,7 +180,6 @@ const Content = styled.span`
   outline: none;
   resize: none;
   cursor: text;
-  word-break: anywhere;
 
   &:empty {
     display: inline-block;

app/components/ContextMenu/MenuItem.tsx

@@ -8,7 +8,6 @@ import breakpoint from "styled-components-breakpoint";
 import MenuIconWrapper from "../MenuIconWrapper";
 
 type Props = {
-  id?: string;
   onClick?: (event: React.SyntheticEvent) => void | Promise<void>;
   active?: boolean;
   selected?: boolean;
@@ -22,7 +21,6 @@ type Props = {
   level?: number;
   icon?: React.ReactElement;
   children?: React.ReactNode;
-  ref?: React.LegacyRef<HTMLButtonElement> | undefined;
 };
 
 const MenuItem = (
@@ -39,26 +37,34 @@ const MenuItem = (
   }: Props,
   ref: React.Ref<HTMLAnchorElement>
 ) => {
-  const content = React.useCallback(
-    (props) => {
-      const handleClick = async (ev: React.MouseEvent) => {
-        hide?.();
-
-        if (onClick) {
-          ev.preventDefault();
-          await onClick(ev);
-        }
-      };
-
-      // Preventing default mousedown otherwise menu items do not work in Firefox,
-      // which triggers the hideOnClickOutside handler first via mousedown – hiding
-      // and un-rendering the menu contents.
-      const handleMouseDown = (ev: React.MouseEvent) => {
+  const handleClick = React.useCallback(
+    (ev) => {
+      if (onClick) {
         ev.preventDefault();
-        ev.stopPropagation();
-      };
+        onClick(ev);
+      }
 
-      return (
+      hide?.();
+    },
+    [onClick, hide]
+  );
+
+  // Preventing default mousedown otherwise menu items do not work in Firefox,
+  // which triggers the hideOnClickOutside handler first via mousedown – hiding
+  // and un-rendering the menu contents.
+  const handleMouseDown = React.useCallback((ev) => {
+    ev.preventDefault();
+    ev.stopPropagation();
+  }, []);
+
+  return (
+    <BaseMenuItem
+      onClick={disabled ? undefined : onClick}
+      disabled={disabled}
+      hide={hide}
+      {...rest}
+    >
+      {(props) => (
         <MenuAnchor
           {...props}
           $active={active}
@@ -79,19 +85,7 @@ const MenuItem = (
           {icon && <MenuIconWrapper>{icon}</MenuIconWrapper>}
           {children}
         </MenuAnchor>
-      );
-    },
-    [active, as, hide, icon, onClick, ref, children, selected]
-  );
-
-  return (
-    <BaseMenuItem
-      onClick={disabled ? undefined : onClick}
-      disabled={disabled}
-      hide={hide}
-      {...rest}
-    >
-      {content}
+      )}
     </BaseMenuItem>
   );
 };

app/components/ContextMenu/MouseSafeArea.tsx

@@ -24,12 +24,8 @@ type Positions = {
 export default function MouseSafeArea(props: {
   parentRef: React.RefObject<HTMLElement | null>;
 }) {
-  const {
-    x = 0,
-    y = 0,
-    height: h = 0,
-    width: w = 0,
-  } = props.parentRef.current?.getBoundingClientRect() || {};
+  const { x = 0, y = 0, height: h = 0, width: w = 0 } =
+    props.parentRef.current?.getBoundingClientRect() || {};
   const [mouseX, mouseY] = useMousePosition();
   const positions = { x, y, h, w, mouseX, mouseY };
 

app/components/ContextMenu/Template.tsx

@@ -44,35 +44,36 @@ type SubMenuProps = MenuStateReturn & {
   title: React.ReactNode;
 };
 
-const SubMenu = React.forwardRef(function _Template(
-  { templateItems, title, parentMenuState, ...rest }: SubMenuProps,
-  ref: React.LegacyRef<HTMLButtonElement>
-) {
-  const { t } = useTranslation();
-  const theme = useTheme();
-  const menu = useMenuState();
+const SubMenu = React.forwardRef(
+  (
+    { templateItems, title, parentMenuState, ...rest }: SubMenuProps,
+    ref: React.LegacyRef<HTMLButtonElement>
+  ) => {
+    const { t } = useTranslation();
+    const theme = useTheme();
+    const menu = useMenuState();
 
-  return (
-    <>
-      <MenuButton ref={ref} {...menu} {...rest}>
-        {(props) => (
-          <MenuAnchor disclosure {...props}>
-            {title} <Disclosure color={theme.textTertiary} />
-          </MenuAnchor>
-        )}
-      </MenuButton>
-      <ContextMenu
-        {...menu}
-        aria-label={t("Submenu")}
-        onClick={parentMenuState.hide}
-        parentMenuState={parentMenuState}
-      >
-        <MouseSafeArea parentRef={menu.unstable_popoverRef} />
-        <Template {...menu} items={templateItems} />
-      </ContextMenu>
-    </>
-  );
-});
+    return (
+      <>
+        <MenuButton ref={ref} {...menu} {...rest}>
+          {(props) => (
+            <MenuAnchor disclosure {...props}>
+              {title} <Disclosure color={theme.textTertiary} />
+            </MenuAnchor>
+          )}
+        </MenuButton>
+        <ContextMenu
+          {...menu}
+          aria-label={t("Submenu")}
+          onClick={parentMenuState.hide}
+        >
+          <MouseSafeArea parentRef={menu.unstable_popoverRef} />
+          <Template {...menu} items={templateItems} />
+        </ContextMenu>
+      </>
+    );
+  }
+);
 
 export function filterTemplateItems(items: TMenuItem[]): TMenuItem[] {
   return items
@@ -133,7 +134,6 @@ function Template({ items, actions, context, ...menu }: Props) {
           return (
             <MenuItem
               as={Link}
-              id={`${item.title}-${index}`}
               to={item.to}
               key={index}
               disabled={item.disabled}
@@ -149,7 +149,6 @@ function Template({ items, actions, context, ...menu }: Props) {
         if (item.type === "link") {
           return (
             <MenuItem
-              id={`${item.title}-${index}`}
               href={item.href}
               key={index}
               disabled={item.disabled}
@@ -168,7 +167,6 @@ function Template({ items, actions, context, ...menu }: Props) {
           return (
             <MenuItem
               as="button"
-              id={`${item.title}-${index}`}
               onClick={item.onClick}
               disabled={item.disabled}
               selected={item.selected}
@@ -187,7 +185,6 @@ function Template({ items, actions, context, ...menu }: Props) {
             <BaseMenuItem
               key={index}
               as={SubMenu}
-              id={`${item.title}-${index}`}
               templateItems={item.items}
               parentMenuState={menu}
               title={<Title title={item.title} icon={item.icon} />}

app/components/ContextMenu/index.tsx

@@ -39,14 +39,13 @@ export type Placement =
 type Props = MenuStateReturn & {
   "aria-label"?: string;
   /** The parent menu state if this is a submenu. */
-  parentMenuState?: Omit<MenuStateReturn, "items">;
+  parentMenuState?: MenuStateReturn;
   /** Called when the context menu is opened. */
   onOpen?: () => void;
   /** Called when the context menu is closed. */
   onClose?: () => void;
   /** Called when the context menu is clicked. */
   onClick?: (ev: React.MouseEvent) => void;
-  children?: React.ReactNode;
 };
 
 const ContextMenu: React.FC<Props> = ({
@@ -55,18 +54,14 @@ const ContextMenu: React.FC<Props> = ({
   onClose,
   parentMenuState,
   ...rest
-}: Props) => {
+}) => {
   const previousVisible = usePrevious(rest.visible);
-  const maxHeight = useMenuHeight({
-    visible: rest.visible,
-    elementRef: rest.unstable_disclosureRef,
-  });
+  const maxHeight = useMenuHeight(rest.visible, rest.unstable_disclosureRef);
   const backgroundRef = React.useRef<HTMLDivElement>(null);
   const { ui } = useStores();
   const { t } = useTranslation();
   const { setIsMenuOpen } = useMenuContext();
   const isMobile = useMobile();
-  const isSubMenu = !!parentMenuState;
 
   useUnmount(() => {
     setIsMenuOpen(false);
@@ -76,7 +71,7 @@ const ContextMenu: React.FC<Props> = ({
     if (rest.visible && !previousVisible) {
       onOpen?.();
 
-      if (!isSubMenu) {
+      if (!parentMenuState) {
         setIsMenuOpen(true);
       }
     }
@@ -84,7 +79,7 @@ const ContextMenu: React.FC<Props> = ({
     if (!rest.visible && previousVisible) {
       onClose?.();
 
-      if (!isSubMenu) {
+      if (!parentMenuState) {
         setIsMenuOpen(false);
       }
     }
@@ -95,7 +90,7 @@ const ContextMenu: React.FC<Props> = ({
     rest.visible,
     ui.sidebarCollapsed,
     setIsMenuOpen,
-    isSubMenu,
+    parentMenuState,
     t,
   ]);
 
@@ -104,15 +99,13 @@ const ContextMenu: React.FC<Props> = ({
   // https://github.com/ariakit/ariakit/issues/469
   React.useEffect(() => {
     const scrollElement = backgroundRef.current;
-    if (rest.visible && scrollElement && !isSubMenu) {
-      disableBodyScroll(scrollElement, {
-        reserveScrollBarGap: true,
-      });
+    if (rest.visible && scrollElement) {
+      disableBodyScroll(scrollElement);
     }
     return () => {
-      scrollElement && !isSubMenu && enableBodyScroll(scrollElement);
+      scrollElement && enableBodyScroll(scrollElement);
     };
-  }, [isSubMenu, rest.visible]);
+  }, [rest.visible]);
 
   // Perf win – don't render anything until the menu has been opened
   if (!rest.visible && !previousVisible) {
@@ -151,7 +144,7 @@ const ContextMenu: React.FC<Props> = ({
                   ref={backgroundRef}
                   hiddenScrollbars
                   style={
-                    topAnchor
+                    maxHeight && topAnchor
                       ? {
                           maxHeight,
                         }

app/components/DefaultCollectionInputSelect.tsx

@@ -28,7 +28,7 @@ const DefaultCollectionInputSelect = ({
   const { showToast } = useToasts();
 
   React.useEffect(() => {
-    async function fetchData() {
+    async function load() {
       if (!collections.isLoaded && !fetching && !fetchError) {
         try {
           setFetching(true);
@@ -48,7 +48,7 @@ const DefaultCollectionInputSelect = ({
         }
       }
     }
-    void fetchData();
+    load();
   }, [showToast, fetchError, t, fetching, collections]);
 
   const options = React.useMemo(

app/components/DesktopEventHandler.tsx

@@ -30,7 +30,7 @@ export default function DesktopEventHandler() {
         action: {
           text: "Install now",
           onClick: () => {
-            void Desktop.bridge?.restartAndInstall();
+            Desktop.bridge?.restartAndInstall();
           },
         },
       });

app/components/DocumentBreadcrumb.tsx

@@ -17,7 +17,6 @@ import {
 } from "~/utils/routeHelpers";
 
 type Props = {
-  children?: React.ReactNode;
   document: Document;
   onlyText?: boolean;
 };
@@ -59,7 +58,7 @@ const DocumentBreadcrumb: React.FC<Props> = ({
   document,
   children,
   onlyText,
-}: Props) => {
+}) => {
   const { collections } = useStores();
   const { t } = useTranslation();
   const category = useCategory(document);
@@ -130,11 +129,7 @@ const DocumentBreadcrumb: React.FC<Props> = ({
     );
   }
 
-  return (
-    <Breadcrumb items={items} highlightFirstItem>
-      {children}
-    </Breadcrumb>
-  );
+  return <Breadcrumb items={items} children={children} highlightFirstItem />;
 };
 
 const SmallSlash = styled(GoToIcon)`

app/components/DocumentCard.tsx

@@ -14,7 +14,6 @@ import Flex from "~/components/Flex";
 import NudeButton from "~/components/NudeButton";
 import Time from "~/components/Time";
 import useStores from "~/hooks/useStores";
-import { hover } from "~/styles";
 import CollectionIcon from "./Icons/CollectionIcon";
 import EmojiIcon from "./Icons/EmojiIcon";
 import Squircle from "./Squircle";
@@ -58,10 +57,10 @@ function DocumentCard(props: Props) {
   };
 
   const handleUnpin = React.useCallback(
-    async (ev) => {
+    (ev) => {
       ev.preventDefault();
       ev.stopPropagation();
-      await pin?.delete();
+      pin?.delete();
     },
     [pin]
   );
@@ -180,7 +179,7 @@ const Fold = styled.svg`
 const PinButton = styled(NudeButton)`
   color: ${s("textTertiary")};
 
-  &:${hover},
+  &:hover,
   &:active {
     color: ${s("text")};
   }
@@ -210,7 +209,7 @@ const Reorderable = styled.div<{ $isDragging: boolean }>`
   z-index: ${(props) => (props.$isDragging ? 1 : "inherit")};
   pointer-events: ${(props) => (props.$isDragging ? "none" : "inherit")};
 
-  &: ${hover} ${Actions} {
+  &:hover ${Actions} {
     opacity: 1;
   }
 `;
@@ -249,7 +248,7 @@ const DocumentLink = styled(Link)<{
     opacity: 0;
   }
 
-  &:${hover},
+  &:hover,
   &:active,
   &:focus,
   &:focus-within {

app/components/DocumentExplorer.tsx

@@ -45,8 +45,10 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
   const [selectedNode, selectNode] = React.useState<NavigationNode | null>(
     null
   );
-  const [initialScrollOffset, setInitialScrollOffset] =
-    React.useState<number>(0);
+  const [initialScrollOffset, setInitialScrollOffset] = React.useState<number>(
+    0
+  );
+  const [nodes, setNodes] = React.useState<NavigationNode[]>([]);
   const [activeNode, setActiveNode] = React.useState<number>(0);
   const [expandedNodes, setExpandedNodes] = React.useState<string[]>([]);
   const [itemRefs, setItemRefs] = React.useState<
@@ -77,6 +79,19 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
     setActiveNode(0);
   }, [searchTerm]);
 
+  React.useEffect(() => {
+    let results;
+
+    if (searchTerm) {
+      results = searchIndex.search(searchTerm);
+    } else {
+      results = items.filter((item) => item.type === "collection");
+    }
+
+    setInitialScrollOffset(0);
+    setNodes(results);
+  }, [searchTerm, items, searchIndex]);
+
   React.useEffect(() => {
     setItemRefs((itemRefs) =>
       map(
@@ -90,22 +105,6 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
     onSelect(selectedNode);
   }, [selectedNode, onSelect]);
 
-  function getNodes() {
-    function includeDescendants(item: NavigationNode): NavigationNode[] {
-      return expandedNodes.includes(item.id)
-        ? [item, ...descendants(item, 1).flatMap(includeDescendants)]
-        : [item];
-    }
-
-    return searchTerm
-      ? searchIndex.search(searchTerm)
-      : items
-          .filter((item) => item.type === "collection")
-          .flatMap(includeDescendants);
-  }
-
-  const nodes = getNodes();
-
   const scrollNodeIntoView = React.useCallback(
     (node: number) => {
       if (itemRefs[node] && itemRefs[node].current) {
@@ -131,7 +130,7 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
         scrollOffset: number;
       };
       const itemsHeight = itemCount * itemSize;
-      return itemsHeight < Number(height) ? 0 : scrollOffset;
+      return itemsHeight < height ? 0 : scrollOffset;
     }
     return 0;
   };
@@ -146,6 +145,7 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
     const newNodes = filter(nodes, (node) => !includes(descendantIds, node.id));
     const scrollOffset = calculateInitialScrollOffset(newNodes.length);
     setInitialScrollOffset(scrollOffset);
+    setNodes(newNodes);
   };
 
   const expand = (node: number) => {
@@ -156,18 +156,9 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
     newNodes.splice(node + 1, 0, ...descendants(nodes[node], 1));
     const scrollOffset = calculateInitialScrollOffset(newNodes.length);
     setInitialScrollOffset(scrollOffset);
+    setNodes(newNodes);
   };
 
-  React.useEffect(() => {
-    collections.orderedData
-      .filter(
-        (collection) => expandedNodes.includes(collection.id) || searchTerm
-      )
-      .forEach((collection) => {
-        void collection.fetchDocuments();
-      });
-  }, [collections, expandedNodes, searchTerm]);
-
   const isSelected = (node: number) => {
     if (!selectedNode) {
       return false;
@@ -178,8 +169,7 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
     return selectedNodeId === nodeId;
   };
 
-  const hasChildren = (node: number) =>
-    nodes[node].children.length > 0 || nodes[node].type === "collection";
+  const hasChildren = (node: number) => nodes[node].children.length > 0;
 
   const toggleCollapse = (node: number) => {
     if (!hasChildren(node)) {
@@ -229,7 +219,7 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
       } else if (doc?.isStarred) {
         icon = <StarredIcon color={theme.yellow} />;
       } else {
-        icon = <DocumentIcon color={theme.textSecondary} />;
+        icon = <DocumentIcon />;
       }
 
       path = ancestors(node)
@@ -291,14 +281,12 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
     switch (ev.key) {
       case "ArrowDown": {
         ev.preventDefault();
-        ev.stopPropagation();
         setActiveNode(next());
         scrollNodeIntoView(next());
         break;
       }
       case "ArrowUp": {
         ev.preventDefault();
-        ev.stopPropagation();
         if (activeNode === 0) {
           focusSearchInput();
         } else {
@@ -335,21 +323,16 @@ function DocumentExplorer({ onSubmit, onSelect, items }: Props) {
   const innerElementType = React.forwardRef<
     HTMLDivElement,
     React.HTMLAttributes<HTMLDivElement>
-  >(function innerElementType(
-    { style, ...rest }: React.HTMLAttributes<HTMLDivElement>,
-    ref
-  ) {
-    return (
-      <div
-        ref={ref}
-        style={{
-          ...style,
-          height: `${parseFloat(style?.height + "") + VERTICAL_PADDING * 2}px`,
-        }}
-        {...rest}
-      />
-    );
-  });
+  >(({ style, ...rest }, ref) => (
+    <div
+      ref={ref}
+      style={{
+        ...style,
+        height: `${parseFloat(style?.height + "") + VERTICAL_PADDING * 2}px`,
+      }}
+      {...rest}
+    />
+  ));
 
   return (
     <Container tabIndex={-1} onKeyDown={handleKeyDown}>

app/components/DocumentListItem.tsx

@@ -181,7 +181,8 @@ const Actions = styled(EventBoundary)`
   color: ${s("textSecondary")};
 
   ${NudeButton} {
-    &: ${hover}, &[aria-expanded= "true"] {
+    &:hover,
+    &[aria-expanded="true"] {
       background: ${s("sidebarControlHoverBackground")};
     }
   }
@@ -233,7 +234,7 @@ const DocumentLink = styled(Link)<{
     ${AnimatedStar} {
       opacity: 0.5;
 
-      &:${hover} {
+      &:hover {
         opacity: 1;
       }
     }
@@ -258,8 +259,10 @@ const Heading = styled.h3<{ rtl?: boolean }>`
   display: flex;
   justify-content: ${(props) => (props.rtl ? "flex-end" : "flex-start")};
   align-items: center;
+  height: 24px;
   margin-top: 0;
   margin-bottom: 0.25em;
+  overflow: hidden;
   white-space: nowrap;
   color: ${s("text")};
   font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Oxygen,

app/components/DocumentMeta.tsx

@@ -6,7 +6,6 @@ import { Link } from "react-router-dom";
 import styled from "styled-components";
 import { s, ellipsis } from "@shared/styles";
 import Document from "~/models/Document";
-import Revision from "~/models/Revision";
 import DocumentBreadcrumb from "~/components/DocumentBreadcrumb";
 import DocumentTasks from "~/components/DocumentTasks";
 import Flex from "~/components/Flex";
@@ -15,13 +14,11 @@ import useCurrentUser from "~/hooks/useCurrentUser";
 import useStores from "~/hooks/useStores";
 
 type Props = {
-  children?: React.ReactNode;
   showCollection?: boolean;
   showPublished?: boolean;
   showLastViewed?: boolean;
   showParentDocuments?: boolean;
   document: Document;
-  revision?: Revision;
   replace?: boolean;
   to?: LocationDescriptor;
 };
@@ -32,12 +29,11 @@ const DocumentMeta: React.FC<Props> = ({
   showLastViewed,
   showParentDocuments,
   document,
-  revision,
   children,
   replace,
   to,
   ...rest
-}: Props) => {
+}) => {
   const { t } = useTranslation();
   const { collections } = useStores();
   const user = useCurrentUser();
@@ -68,16 +64,7 @@ const DocumentMeta: React.FC<Props> = ({
   const userName = updatedBy.name;
   let content;
 
-  if (revision) {
-    content = (
-      <span>
-        {revision.createdBy?.id === user.id
-          ? t("You updated")
-          : t("{{ userName }} updated", { userName })}{" "}
-        <Time dateTime={revision.createdAt} addSuffix />
-      </span>
-    );
-  } else if (deletedAt) {
+  if (deletedAt) {
     content = (
       <span>
         {lastUpdatedByCurrentUser

app/components/DocumentViews.tsx

@@ -1,8 +1,8 @@
+import { formatDistanceToNow } from "date-fns";
 import { sortBy } from "lodash";
 import { observer } from "mobx-react";
 import * as React from "react";
 import { useTranslation } from "react-i18next";
-import { dateToRelative } from "@shared/utils/date";
 import Document from "~/models/Document";
 import User from "~/models/User";
 import Avatar from "~/components/Avatar";
@@ -33,10 +33,9 @@ function DocumentViews({ document, isOpen }: Props) {
     documentViews,
     (view) => !presentIds.includes(view.user.id)
   );
-  const users = React.useMemo(
-    () => sortedViews.map((v) => v.user),
-    [sortedViews]
-  );
+  const users = React.useMemo(() => sortedViews.map((v) => v.user), [
+    sortedViews,
+  ]);
 
   return (
     <>
@@ -53,7 +52,7 @@ function DocumentViews({ document, isOpen }: Props) {
                 ? t("Currently editing")
                 : t("Currently viewing")
               : t("Viewed {{ timeAgo }} ago", {
-                  timeAgo: dateToRelative(
+                  timeAgo: formatDistanceToNow(
                     view ? Date.parse(view.lastViewedAt) : new Date()
                   ),
                 });

app/components/DocumentsLoader.tsx

@@ -1,21 +0,0 @@
-import { observer } from "mobx-react";
-import * as React from "react";
-import Collection from "~/models/Collection";
-
-type Props = {
-  enabled: boolean;
-  collection: Collection;
-  children: React.ReactNode;
-};
-
-function DocumentsLoader({ collection, enabled, children }: Props) {
-  React.useEffect(() => {
-    if (enabled) {
-      void collection.fetchDocuments();
-    }
-  }, [collection, enabled]);
-
-  return <>{children}</>;
-}
-
-export default observer(DocumentsLoader);

app/components/Editor.tsx

@@ -1,3 +1,4 @@
+import { formatDistanceToNow } from "date-fns";
 import { deburr, difference, sortBy } from "lodash";
 import { observer } from "mobx-react";
 import { DOMParser as ProsemirrorDOMParser } from "prosemirror-model";
@@ -9,7 +10,6 @@ import { Optional } from "utility-types";
 import insertFiles from "@shared/editor/commands/insertFiles";
 import { AttachmentPreset } from "@shared/types";
 import { Heading } from "@shared/utils/ProsemirrorHelper";
-import { dateLocale, dateToRelative } from "@shared/utils/date";
 import { getDataTransferFiles } from "@shared/utils/files";
 import parseDocumentSlug from "@shared/utils/parseDocumentSlug";
 import { isInternalUrl } from "@shared/utils/urls";
@@ -23,16 +23,14 @@ import useDictionary from "~/hooks/useDictionary";
 import useEmbeds from "~/hooks/useEmbeds";
 import useStores from "~/hooks/useStores";
 import useToasts from "~/hooks/useToasts";
-import useUserLocale from "~/hooks/useUserLocale";
 import { NotFoundError } from "~/utils/errors";
 import { uploadFile } from "~/utils/files";
 import { isModKey } from "~/utils/keyboard";
-import lazyWithRetry from "~/utils/lazyWithRetry";
 import { sharedDocumentPath } from "~/utils/routeHelpers";
 import { isHash } from "~/utils/urls";
 import DocumentBreadcrumb from "./DocumentBreadcrumb";
 
-const LazyLoadedEditor = lazyWithRetry(() => import("~/editor"));
+const LazyLoadedEditor = React.lazy(() => import("~/editor"));
 
 export type Props = Optional<
   EditorProps,
@@ -61,8 +59,6 @@ function Editor(props: Props, ref: React.RefObject<SharedEditor> | null) {
     onCreateCommentMark,
     onDeleteCommentMark,
   } = props;
-  const userLocale = useUserLocale();
-  const locale = dateLocale(userLocale);
   const { auth, comments, documents } = useStores();
   const { showToast } = useToasts();
   const dictionary = useDictionary();
@@ -71,8 +67,10 @@ function Editor(props: Props, ref: React.RefObject<SharedEditor> | null) {
   const localRef = React.useRef<SharedEditor>();
   const preferences = auth.user?.preferences;
   const previousHeadings = React.useRef<Heading[] | null>(null);
-  const [activeLinkElement, setActiveLink] =
-    React.useState<HTMLAnchorElement | null>(null);
+  const [
+    activeLinkElement,
+    setActiveLink,
+  ] = React.useState<HTMLAnchorElement | null>(null);
   const previousCommentIds = React.useRef<string[]>();
 
   const handleLinkActive = React.useCallback((element: HTMLAnchorElement) => {
@@ -95,10 +93,8 @@ function Editor(props: Props, ref: React.RefObject<SharedEditor> | null) {
 
         try {
           const document = await documents.fetch(slug);
-          const time = dateToRelative(Date.parse(document.updatedAt), {
+          const time = formatDistanceToNow(Date.parse(document.updatedAt), {
             addSuffix: true,
-            shorten: true,
-            locale,
           });
 
           return [
@@ -354,6 +350,7 @@ function Editor(props: Props, ref: React.RefObject<SharedEditor> | null) {
         )}
         {activeLinkElement && !shareId && (
           <HoverPreview
+            id={props.id}
             element={activeLinkElement}
             onClose={handleLinkInactive}
           />

app/components/EventBoundary.tsx

@@ -1,11 +1,10 @@
 import * as React from "react";
 
 type Props = {
-  children?: React.ReactNode;
   className?: string;
 };
 
-const EventBoundary: React.FC<Props> = ({ children, className }: Props) => {
+const EventBoundary: React.FC<Props> = ({ children, className }) => {
   const handleClick = React.useCallback((event: React.SyntheticEvent) => {
     event.preventDefault();
     event.stopPropagation();

app/components/EventListItem.tsx

@@ -7,6 +7,7 @@ import {
   PublishIcon,
   MoveIcon,
   UnpublishIcon,
+  LightningIcon,
 } from "outline-icons";
 import * as React from "react";
 import { useTranslation } from "react-i18next";
@@ -24,7 +25,6 @@ import Item, { Actions } from "~/components/List/Item";
 import Time from "~/components/Time";
 import useStores from "~/hooks/useStores";
 import RevisionMenu from "~/menus/RevisionMenu";
-import { hover } from "~/styles";
 import Logger from "~/utils/Logger";
 import { documentHistoryPath } from "~/utils/routeHelpers";
 
@@ -51,24 +51,27 @@ const EventListItem = ({ event, latest, document, ...rest }: Props) => {
     ref.current?.focus();
   };
 
-  const prefetchRevision = async () => {
+  const prefetchRevision = () => {
     if (event.name === "revisions.create" && event.modelId) {
-      await revisions.fetch(event.modelId);
+      revisions.fetch(event.modelId);
     }
   };
 
   switch (event.name) {
     case "revisions.create":
       icon = <EditIcon size={16} />;
-      meta = latest ? (
-        <>
-          {t("Current version")} &middot; {event.actor.name}
-        </>
-      ) : (
-        t("{{userName}} edited", opts)
-      );
+      meta = t("{{userName}} edited", opts);
       to = {
-        pathname: documentHistoryPath(document, event.modelId || "latest"),
+        pathname: documentHistoryPath(document, event.modelId || ""),
+        state: { retainScrollPosition: true },
+      };
+      break;
+
+    case "documents.live_editing":
+      icon = <LightningIcon size={16} />;
+      meta = t("Latest");
+      to = {
+        pathname: documentHistoryPath(document),
         state: { retainScrollPosition: true },
       };
       break;
@@ -149,7 +152,7 @@ const EventListItem = ({ event, latest, document, ...rest }: Props) => {
         </Subtitle>
       }
       actions={
-        isRevision && isActive && event.modelId && !latest ? (
+        isRevision && isActive && event.modelId ? (
           <RevisionMenu document={document} revisionId={event.modelId} />
         ) : undefined
       }
@@ -160,16 +163,15 @@ const EventListItem = ({ event, latest, document, ...rest }: Props) => {
   );
 };
 
-const BaseItem = React.forwardRef(function _BaseItem(
-  { to, ...rest }: ItemProps,
-  ref?: React.Ref<HTMLAnchorElement>
-) {
-  if (to) {
-    return <CompositeListItem to={to} ref={ref} {...rest} />;
-  }
+const BaseItem = React.forwardRef(
+  ({ to, ...rest }: ItemProps, ref?: React.Ref<HTMLAnchorElement>) => {
+    if (to) {
+      return <CompositeListItem to={to} ref={ref} {...rest} />;
+    }
 
-  return <ListItem ref={ref} {...rest} />;
-});
+    return <ListItem ref={ref} {...rest} />;
+  }
+);
 
 const Subtitle = styled.span`
   svg {
@@ -218,7 +220,7 @@ const ItemStyle = css`
   ${Actions} {
     opacity: 0.5;
 
-    &: ${hover} {
+    &:hover {
       opacity: 1;
     }
   }

app/components/ExportDialog.tsx

@@ -21,8 +21,6 @@ function ExportDialog({ collection, onSubmit }: Props) {
   const [format, setFormat] = React.useState<FileOperationFormat>(
     FileOperationFormat.MarkdownZip
   );
-  const [includeAttachments, setIncludeAttachments] =
-    React.useState<boolean>(true);
   const user = useCurrentUser();
   const { showToast } = useToasts();
   const { collections } = useStores();
@@ -36,18 +34,11 @@ function ExportDialog({ collection, onSubmit }: Props) {
     []
   );
 
-  const handleIncludeAttachmentsChange = React.useCallback(
-    (ev: React.ChangeEvent<HTMLInputElement>) => {
-      setIncludeAttachments(ev.target.checked);
-    },
-    []
-  );
-
   const handleSubmit = async () => {
     if (collection) {
-      await collection.export(format, includeAttachments);
+      await collection.export(format);
     } else {
-      await collections.export(format, includeAttachments);
+      await collections.export(format);
     }
     onSubmit();
     showToast(t("Export started"), { type: "success" });
@@ -99,7 +90,7 @@ function ExportDialog({ collection, onSubmit }: Props) {
       )}
       <Flex gap={12} column>
         {items.map((item) => (
-          <Option key={item.value}>
+          <Option>
             <input
               type="radio"
               name="format"
@@ -116,23 +107,6 @@ function ExportDialog({ collection, onSubmit }: Props) {
           </Option>
         ))}
       </Flex>
-      <hr />
-      <Option>
-        <input
-          type="checkbox"
-          name="includeAttachments"
-          checked={includeAttachments}
-          onChange={handleIncludeAttachmentsChange}
-        />
-        <div>
-          <Text size="small" weight="bold">
-            {t("Include attachments")}
-          </Text>
-          <Text size="small">
-            {t("Including uploaded images and files in the exported data")}.
-          </Text>{" "}
-        </div>
-      </Option>
     </ConfirmationDialog>
   );
 }

app/components/Flex.tsx

@@ -28,8 +28,7 @@ const Flex = styled.div<{
   align-items: ${({ align }) => align};
   justify-content: ${({ justify }) => justify};
   flex-wrap: ${({ wrap }) => (wrap ? "wrap" : "initial")};
-  flex-shrink: ${({ shrink }) =>
-    shrink === true ? 1 : shrink === false ? 0 : "initial"};
+  flex-shrink: ${({ shrink }) => (shrink ? 1 : "initial")};
   gap: ${({ gap }) => (gap ? `${gap}px` : "initial")};
   min-height: 0;
   min-width: 0;

app/components/GroupListItem.tsx

@@ -14,7 +14,6 @@ import ListItem from "~/components/List/Item";
 import Modal from "~/components/Modal";
 import useBoolean from "~/hooks/useBoolean";
 import useStores from "~/hooks/useStores";
-import { hover } from "~/styles";
 import NudeButton from "./NudeButton";
 
 type Props = {
@@ -28,8 +27,11 @@ type Props = {
 function GroupListItem({ group, showFacepile, renderActions }: Props) {
   const { groupMemberships } = useStores();
   const { t } = useTranslation();
-  const [membersModalOpen, setMembersModalOpen, setMembersModalClosed] =
-    useBoolean();
+  const [
+    membersModalOpen,
+    setMembersModalOpen,
+    setMembersModalClosed,
+  ] = useBoolean();
   const memberCount = group.memberCount;
   const membershipsInGroup = groupMemberships.inGroup(group.id);
   const users = membershipsInGroup
@@ -85,7 +87,7 @@ const Image = styled(Flex)`
 `;
 
 const Title = styled.span`
-  &: ${hover} {
+  &:hover {
     text-decoration: underline;
     cursor: var(--pointer);
   }

app/components/Guide.tsx

@@ -6,7 +6,6 @@ import Scrollable from "~/components/Scrollable";
 import usePrevious from "~/hooks/usePrevious";
 
 type Props = {
-  children?: React.ReactNode;
   isOpen: boolean;
   title?: string;
   onRequestClose: () => void;
@@ -18,7 +17,7 @@ const Guide: React.FC<Props> = ({
   title = "Untitled",
   onRequestClose,
   ...rest
-}: Props) => {
+}) => {
   const dialog = useDialogState({
     animated: 250,
   });

app/components/HoverPreview.tsx

@@ -0,0 +1,247 @@
+import { transparentize } from "polished";
+import * as React from "react";
+import { Portal } from "react-portal";
+import styled from "styled-components";
+import { depths, s } from "@shared/styles";
+import parseDocumentSlug from "@shared/utils/parseDocumentSlug";
+import { isExternalUrl } from "@shared/utils/urls";
+import HoverPreviewDocument from "~/components/HoverPreviewDocument";
+import useMobile from "~/hooks/useMobile";
+import useStores from "~/hooks/useStores";
+import { fadeAndSlideDown } from "~/styles/animations";
+
+const DELAY_OPEN = 300;
+const DELAY_CLOSE = 300;
+
+type Props = {
+  /* The document associated with the editor, if any */
+  id?: string;
+  /* The HTML element that is being hovered over */
+  element: HTMLAnchorElement;
+  /* A callback on close of the hover preview */
+  onClose: () => void;
+};
+
+function HoverPreviewInternal({ element, id, onClose }: Props) {
+  const { documents } = useStores();
+  const slug = parseDocumentSlug(element.href);
+  const [isVisible, setVisible] = React.useState(false);
+  const timerClose = React.useRef<ReturnType<typeof setTimeout>>();
+  const timerOpen = React.useRef<ReturnType<typeof setTimeout>>();
+  const cardRef = React.useRef<HTMLDivElement>(null);
+
+  const startCloseTimer = () => {
+    stopOpenTimer();
+    timerClose.current = setTimeout(() => {
+      if (isVisible) {
+        setVisible(false);
+      }
+      onClose();
+    }, DELAY_CLOSE);
+  };
+
+  const stopCloseTimer = () => {
+    if (timerClose.current) {
+      clearTimeout(timerClose.current);
+    }
+  };
+
+  const startOpenTimer = () => {
+    timerOpen.current = setTimeout(() => setVisible(true), DELAY_OPEN);
+  };
+
+  const stopOpenTimer = () => {
+    if (timerOpen.current) {
+      clearTimeout(timerOpen.current);
+    }
+  };
+
+  React.useEffect(() => {
+    if (slug) {
+      documents.prefetchDocument(slug);
+    }
+
+    startOpenTimer();
+
+    if (cardRef.current) {
+      cardRef.current.addEventListener("mouseenter", stopCloseTimer);
+    }
+
+    if (cardRef.current) {
+      cardRef.current.addEventListener("mouseleave", startCloseTimer);
+    }
+
+    element.addEventListener("mouseout", startCloseTimer);
+    element.addEventListener("mouseover", stopCloseTimer);
+    element.addEventListener("mouseover", startOpenTimer);
+    return () => {
+      element.removeEventListener("mouseout", startCloseTimer);
+      element.removeEventListener("mouseover", stopCloseTimer);
+      element.removeEventListener("mouseover", startOpenTimer);
+
+      if (cardRef.current) {
+        cardRef.current.removeEventListener("mouseenter", stopCloseTimer);
+      }
+
+      if (cardRef.current) {
+        cardRef.current.removeEventListener("mouseleave", startCloseTimer);
+      }
+
+      if (timerClose.current) {
+        clearTimeout(timerClose.current);
+      }
+    };
+  }, [element, slug]);
+
+  const anchorBounds = element.getBoundingClientRect();
+  const cardBounds = cardRef.current?.getBoundingClientRect();
+  const left = cardBounds
+    ? Math.min(anchorBounds.left, window.innerWidth - 16 - 350)
+    : anchorBounds.left;
+  const leftOffset = anchorBounds.left - left;
+
+  return (
+    <Portal>
+      <Position
+        top={anchorBounds.bottom + window.scrollY}
+        left={left}
+        aria-hidden
+      >
+        <div ref={cardRef}>
+          <HoverPreviewDocument url={element.href} id={id}>
+            {(content: React.ReactNode) =>
+              isVisible ? (
+                <Animate>
+                  <Card>
+                    <Margin />
+                    <CardContent>{content}</CardContent>
+                  </Card>
+                  <Pointer offset={leftOffset + anchorBounds.width / 2} />
+                </Animate>
+              ) : null
+            }
+          </HoverPreviewDocument>
+        </div>
+      </Position>
+    </Portal>
+  );
+}
+
+function HoverPreview({ element, ...rest }: Props) {
+  const isMobile = useMobile();
+  if (isMobile) {
+    return null;
+  }
+
+  // previews only work for internal doc links for now
+  if (isExternalUrl(element.href)) {
+    return null;
+  }
+
+  return <HoverPreviewInternal {...rest} element={element} />;
+}
+
+const Animate = styled.div`
+  animation: ${fadeAndSlideDown} 150ms ease;
+
+  @media print {
+    display: none;
+  }
+`;
+
+// fills the gap between the card and pointer to avoid a dead zone
+const Margin = styled.div`
+  position: absolute;
+  top: -11px;
+  left: 0;
+  right: 0;
+  height: 11px;
+`;
+
+const CardContent = styled.div`
+  overflow: hidden;
+  max-height: 20em;
+  user-select: none;
+`;
+
+// &:after — gradient mask for overflow text
+const Card = styled.div`
+  backdrop-filter: blur(10px);
+  background: ${s("background")};
+  border-radius: 4px;
+  box-shadow: 0 30px 90px -20px rgba(0, 0, 0, 0.3),
+    0 0 1px 1px rgba(0, 0, 0, 0.05);
+  padding: 16px;
+  width: 350px;
+  font-size: 0.9em;
+  position: relative;
+
+  .placeholder,
+  .heading-anchor {
+    display: none;
+  }
+
+  &:after {
+    content: "";
+    display: block;
+    position: absolute;
+    pointer-events: none;
+    background: linear-gradient(
+      90deg,
+      ${(props) => transparentize(1, props.theme.background)} 0%,
+      ${(props) => transparentize(1, props.theme.background)} 75%,
+      ${s("background")} 90%
+    );
+    bottom: 0;
+    left: 0;
+    right: 0;
+    height: 1.7em;
+    border-bottom: 16px solid ${s("background")};
+    border-bottom-left-radius: 4px;
+    border-bottom-right-radius: 4px;
+  }
+`;
+
+const Position = styled.div<{ fixed?: boolean; top?: number; left?: number }>`
+  margin-top: 10px;
+  position: ${({ fixed }) => (fixed ? "fixed" : "absolute")};
+  z-index: ${depths.hoverPreview};
+  display: flex;
+  max-height: 75%;
+
+  ${({ top }) => (top !== undefined ? `top: ${top}px` : "")};
+  ${({ left }) => (left !== undefined ? `left: ${left}px` : "")};
+`;
+
+const Pointer = styled.div<{ offset: number }>`
+  top: -22px;
+  left: ${(props) => props.offset}px;
+  width: 22px;
+  height: 22px;
+  position: absolute;
+  transform: translateX(-50%);
+  pointer-events: none;
+
+  &:before,
+  &:after {
+    content: "";
+    display: inline-block;
+    position: absolute;
+    bottom: 0;
+    right: 0;
+  }
+
+  &:before {
+    border: 8px solid transparent;
+    border-bottom-color: ${(props) =>
+      props.theme.menuBorder || "rgba(0, 0, 0, 0.1)"};
+    right: -1px;
+  }
+
+  &:after {
+    border: 7px solid transparent;
+    border-bottom-color: ${s("background")};
+  }
+`;
+
+export default HoverPreview;

app/components/HoverPreview/Components.tsx

@@ -1,108 +0,0 @@
-import { transparentize } from "polished";
-import { Link } from "react-router-dom";
-import styled, { css } from "styled-components";
-import { s } from "@shared/styles";
-import Text from "~/components/Text";
-
-export const CARD_MARGIN = 16;
-
-const NUMBER_OF_LINES = 10;
-
-const sharedVars = css`
-  --line-height: 1.6em;
-`;
-
-const StyledText = styled(Text)`
-  margin-bottom: 0;
-`;
-
-export const Preview = styled(Link)`
-  cursor: ${(props: any) =>
-    props.as === "div" ? "default" : "var(--pointer)"};
-  border-radius: 4px;
-  box-shadow: 0 30px 90px -20px rgba(0, 0, 0, 0.3),
-    0 0 1px 1px rgba(0, 0, 0, 0.05);
-  overflow: hidden;
-  position: absolute;
-  min-width: 350px;
-  max-width: 375px;
-`;
-
-export const Title = styled.h2`
-  font-size: 1.25em;
-  margin: 0;
-  color: ${s("text")};
-`;
-
-export const Info = styled(StyledText).attrs(() => ({
-  type: "tertiary",
-  size: "xsmall",
-}))`
-  white-space: nowrap;
-`;
-
-export const Description = styled(StyledText)`
-  ${sharedVars}
-  margin-top: 0.5em;
-  line-height: var(--line-height);
-  max-height: calc(var(--line-height) * ${NUMBER_OF_LINES});
-`;
-
-export const Thumbnail = styled.img`
-  object-fit: cover;
-  height: 200px;
-  background: ${s("menuBackground")};
-`;
-
-export const CardContent = styled.div`
-  overflow: hidden;
-  user-select: none;
-`;
-
-// &:after — gradient mask for overflow text
-export const Card = styled.div<{ fadeOut?: boolean; $borderRadius?: string }>`
-  backdrop-filter: blur(10px);
-  background: ${s("menuBackground")};
-  padding: 16px;
-  font-size: 0.9em;
-  position: relative;
-
-  .placeholder,
-  .heading-anchor {
-    display: none;
-  }
-
-  // fills the gap between the card and pointer to avoid a dead zone
-  &::before {
-    content: "";
-    position: absolute;
-    top: -10px;
-    left: 0;
-    right: 0;
-    height: 10px;
-  }
-
-  ${(props) =>
-    props.fadeOut !== false
-      ? `&:after {
-          ${sharedVars}
-          content: "";
-          display: block;
-          position: absolute;
-          pointer-events: none;
-          background: linear-gradient(
-            90deg,
-            ${transparentize(1, props.theme.menuBackground)} 0%,
-            ${transparentize(1, props.theme.menuBackground)} 75%,
-            ${props.theme.menuBackground} 90%
-          );
-          bottom: 0;
-          left: 0;
-          right: 0;
-          height: var(--line-height);
-          border-bottom: 16px solid ${props.theme.menuBackground};
-          border-bottom-left-radius: 4px;
-          border-bottom-right-radius: 4px;
-        }`
-      : ""}
-`;

app/components/HoverPreview/HoverPreview.tsx

@@ -1,261 +0,0 @@
-import { m } from "framer-motion";
-import * as React from "react";
-import { Portal } from "react-portal";
-import styled from "styled-components";
-import { depths, s } from "@shared/styles";
-import { UnfurlType } from "@shared/types";
-import LoadingIndicator from "~/components/LoadingIndicator";
-import useEventListener from "~/hooks/useEventListener";
-import useKeyDown from "~/hooks/useKeyDown";
-import useMobile from "~/hooks/useMobile";
-import useOnClickOutside from "~/hooks/useOnClickOutside";
-import useRequest from "~/hooks/useRequest";
-import useStores from "~/hooks/useStores";
-import { client } from "~/utils/ApiClient";
-import { CARD_MARGIN } from "./Components";
-import HoverPreviewDocument from "./HoverPreviewDocument";
-import HoverPreviewLink from "./HoverPreviewLink";
-import HoverPreviewMention from "./HoverPreviewMention";
-
-const DELAY_OPEN = 300;
-const DELAY_CLOSE = 600;
-
-type Props = {
-  /* The HTML element that is being hovered over */
-  element: HTMLAnchorElement;
-  /* A callback on close of the hover preview */
-  onClose: () => void;
-};
-
-function HoverPreviewInternal({ element, onClose }: Props) {
-  const url = element.href || element.dataset.url;
-  const [isVisible, setVisible] = React.useState(false);
-  const timerClose = React.useRef<ReturnType<typeof setTimeout>>();
-  const timerOpen = React.useRef<ReturnType<typeof setTimeout>>();
-  const cardRef = React.useRef<HTMLDivElement>(null);
-  const stores = useStores();
-  const [cardLeft, setCardLeft] = React.useState(0);
-  const [cardTop, setCardTop] = React.useState(0);
-  const [pointerOffset, setPointerOffset] = React.useState(0);
-
-  React.useLayoutEffect(() => {
-    if (isVisible && cardRef.current) {
-      const elem = element.getBoundingClientRect();
-      const card = cardRef.current.getBoundingClientRect();
-
-      const top = elem.bottom + window.scrollY;
-      setCardTop(top);
-
-      let left = elem.left;
-      let pointerOffset = elem.width / 2;
-      if (left + card.width > window.innerWidth) {
-        // shift card leftwards by the amount it went out of screen
-        let shiftBy = left + card.width - window.innerWidth;
-        // shift a littler further to leave some margin between card and window boundary
-        shiftBy += CARD_MARGIN;
-        left -= shiftBy;
-
-        // shift pointer rightwards by same amount so as to position it back correctly
-        pointerOffset += shiftBy;
-      }
-      setCardLeft(left);
-
-      setPointerOffset(pointerOffset);
-    }
-  }, [isVisible, element]);
-
-  const { data, request, loading } = useRequest(
-    React.useCallback(
-      () =>
-        client.post("/urls.unfurl", {
-          url,
-          documentId: stores.ui.activeDocumentId,
-        }),
-      [url, stores.ui.activeDocumentId]
-    )
-  );
-
-  React.useEffect(() => {
-    if (url) {
-      stopOpenTimer();
-      setVisible(false);
-
-      void request();
-    }
-  }, [url, request]);
-
-  const stopOpenTimer = () => {
-    if (timerOpen.current) {
-      clearTimeout(timerOpen.current);
-      timerOpen.current = undefined;
-    }
-  };
-
-  const closePreview = React.useCallback(() => {
-    if (isVisible) {
-      stopOpenTimer();
-      setVisible(false);
-      onClose();
-    }
-  }, [isVisible, onClose]);
-
-  useOnClickOutside(cardRef, closePreview);
-  useKeyDown("Escape", closePreview);
-  useEventListener("scroll", closePreview, window, { capture: true });
-
-  const stopCloseTimer = () => {
-    if (timerClose.current) {
-      clearTimeout(timerClose.current);
-      timerClose.current = undefined;
-    }
-  };
-
-  const startOpenTimer = () => {
-    if (!timerOpen.current) {
-      timerOpen.current = setTimeout(() => setVisible(true), DELAY_OPEN);
-    }
-  };
-
-  const startCloseTimer = React.useCallback(() => {
-    stopOpenTimer();
-    timerClose.current = setTimeout(closePreview, DELAY_CLOSE);
-  }, [closePreview]);
-
-  React.useEffect(() => {
-    const card = cardRef.current;
-
-    if (data) {
-      startOpenTimer();
-
-      if (card) {
-        card.addEventListener("mouseenter", stopCloseTimer);
-        card.addEventListener("mouseleave", startCloseTimer);
-      }
-
-      element.addEventListener("mouseout", startCloseTimer);
-      element.addEventListener("mouseover", stopCloseTimer);
-      element.addEventListener("mouseover", startOpenTimer);
-    }
-
-    return () => {
-      element.removeEventListener("mouseout", startCloseTimer);
-      element.removeEventListener("mouseover", stopCloseTimer);
-      element.removeEventListener("mouseover", startOpenTimer);
-
-      if (card) {
-        card.removeEventListener("mouseenter", stopCloseTimer);
-        card.removeEventListener("mouseleave", startCloseTimer);
-      }
-
-      stopCloseTimer();
-    };
-  }, [element, startCloseTimer, data]);
-
-  if (loading) {
-    return <LoadingIndicator />;
-  }
-
-  if (!data) {
-    return null;
-  }
-
-  return (
-    <Portal>
-      <Position top={cardTop} left={cardLeft} aria-hidden>
-        {isVisible ? (
-          <Animate
-            initial={{ opacity: 0, y: -20, pointerEvents: "none" }}
-            animate={{ opacity: 1, y: 0, pointerEvents: "auto" }}
-          >
-            {data.type === UnfurlType.Mention ? (
-              <HoverPreviewMention
-                ref={cardRef}
-                url={data.thumbnailUrl}
-                title={data.title}
-                info={data.meta.info}
-                color={data.meta.color}
-              />
-            ) : data.type === UnfurlType.Document ? (
-              <HoverPreviewDocument
-                ref={cardRef}
-                id={data.meta.id}
-                url={data.url}
-                title={data.title}
-                description={data.description}
-                info={data.meta.info}
-              />
-            ) : (
-              <HoverPreviewLink
-                ref={cardRef}
-                url={data.url}
-                thumbnailUrl={data.thumbnailUrl}
-                title={data.title}
-                description={data.description}
-              />
-            )}
-            <Pointer offset={pointerOffset} />
-          </Animate>
-        ) : null}
-      </Position>
-    </Portal>
-  );
-}
-
-function HoverPreview({ element, ...rest }: Props) {
-  const isMobile = useMobile();
-  if (isMobile) {
-    return null;
-  }
-
-  return <HoverPreviewInternal {...rest} element={element} />;
-}
-
-const Animate = styled(m.div)`
-  @media print {
-    display: none;
-  }
-`;
-
-const Position = styled.div<{ fixed?: boolean; top?: number; left?: number }>`
-  margin-top: 10px;
-  position: ${({ fixed }) => (fixed ? "fixed" : "absolute")};
-  z-index: ${depths.hoverPreview};
-  display: flex;
-  max-height: 75%;
-
-  ${({ top }) => (top !== undefined ? `top: ${top}px` : "")};
-  ${({ left }) => (left !== undefined ? `left: ${left}px` : "")};
-`;
-
-const Pointer = styled.div<{ offset: number }>`
-  top: -22px;
-  left: ${(props) => props.offset}px;
-  width: 22px;
-  height: 22px;
-  position: absolute;
-  transform: translateX(-50%);
-  pointer-events: none;
-
-  &:before,
-  &:after {
-    content: "";
-    display: inline-block;
-    position: absolute;
-    bottom: 0;
-    right: 0;
-  }
-
-  &:before {
-    border: 8px solid transparent;
-    border-bottom-color: ${(props) =>
-      props.theme.menuBorder || "rgba(0, 0, 0, 0.1)"};
-    right: -1px;
-  }
-
-  &:after {
-    border: 7px solid transparent;
-    border-bottom-color: ${s("menuBackground")};
-  }
-`;
-
-export default HoverPreview;

app/components/HoverPreview/HoverPreviewDocument.tsx

@@ -1,54 +0,0 @@
-import * as React from "react";
-import Editor from "~/components/Editor";
-import Flex from "~/components/Flex";
-import {
-  Preview,
-  Title,
-  Info,
-  Card,
-  CardContent,
-  Description,
-} from "./Components";
-
-type Props = {
-  /** Document id associated with the editor, if any */
-  id?: string;
-  /** Document url */
-  url: string;
-  /** Title for the preview card */
-  title: string;
-  /** Info about last activity on the document */
-  info: string;
-  /** Text preview of document content */
-  description: string;
-};
-
-const HoverPreviewDocument = React.forwardRef(function _HoverPreviewDocument(
-  { id, url, title, info, description }: Props,
-  ref: React.Ref<HTMLDivElement>
-) {
-  return (
-    <Preview to={url}>
-      <Card ref={ref}>
-        <CardContent>
-          <Flex column gap={2}>
-            <Title>{title}</Title>
-            <Info>{info}</Info>
-            <Description as="div">
-              <React.Suspense fallback={<div />}>
-                <Editor
-                  key={id}
-                  defaultValue={description}
-                  embedsDisabled
-                  readOnly
-                />
-              </React.Suspense>
-            </Description>
-          </Flex>
-        </CardContent>
-      </Card>
-    </Preview>
-  );
-});
-
-export default HoverPreviewDocument;

app/components/HoverPreview/HoverPreviewLink.tsx

@@ -1,44 +0,0 @@
-import * as React from "react";
-import Flex from "~/components/Flex";
-import {
-  Preview,
-  Title,
-  Description,
-  Card,
-  CardContent,
-  Thumbnail,
-} from "./Components";
-
-type Props = {
-  /** Link url */
-  url: string;
-  /** Title for the preview card */
-  title: string;
-  /** Url for thumbnail served by the link provider */
-  thumbnailUrl: string;
-  /** Some description about the link provider */
-  description: string;
-};
-
-const HoverPreviewLink = React.forwardRef(function _HoverPreviewLink(
-  { url, thumbnailUrl, title, description }: Props,
-  ref: React.Ref<HTMLDivElement>
-) {
-  return (
-    <Preview as="a" href={url} target="_blank" rel="noopener noreferrer">
-      <Flex column>
-        {thumbnailUrl ? <Thumbnail src={thumbnailUrl} alt={""} /> : null}
-        <Card ref={ref}>
-          <CardContent>
-            <Flex column>
-              <Title>{title}</Title>
-              <Description>{description}</Description>
-            </Flex>
-          </CardContent>
-        </Card>
-      </Flex>
-    </Preview>
-  );
-});
-
-export default HoverPreviewLink;

app/components/HoverPreview/HoverPreviewMention.tsx

@@ -1,46 +0,0 @@
-import * as React from "react";
-import Avatar from "~/components/Avatar";
-import { AvatarSize } from "~/components/Avatar/Avatar";
-import Flex from "~/components/Flex";
-import { Preview, Title, Info, Card, CardContent } from "./Components";
-
-type Props = {
-  /** Resource url, avatar url in case of user mention */
-  url: string;
-  /** Title for the preview card*/
-  title: string;
-  /** Info about mentioned user's recent activity */
-  info: string;
-  /** Used for avatar's background color in absence of avatar url */
-  color: string;
-};
-
-const HoverPreviewMention = React.forwardRef(function _HoverPreviewMention(
-  { url, title, info, color }: Props,
-  ref: React.Ref<HTMLDivElement>
-) {
-  return (
-    <Preview as="div">
-      <Card fadeOut={false} ref={ref}>
-        <CardContent>
-          <Flex gap={12}>
-            <Avatar
-              model={{
-                avatarUrl: url,
-                initial: title ? title[0] : "?",
-                color,
-              }}
-              size={AvatarSize.XLarge}
-            />
-            <Flex column gap={2} justify="center">
-              <Title>{title}</Title>
-              <Info>{info}</Info>
-            </Flex>
-          </Flex>
-        </CardContent>
-      </Card>
-    </Preview>
-  );
-});
-
-export default HoverPreviewMention;

app/components/HoverPreview/index.ts

@@ -1,3 +0,0 @@
-import HoverPreview from "./HoverPreview";
-
-export default HoverPreview;

app/components/HoverPreviewDocument.tsx

@@ -0,0 +1,62 @@
+import { observer } from "mobx-react";
+import * as React from "react";
+import { Link } from "react-router-dom";
+import styled from "styled-components";
+import { s } from "@shared/styles";
+import parseDocumentSlug from "@shared/utils/parseDocumentSlug";
+import DocumentMeta from "~/components/DocumentMeta";
+import Editor from "~/components/Editor";
+import useStores from "~/hooks/useStores";
+
+type Props = {
+  /* The document associated with the editor, if any */
+  id?: string;
+  /* The URL we want a preview for */
+  url: string;
+  children: (content: React.ReactNode) => React.ReactNode;
+};
+
+function HoverPreviewDocument({ url, id, children }: Props) {
+  const { documents } = useStores();
+  const slug = parseDocumentSlug(url);
+
+  if (slug) {
+    documents.prefetchDocument(slug);
+  }
+
+  const document = slug ? documents.getByUrl(slug) : undefined;
+  if (!document || document.id === id) {
+    return null;
+  }
+
+  return (
+    <>
+      {children(
+        <Content to={document.url}>
+          <Heading>{document.titleWithDefault}</Heading>
+          <DocumentMeta document={document} />
+
+          <React.Suspense fallback={<div />}>
+            <Editor
+              key={document.id}
+              defaultValue={document.getSummary()}
+              embedsDisabled
+              readOnly
+            />
+          </React.Suspense>
+        </Content>
+      )}
+    </>
+  );
+}
+
+const Content = styled(Link)`
+  cursor: var(--pointer);
+`;
+
+const Heading = styled.h2`
+  margin: 0 0 0.75em;
+  color: ${s("text")};
+`;
+
+export default observer(HoverPreviewDocument);

app/components/IconPicker.tsx

@@ -47,7 +47,6 @@ import Flex from "~/components/Flex";
 import { LabelText } from "~/components/Input";
 import NudeButton from "~/components/NudeButton";
 import Text from "~/components/Text";
-import lazyWithRetry from "~/utils/lazyWithRetry";
 import DelayedMount from "./DelayedMount";
 
 const style = {
@@ -55,7 +54,7 @@ const style = {
   height: 30,
 };
 
-const TwitterPicker = lazyWithRetry(
+const TwitterPicker = React.lazy(
   () => import("react-color/lib/components/twitter/Twitter")
 );
 

app/components/Icons/MarkdownIcon.tsx

@@ -24,7 +24,7 @@ export default function MarkdownIcon({
       <path
         d="M19.2692 7H3.86538C3.38745 7 3 7.38476 3 7.85938V16.2812C3 16.7559 3.38745 17.1406 3.86538 17.1406H19.2692C19.7472 17.1406 20.1346 16.7559 20.1346 16.2812V7.85938C20.1346 7.38476 19.7472 7 19.2692 7Z"
         stroke={color}
-        strokeWidth="2"
+        stroke-width="2"
       />
       <path
         d="M5.16345 14.9922V9.14844H6.89422L8.62499 11.2969L10.3558 9.14844H12.0865V14.9922H10.3558V11.6406L8.62499 13.7891L6.89422 11.6406V14.9922H5.16345ZM15.9808 14.9922L13.3846 12.1562H15.1154V9.14844H16.8461V12.1562H18.5769L15.9808 14.9922Z"

app/components/Input.tsx

@@ -30,8 +30,6 @@ const RealInput = styled.input<{ hasIcon?: boolean }>`
   color: ${s("text")};
   height: 30px;
   min-width: 0;
-  font-size: 15px;
-
   ${ellipsis()}
   ${undraggableOnDesktop()}
 
@@ -177,8 +175,6 @@ function Input(
     labelHidden,
     onFocus,
     onBlur,
-    onRequestSubmit,
-    children,
     ...rest
   } = props;
 
@@ -215,7 +211,6 @@ function Input(
               {...rest}
             />
           )}
-          {children}
         </Outline>
       </label>
       {error && (

app/components/InputColor.tsx

@@ -3,7 +3,6 @@ import { useTranslation } from "react-i18next";
 import { MenuButton, useMenuState } from "reakit/Menu";
 import styled from "styled-components";
 import { s } from "@shared/styles";
-import lazyWithRetry from "~/utils/lazyWithRetry";
 import ContextMenu from "./ContextMenu";
 import DelayedMount from "./DelayedMount";
 import Input, { Props as InputProps } from "./Input";
@@ -16,7 +15,7 @@ type Props = Omit<InputProps, "onChange"> & {
   onChange: (value: string) => void;
 };
 
-const InputColor: React.FC<Props> = ({ value, onChange, ...rest }: Props) => {
+const InputColor: React.FC<Props> = ({ value, onChange, ...rest }) => {
   const { t } = useTranslation();
   const menu = useMenuState({
     modal: true,
@@ -27,7 +26,7 @@ const InputColor: React.FC<Props> = ({ value, onChange, ...rest }: Props) => {
     <Relative>
       <Input
         value={value}
-        onChange={(event) => onChange(event.target.value.replace(/^#?/, "#"))}
+        onChange={(event) => onChange(event.target.value)}
         placeholder="#"
         maxLength={7}
         {...rest}
@@ -69,7 +68,7 @@ const SwatchButton = styled(NudeButton)<{ $background: string | undefined }>`
   right: 6px;
 `;
 
-const ColorPicker = lazyWithRetry(
+const ColorPicker = React.lazy(
   () => import("react-color/lib/components/chrome/Chrome")
 );
 

app/components/InputSearchPage.tsx

@@ -45,10 +45,6 @@ function InputSearchPage({
 
   const handleKeyDown = React.useCallback(
     (ev: React.KeyboardEvent<HTMLInputElement>) => {
-      if (ev.nativeEvent.isComposing) {
-        return;
-      }
-
       if (ev.key === "Enter") {
         ev.preventDefault();
         history.push(

app/components/InputSelect.tsx

@@ -85,11 +85,11 @@ const InputSelect = (props: Props) => {
   const contentRef = React.useRef<HTMLDivElement>(null);
   const minWidth = buttonRef.current?.offsetWidth || 0;
   const margin = 8;
-  const menuMaxHeight = useMenuHeight({
-    visible: select.visible,
-    elementRef: select.unstable_disclosureRef,
-    margin,
-  });
+  const menuMaxHeight = useMenuHeight(
+    select.visible,
+    select.unstable_disclosureRef,
+    margin
+  );
   const maxHeight = Math.min(
     menuMaxHeight ?? 0,
     window.innerHeight -
@@ -108,7 +108,11 @@ const InputSelect = (props: Props) => {
     }
     previousValue.current = select.selectedValue;
 
-    onChange?.(select.selectedValue);
+    async function load() {
+      await onChange?.(select.selectedValue);
+    }
+
+    load();
   }, [onChange, select.selectedValue]);
 
   React.useLayoutEffect(() => {

app/components/Labeled.tsx

@@ -5,11 +5,10 @@ import { s } from "@shared/styles";
 import Flex from "~/components/Flex";
 
 type Props = {
-  children?: React.ReactNode;
   label: React.ReactNode | string;
 };
 
-const Labeled: React.FC<Props> = ({ label, children, ...props }: Props) => (
+const Labeled: React.FC<Props> = ({ label, children, ...props }) => (
   <Flex column {...props}>
     <Label>{label}</Label>
     {children}

app/components/LanguagePrompt.tsx

@@ -21,14 +21,14 @@ function Icon({ className }: { className?: string }) {
       className={className}
     >
       <path
-        fillRule="evenodd"
-        clipRule="evenodd"
+        fill-rule="evenodd"
+        clip-rule="evenodd"
         d="M21 18H16L14 16V6C14 4.89543 14.8954 4 16 4H28C29.1046 4 30 4.89543 30 6V16C30 17.1046 29.1046 18 28 18H27L25.4142 19.5858C24.6332 20.3668 23.3668 20.3668 22.5858 19.5858L21 18ZM16 15.1716V6H28V16H27H26.1716L25.5858 16.5858L24 18.1716L22.4142 16.5858L21.8284 16H21H16.8284L16 15.1716Z"
         fill="#2B2F35"
       />
       <path
-        fillRule="evenodd"
-        clipRule="evenodd"
+        fill-rule="evenodd"
+        clip-rule="evenodd"
         d="M16 13H4C2.89543 13 2 13.8954 2 15V25C2 26.1046 2.89543 27 4 27H5L6.58579 28.5858C7.36684 29.3668 8.63316 29.3668 9.41421 28.5858L11 27H16C17.1046 27 18 26.1046 18 25V15C18 13.8954 17.1046 13 16 13ZM9 17L6 16.9681C6 16.9681 5 17.016 5 18C5 18.984 6 19 6 19H8.5H10C10 19 9.57627 20.1885 8.38983 21.0831C7.20339 21.9777 5.7197 23 5.7197 23C5.7197 23 4.99153 23.6054 5.5 24.5C6.00847 25.3946 7 24.8403 7 24.8403L9.74576 22.8722L11.9492 24.6614C11.9492 24.6614 12.6271 25.3771 13.3051 24.4825C13.9831 23.5879 13.3051 23.0512 13.3051 23.0512L11.1017 21.262C11.1017 21.262 11.5 21 12 20L12.5 19H14C14 19 15 19.0319 15 18C15 16.9681 14 16.9681 14 16.9681L11 17V16C11 16 11.0169 15 10 15C8.98305 15 9 16 9 16V17Z"
         fill="#2B2F35"
       />
@@ -73,11 +73,11 @@ export default function LanguagePrompt() {
           </Trans>
           <br />
           <Link
-            onClick={async () => {
-              ui.setLanguagePromptDismissed();
-              await auth.updateUser({
+            onClick={() => {
+              auth.updateUser({
                 language,
               });
+              ui.setLanguagePromptDismissed();
             }}
           >
             {t("Change Language")}

app/components/Layout.tsx

@@ -16,7 +16,6 @@ import useStores from "~/hooks/useStores";
 import { isModKey } from "~/utils/keyboard";
 
 type Props = {
-  children?: React.ReactNode;
   title?: string;
   sidebar?: React.ReactNode;
   sidebarRight?: React.ReactNode;
@@ -27,7 +26,7 @@ const Layout: React.FC<Props> = ({
   children,
   sidebar,
   sidebarRight,
-}: Props) => {
+}) => {
   const { ui } = useStores();
   const sidebarCollapsed = !sidebar || ui.sidebarIsClosed;
 

app/components/LazyPolyfills.tsx

@@ -1,25 +1,16 @@
 import * as React from "react";
-import Logger from "~/utils/Logger";
 import { loadPolyfills } from "~/utils/polyfills";
 
-type Props = {
-  children?: React.ReactNode;
-};
-
 /**
  * Asyncronously load required polyfills. Should wrap the React tree.
  */
-export const LazyPolyfill: React.FC = ({ children }: Props) => {
+export const LazyPolyfill: React.FC = ({ children }) => {
   const [isLoaded, setIsLoaded] = React.useState(false);
 
   React.useEffect(() => {
-    loadPolyfills()
-      .then(() => {
-        setIsLoaded(true);
-      })
-      .catch((error) => {
-        Logger.error("Polyfills failed to load", error);
-      });
+    loadPolyfills().then(() => {
+      setIsLoaded(true);
+    });
   }, []);
 
   if (!isLoaded) {

app/components/LocaleTime.tsx

@@ -1,8 +1,8 @@
-import { format as formatDate } from "date-fns";
+import { format as formatDate, formatDistanceToNow } from "date-fns";
 import * as React from "react";
-import { dateLocale, dateToRelative, locales } from "@shared/utils/date";
 import Tooltip from "~/components/Tooltip";
 import useUserLocale from "~/hooks/useUserLocale";
+import { dateLocale, locales } from "~/utils/i18n";
 
 let callbacks: (() => void)[] = [];
 
@@ -21,7 +21,6 @@ function eachMinute(fn: () => void) {
 }
 
 type Props = {
-  children?: React.ReactNode;
   dateTime: string;
   tooltipDelay?: number;
   addSuffix?: boolean;
@@ -38,7 +37,7 @@ const LocaleTime: React.FC<Props> = ({
   format,
   relative,
   tooltipDelay,
-}: Props) => {
+}) => {
   const userLocale: string = useUserLocale() || "";
   const dateFormatLong = {
     en_US: "MMMM do, yyyy h:mm a",
@@ -60,21 +59,26 @@ const LocaleTime: React.FC<Props> = ({
     };
   }, []);
 
-  const date = new Date(Date.parse(dateTime));
   const locale = dateLocale(userLocale);
-  const relativeContent = dateToRelative(date, {
+  let relativeContent = formatDistanceToNow(Date.parse(dateTime), {
     addSuffix,
     locale,
-    shorten,
   });
 
-  const tooltipContent = formatDate(date, formatLocaleLong, {
+  if (shorten) {
+    relativeContent = relativeContent
+      .replace("about", "")
+      .replace("less than a minute ago", "just now")
+      .replace("minute", "min");
+  }
+
+  const tooltipContent = formatDate(Date.parse(dateTime), formatLocaleLong, {
     locale,
   });
   const content =
     relative !== false
       ? relativeContent
-      : formatDate(date, formatLocale, {
+      : formatDate(Date.parse(dateTime), formatLocale, {
           locale,
         });
 

app/components/Modal.tsx

@@ -19,9 +19,7 @@ import Desktop from "~/utils/Desktop";
 import ErrorBoundary from "./ErrorBoundary";
 
 let openModals = 0;
-
 type Props = {
-  children?: React.ReactNode;
   isOpen: boolean;
   isCentered?: boolean;
   title?: React.ReactNode;
@@ -34,7 +32,7 @@ const Modal: React.FC<Props> = ({
   isCentered,
   title = "Untitled",
   onRequestClose,
-}: Props) => {
+}) => {
   const dialog = useDialogState({
     animated: 250,
   });

app/components/Notice.tsx

@@ -5,12 +5,11 @@ import Flex from "./Flex";
 import Text from "./Text";
 
 type Props = {
-  children?: React.ReactNode;
   icon?: JSX.Element;
   description?: JSX.Element;
 };
 
-const Notice: React.FC<Props> = ({ children, icon, description }: Props) => (
+const Notice: React.FC<Props> = ({ children, icon, description }) => (
   <Container>
     <Flex as="span" gap={8}>
       {icon}

app/components/Notifications/NotificationIcon.tsx

@@ -1,32 +0,0 @@
-import { observer } from "mobx-react";
-import { SubscribeIcon } from "outline-icons";
-import * as React from "react";
-import styled, { useTheme } from "styled-components";
-import { s } from "@shared/styles";
-import useStores from "~/hooks/useStores";
-import Relative from "../Sidebar/components/Relative";
-
-const NotificationIcon = () => {
-  const { notifications } = useStores();
-  const theme = useTheme();
-  const count = notifications.approximateUnreadCount;
-
-  return (
-    <Relative style={{ height: 24 }}>
-      <SubscribeIcon color={theme.textTertiary} />
-      {count > 0 && <Badge />}
-    </Relative>
-  );
-};
-
-const Badge = styled.div`
-  position: absolute;
-  width: 8px;
-  height: 8px;
-  border-radius: 50%;
-  background: ${s("accent")};
-  top: 0;
-  right: 0;
-`;
-
-export default observer(NotificationIcon);

app/components/Notifications/NotificationListItem.tsx

@@ -1,110 +0,0 @@
-import { toJS } from "mobx";
-import { observer } from "mobx-react";
-import * as React from "react";
-import { useTranslation } from "react-i18next";
-import { Link } from "react-router-dom";
-import styled from "styled-components";
-import { s } from "@shared/styles";
-import Notification from "~/models/Notification";
-import CommentEditor from "~/scenes/Document/components/CommentEditor";
-import useStores from "~/hooks/useStores";
-import { hover, truncateMultiline } from "~/styles";
-import Avatar from "../Avatar";
-import { AvatarSize } from "../Avatar/Avatar";
-import Flex from "../Flex";
-import Text from "../Text";
-import Time from "../Time";
-
-type Props = {
-  notification: Notification;
-  onNavigate: () => void;
-};
-
-function NotificationListItem({ notification, onNavigate }: Props) {
-  const { t } = useTranslation();
-  const { collections } = useStores();
-  const collectionId = notification.document?.collectionId;
-  const collection = collectionId ? collections.get(collectionId) : undefined;
-
-  const handleClick: React.MouseEventHandler<HTMLAnchorElement> = (event) => {
-    if (event.altKey) {
-      event.preventDefault();
-      event.stopPropagation();
-      void notification.toggleRead();
-      return;
-    }
-
-    void notification.markAsRead();
-
-    onNavigate();
-  };
-
-  return (
-    <Link to={notification.path} onClick={handleClick}>
-      <Container gap={8} $unread={!notification.viewedAt}>
-        <StyledAvatar model={notification.actor} size={AvatarSize.Large} />
-        <Flex column>
-          <Text as="div" size="small">
-            <Text as="span" weight="bold">
-              {notification.actor?.name ?? t("Unknown")}
-            </Text>{" "}
-            {notification.eventText(t)}{" "}
-            <Text as="span" weight="bold">
-              {notification.subject}
-            </Text>
-          </Text>
-          <Text as="span" type="tertiary" size="xsmall">
-            <Time
-              dateTime={notification.createdAt}
-              tooltipDelay={1000}
-              addSuffix
-            />{" "}
-            {collection && <>&middot; {collection.name}</>}
-          </Text>
-          {notification.comment && (
-            <StyledCommentEditor
-              defaultValue={toJS(notification.comment.data)}
-            />
-          )}
-        </Flex>
-        {notification.viewedAt ? null : <Unread />}
-      </Container>
-    </Link>
-  );
-}
-
-const StyledCommentEditor = styled(CommentEditor)`
-  font-size: 0.9em;
-  margin-top: 4px;
-
-  ${truncateMultiline(3)}
-`;
-
-const StyledAvatar = styled(Avatar)`
-  margin-top: 4px;
-`;
-
-const Container = styled(Flex)<{ $unread: boolean }>`
-  position: relative;
-  padding: 8px 12px;
-  margin: 0 8px;
-  border-radius: 4px;
-
-  &:${hover},
-  &:active {
-    background: ${s("listItemHoverBackground")};
-    cursor: var(--pointer);
-  }
-`;
-
-const Unread = styled.div`
-  width: 8px;
-  height: 8px;
-  background: ${s("accent")};
-  border-radius: 8px;
-  align-self: center;
-  position: absolute;
-  right: 20px;
-`;
-
-export default observer(NotificationListItem);

app/components/Notifications/Notifications.tsx

@@ -1,131 +0,0 @@
-import { observer } from "mobx-react";
-import { MarkAsReadIcon, SettingsIcon } from "outline-icons";
-import * as React from "react";
-import { useTranslation } from "react-i18next";
-import styled from "styled-components";
-import { s } from "@shared/styles";
-import Notification from "~/models/Notification";
-import { navigateToNotificationSettings } from "~/actions/definitions/navigation";
-import { markNotificationsAsRead } from "~/actions/definitions/notifications";
-import useActionContext from "~/hooks/useActionContext";
-import useStores from "~/hooks/useStores";
-import { hover } from "~/styles";
-import Desktop from "~/utils/Desktop";
-import Empty from "../Empty";
-import ErrorBoundary from "../ErrorBoundary";
-import Flex from "../Flex";
-import NudeButton from "../NudeButton";
-import PaginatedList from "../PaginatedList";
-import Scrollable from "../Scrollable";
-import Text from "../Text";
-import Tooltip from "../Tooltip";
-import NotificationListItem from "./NotificationListItem";
-
-type Props = {
-  /* Callback when the notification panel wants to close. */
-  onRequestClose: () => void;
-};
-
-/**
- * A panel containing a list of notifications and controls to manage them.
- */
-function Notifications(
-  { onRequestClose }: Props,
-  ref: React.RefObject<HTMLDivElement>
-) {
-  const context = useActionContext();
-  const { notifications } = useStores();
-  const { t } = useTranslation();
-  const isEmpty = notifications.orderedData.length === 0;
-
-  // Update the notification count in the dock icon, if possible.
-  React.useEffect(() => {
-    // Account for old versions of the desktop app that don't have the
-    // setNotificationCount method on the bridge.
-    if (Desktop.bridge && "setNotificationCount" in Desktop.bridge) {
-      void Desktop.bridge.setNotificationCount(
-        notifications.approximateUnreadCount
-      );
-    }
-  }, [notifications.approximateUnreadCount]);
-
-  return (
-    <ErrorBoundary>
-      <Flex style={{ width: "100%" }} column>
-        <Header justify="space-between">
-          <Text weight="bold" as="span">
-            {t("Notifications")}
-          </Text>
-          <Text color="textSecondary" as={Flex} gap={8}>
-            {notifications.approximateUnreadCount > 0 && (
-              <Tooltip delay={500} tooltip={t("Mark all as read")}>
-                <Button action={markNotificationsAsRead} context={context}>
-                  <MarkAsReadIcon />
-                </Button>
-              </Tooltip>
-            )}
-            <Tooltip delay={500} tooltip={t("Settings")}>
-              <Button action={navigateToNotificationSettings} context={context}>
-                <SettingsIcon />
-              </Button>
-            </Tooltip>
-          </Text>
-        </Header>
-        <React.Suspense fallback={null}>
-          <Scrollable ref={ref} flex topShadow>
-            <PaginatedList
-              fetch={notifications.fetchPage}
-              items={notifications.orderedData}
-              renderItem={(item: Notification) => (
-                <NotificationListItem
-                  key={item.id}
-                  notification={item}
-                  onNavigate={onRequestClose}
-                />
-              )}
-            />
-          </Scrollable>
-        </React.Suspense>
-        {isEmpty && (
-          <EmptyNotifications>{t("You're all caught up")}.</EmptyNotifications>
-        )}
-      </Flex>
-    </ErrorBoundary>
-  );
-}
-
-const EmptyNotifications = styled(Empty)`
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  min-height: 200px;
-`;
-
-const Button = styled(NudeButton)`
-  color: ${s("textSecondary")};
-
-  &:${hover},
-  &:active {
-    color: ${s("text")};
-    background: ${s("sidebarControlHoverBackground")};
-  }
-`;
-
-const Header = styled(Flex)`
-  padding: 8px 12px 12px;
-  height: 44px;
-
-  ${Button} {
-    opacity: 0.75;
-    transition: opacity 250ms ease-in-out;
-  }
-
-  &:${hover},
-  &:focus-within {
-    ${Button} {
-      opacity: 1;
-    }
-  }
-`;
-
-export default observer(React.forwardRef(Notifications));

app/components/Notifications/NotificationsPopover.tsx

@@ -1,53 +0,0 @@
-import { observer } from "mobx-react";
-import * as React from "react";
-import { useTranslation } from "react-i18next";
-import { usePopoverState, PopoverDisclosure } from "reakit/Popover";
-import styled from "styled-components";
-import { depths } from "@shared/styles";
-import Popover from "~/components/Popover";
-import Notifications from "./Notifications";
-
-type Props = {
-  children?: React.ReactNode;
-};
-
-const NotificationsPopover: React.FC = ({ children }: Props) => {
-  const { t } = useTranslation();
-  const scrollableRef = React.useRef<HTMLDivElement>(null);
-
-  const popover = usePopoverState({
-    gutter: 0,
-    placement: "top-start",
-    unstable_fixed: true,
-  });
-
-  // Reset scroll position to the top when popover is opened
-  React.useEffect(() => {
-    if (popover.visible && scrollableRef.current) {
-      scrollableRef.current.scrollTop = 0;
-    }
-  }, [popover.visible]);
-
-  return (
-    <>
-      <PopoverDisclosure {...popover}>{children}</PopoverDisclosure>
-      <StyledPopover
-        {...popover}
-        scrollable={false}
-        mobilePosition="bottom"
-        aria-label={t("Notifications")}
-        unstable_initialFocusRef={scrollableRef}
-        shrink
-        flex
-      >
-        <Notifications onRequestClose={popover.hide} ref={scrollableRef} />
-      </StyledPopover>
-    </>
-  );
-};
-
-const StyledPopover = styled(Popover)`
-  z-index: ${depths.menu};
-`;
-
-export default observer(NotificationsPopover);

app/components/PageTitle.tsx

@@ -1,34 +1,35 @@
 import { observer } from "mobx-react";
 import * as React from "react";
 import { Helmet } from "react-helmet-async";
+import { cdnPath } from "@shared/utils/urls";
 import env from "~/env";
 import useStores from "~/hooks/useStores";
-import { useTeamContext } from "./TeamContext";
 
 type Props = {
   title: React.ReactNode;
   favicon?: string;
 };
 
-const originalShortcutHref = document
-  .querySelector('link[rel="shortcut icon"]')
-  ?.getAttribute("href") as string;
-
 const PageTitle = ({ title, favicon }: Props) => {
   const { auth } = useStores();
-  const team = useTeamContext() ?? auth.team;
+  const { team } = auth;
 
   return (
     <Helmet>
       <title>
         {team?.name ? `${title} - ${team.name}` : `${title} - ${env.APP_NAME}`}
       </title>
-      <link
-        rel="shortcut icon"
-        type="image/png"
-        href={favicon ?? originalShortcutHref}
-        key={favicon ?? originalShortcutHref}
-      />
+      {favicon ? (
+        <link rel="shortcut icon" href={favicon} key={favicon} />
+      ) : (
+        <link
+          rel="shortcut icon"
+          type="image/png"
+          key="favicon"
+          href={cdnPath("/images/favicon-32.png")}
+          sizes="32x32"
+        />
+      )}
       <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     </Helmet>
   );

app/components/PaginatedList.tsx

@@ -70,7 +70,7 @@ class PaginatedList<T extends PaginatedItem> extends React.Component<Props<T>> {
   allowLoadMore = true;
 
   componentDidMount() {
-    void this.fetchResults();
+    this.fetchResults();
   }
 
   componentDidUpdate(prevProps: Props<T>) {
@@ -79,7 +79,7 @@ class PaginatedList<T extends PaginatedItem> extends React.Component<Props<T>> {
       !isEqual(prevProps.options, this.props.options)
     ) {
       this.reset();
-      void this.fetchResults();
+      this.fetchResults();
     }
   }
 

app/components/PinnedDocuments.tsx

@@ -23,7 +23,6 @@ import breakpoint from "styled-components-breakpoint";
 import Pin from "~/models/Pin";
 import DocumentCard from "~/components/DocumentCard";
 import useStores from "~/hooks/useStores";
-import { ResizingHeightContainer } from "./ResizingHeightContainer";
 
 type Props = {
   /** Pins to display */
@@ -99,36 +98,27 @@ function PinnedDocuments({ limit, pins, canUpdate, ...rest }: Props) {
       collisionDetection={closestCenter}
       onDragEnd={handleDragEnd}
     >
-      <ResizingHeightContainer
-        config={{
-          transition: {
-            duration: 0.2,
-            ease: "easeInOut",
-          },
-        }}
-      >
-        <SortableContext items={items} strategy={rectSortingStrategy}>
-          <List>
-            <AnimatePresence initial={false}>
-              {items.map((documentId) => {
-                const document = documents.get(documentId);
-                const pin = pins.find((pin) => pin.documentId === documentId);
+      <SortableContext items={items} strategy={rectSortingStrategy}>
+        <List>
+          <AnimatePresence initial={false}>
+            {items.map((documentId) => {
+              const document = documents.get(documentId);
+              const pin = pins.find((pin) => pin.documentId === documentId);
 
-                return document ? (
-                  <DocumentCard
-                    key={documentId}
-                    document={document}
-                    canUpdatePin={canUpdate}
-                    isDraggable={items.length > 1}
-                    pin={pin}
-                    {...rest}
-                  />
-                ) : null;
-              })}
-            </AnimatePresence>
-          </List>
-        </SortableContext>
-      </ResizingHeightContainer>
+              return document ? (
+                <DocumentCard
+                  key={documentId}
+                  document={document}
+                  canUpdatePin={canUpdate}
+                  isDraggable={items.length > 1}
+                  pin={pin}
+                  {...rest}
+                />
+              ) : null;
+            })}
+          </AnimatePresence>
+        </List>
+      </SortableContext>
     </DndContext>
   );
 }

app/components/Popover.tsx

@@ -1,7 +1,7 @@
 import * as React from "react";
 import { Dialog } from "reakit/Dialog";
 import { Popover as ReakitPopover, PopoverProps } from "reakit/Popover";
-import styled, { css } from "styled-components";
+import styled from "styled-components";
 import breakpoint from "styled-components-breakpoint";
 import { depths, s } from "@shared/styles";
 import useMobile from "~/hooks/useMobile";
@@ -11,87 +11,52 @@ type Props = PopoverProps & {
   children: React.ReactNode;
   width?: number;
   shrink?: boolean;
-  flex?: boolean;
   tabIndex?: number;
-  scrollable?: boolean;
-  mobilePosition?: "top" | "bottom";
 };
 
 const Popover: React.FC<Props> = ({
   children,
   shrink,
   width = 380,
-  scrollable = true,
-  flex,
-  mobilePosition,
   ...rest
-}: Props) => {
+}) => {
   const isMobile = useMobile();
 
   if (isMobile) {
     return (
       <Dialog {...rest} modal>
-        <Contents
-          $shrink={shrink}
-          $scrollable={scrollable}
-          $flex={flex}
-          $mobilePosition={mobilePosition}
-        >
-          {children}
-        </Contents>
+        <Contents $shrink={shrink}>{children}</Contents>
       </Dialog>
     );
   }
 
   return (
     <ReakitPopover {...rest}>
-      <Contents
-        $shrink={shrink}
-        $width={width}
-        $scrollable={scrollable}
-        $flex={flex}
-      >
+      <Contents $shrink={shrink} $width={width}>
         {children}
       </Contents>
     </ReakitPopover>
   );
 };
 
-type ContentsProps = {
-  $shrink?: boolean;
-  $width?: number;
-  $flex?: boolean;
-  $scrollable: boolean;
-  $mobilePosition?: "top" | "bottom";
-};
-
-const Contents = styled.div<ContentsProps>`
-  display: ${(props) => (props.$flex ? "flex" : "block")};
+const Contents = styled.div<{ $shrink?: boolean; $width?: number }>`
   animation: ${fadeAndScaleIn} 200ms ease;
   transform-origin: 75% 0;
   background: ${s("menuBackground")};
   border-radius: 6px;
   padding: ${(props) => (props.$shrink ? "6px 0" : "12px 24px")};
   max-height: 75vh;
+  overflow-x: hidden;
+  overflow-y: auto;
   box-shadow: ${s("menuShadow")};
   width: ${(props) => props.$width}px;
 
-  ${(props) =>
-    props.$scrollable &&
-    css`
-      overflow-x: hidden;
-      overflow-y: auto;
-    `}
-
   ${breakpoint("mobile", "tablet")`
     position: fixed;
     z-index: ${depths.menu};
 
     // 50 is a magic number that positions us nicely under the top bar
-    top: ${(props: ContentsProps) =>
-      props.$mobilePosition === "bottom" ? "auto" : "50px"};
-    bottom: ${(props: ContentsProps) =>
-      props.$mobilePosition === "bottom" ? "0" : "auto"};
+    top: 50px;
     left: 8px;
     right: 8px;
     width: auto;

app/components/Scene.tsx

@@ -11,7 +11,6 @@ type Props = {
   left?: React.ReactNode;
   actions?: React.ReactNode;
   centered?: boolean;
-  children?: React.ReactNode;
 };
 
 const Scene: React.FC<Props> = ({
@@ -22,7 +21,7 @@ const Scene: React.FC<Props> = ({
   left,
   children,
   centered,
-}: Props) => (
+}) => (
   <FillWidth>
     <PageTitle title={textTitle || title} />
     <Header

app/components/SearchActions.ts

@@ -11,7 +11,7 @@ export default function SearchActions() {
 
   React.useEffect(() => {
     if (!searches.isLoaded) {
-      void searches.fetchPage({});
+      searches.fetchPage({});
     }
   }, [searches]);
 
@@ -20,8 +20,7 @@ export default function SearchActions() {
   }));
 
   useCommandBarActions(
-    searchQuery ? [searchDocumentsForQuery(searchQuery)] : [],
-    [searchQuery]
+    searchQuery ? [searchDocumentsForQuery(searchQuery)] : []
   );
 
   useCommandBarActions(searches.recent.map(navigateToRecentSearchQuery));

app/components/SearchPopover.tsx

@@ -17,7 +17,7 @@ import useStores from "~/hooks/useStores";
 import { SearchResult } from "~/types";
 import SearchListItem from "./SearchListItem";
 
-type Props = React.HTMLAttributes<HTMLInputElement> & { shareId: string };
+type Props = { shareId: string };
 
 function SearchPopover({ shareId }: Props) {
   const { t } = useTranslation();
@@ -32,7 +32,6 @@ function SearchPopover({ shareId }: Props) {
 
   const [query, setQuery] = React.useState("");
   const searchResults = documents.searchResults(query);
-  const { show, hide } = popover;
 
   const [cachedQuery, setCachedQuery] = React.useState(query);
   const [cachedSearchResults, setCachedSearchResults] = React.useState<
@@ -43,9 +42,9 @@ function SearchPopover({ shareId }: Props) {
     if (searchResults) {
       setCachedQuery(query);
       setCachedSearchResults(searchResults);
-      show();
+      popover.show();
     }
-  }, [searchResults, query, show]);
+  }, [searchResults, query, popover.show]);
 
   const performSearch = React.useCallback(
     async ({ query, ...options }) => {
@@ -77,8 +76,9 @@ function SearchPopover({ shareId }: Props) {
     [popover, cachedQuery]
   );
 
-  const searchInputRef =
-    popover.unstable_referenceRef as React.RefObject<HTMLInputElement>;
+  const searchInputRef = popover.unstable_referenceRef as React.RefObject<
+    HTMLInputElement
+  >;
 
   const firstSearchItem = React.useRef<HTMLAnchorElement>(null);
 
@@ -89,14 +89,10 @@ function SearchPopover({ shareId }: Props) {
 
   const handleSearchInputFocus = React.useCallback(() => {
     focusRef.current = searchInputRef.current;
-  }, [searchInputRef]);
+  }, []);
 
   const handleKeyDown = React.useCallback(
     (ev: React.KeyboardEvent<HTMLInputElement>) => {
-      if (ev.nativeEvent.isComposing) {
-        return;
-      }
-
       if (ev.key === "Enter") {
         if (searchResults) {
           popover.show();
@@ -142,12 +138,12 @@ function SearchPopover({ shareId }: Props) {
   );
 
   const handleSearchItemClick = React.useCallback(() => {
-    hide();
+    popover.hide();
     if (searchInputRef.current) {
       searchInputRef.current.value = "";
       focusRef.current = document.getElementById(bodyContentId);
     }
-  }, [searchInputRef, hide]);
+  }, [popover.hide]);
 
   useKeyDown("/", (ev) => {
     if (

app/components/Sidebar/App.tsx

@@ -43,8 +43,8 @@ function AppSidebar() {
 
   React.useEffect(() => {
     if (!user.isViewer) {
-      void documents.fetchDrafts();
-      void documents.fetchTemplates();
+      documents.fetchDrafts();
+      documents.fetchTemplates();
     }
   }, [documents, user.isViewer]);
 

app/components/Sidebar/Shared.tsx

@@ -8,7 +8,7 @@ import SearchPopover from "~/components/SearchPopover";
 import useStores from "~/hooks/useStores";
 import history from "~/utils/history";
 import { homePath, sharedDocumentPath } from "~/utils/routeHelpers";
-import { useTeamContext } from "../TeamContext";
+import { IAvatar } from "../Avatar/Avatar";
 import TeamLogo from "../TeamLogo";
 import Sidebar from "./Sidebar";
 import HeaderButton from "./components/HeaderButton";
@@ -16,12 +16,12 @@ import Section from "./components/Section";
 import DocumentLink from "./components/SharedDocumentLink";
 
 type Props = {
+  team?: IAvatar & { name: string };
   rootNode: NavigationNode;
   shareId: string;
 };
 
-function SharedSidebar({ rootNode, shareId }: Props) {
-  const team = useTeamContext();
+function SharedSidebar({ rootNode, team, shareId }: Props) {
   const { ui, documents, auth } = useStores();
   const { t } = useTranslation();
 

app/components/Sidebar/Sidebar.tsx

@@ -15,8 +15,6 @@ import { draggableOnDesktop, fadeOnDesktopBackgrounded } from "~/styles";
 import { fadeIn } from "~/styles/animations";
 import Desktop from "~/utils/Desktop";
 import Avatar from "../Avatar";
-import NotificationIcon from "../Notifications/NotificationIcon";
-import NotificationsPopover from "../Notifications/NotificationsPopover";
 import HeaderButton, { HeaderButtonProps } from "./components/HeaderButton";
 import ResizeBorder from "./components/ResizeBorder";
 import Toggle, { ToggleButton, Positioner } from "./components/Toggle";
@@ -27,198 +25,191 @@ type Props = {
   children: React.ReactNode;
 };
 
-const Sidebar = React.forwardRef<HTMLDivElement, Props>(function _Sidebar(
-  { children }: Props,
-  ref: React.RefObject<HTMLDivElement>
-) {
-  const [isCollapsing, setCollapsing] = React.useState(false);
-  const theme = useTheme();
-  const { t } = useTranslation();
-  const { ui, auth } = useStores();
-  const location = useLocation();
-  const previousLocation = usePrevious(location);
-  const { isMenuOpen } = useMenuContext();
-  const { user } = auth;
-  const width = ui.sidebarWidth;
-  const collapsed = ui.sidebarIsClosed && !isMenuOpen;
-  const maxWidth = theme.sidebarMaxWidth;
-  const minWidth = theme.sidebarMinWidth + 16; // padding
+const Sidebar = React.forwardRef<HTMLDivElement, Props>(
+  ({ children }: Props, ref: React.RefObject<HTMLDivElement>) => {
+    const [isCollapsing, setCollapsing] = React.useState(false);
+    const theme = useTheme();
+    const { t } = useTranslation();
+    const { ui, auth } = useStores();
+    const location = useLocation();
+    const previousLocation = usePrevious(location);
+    const { isMenuOpen } = useMenuContext();
+    const { user } = auth;
+    const width = ui.sidebarWidth;
+    const collapsed = ui.sidebarIsClosed && !isMenuOpen;
+    const maxWidth = theme.sidebarMaxWidth;
+    const minWidth = theme.sidebarMinWidth + 16; // padding
 
-  const setWidth = ui.setSidebarWidth;
-  const [offset, setOffset] = React.useState(0);
-  const [isAnimating, setAnimating] = React.useState(false);
-  const [isResizing, setResizing] = React.useState(false);
-  const isSmallerThanMinimum = width < minWidth;
+    const setWidth = ui.setSidebarWidth;
+    const [offset, setOffset] = React.useState(0);
+    const [isAnimating, setAnimating] = React.useState(false);
+    const [isResizing, setResizing] = React.useState(false);
+    const isSmallerThanMinimum = width < minWidth;
 
-  const handleDrag = React.useCallback(
-    (event: MouseEvent) => {
-      // suppresses text selection
-      event.preventDefault();
-      // this is simple because the sidebar is always against the left edge
-      const width = Math.min(event.pageX - offset, maxWidth);
-      const isSmallerThanCollapsePoint = width < minWidth / 2;
+    const handleDrag = React.useCallback(
+      (event: MouseEvent) => {
+        // suppresses text selection
+        event.preventDefault();
+        // this is simple because the sidebar is always against the left edge
+        const width = Math.min(event.pageX - offset, maxWidth);
+        const isSmallerThanCollapsePoint = width < minWidth / 2;
 
-      if (isSmallerThanCollapsePoint) {
-        setWidth(theme.sidebarCollapsedWidth);
+        if (isSmallerThanCollapsePoint) {
+          setWidth(theme.sidebarCollapsedWidth);
+        } else {
+          setWidth(width);
+        }
+      },
+      [theme, offset, minWidth, maxWidth, setWidth]
+    );
+
+    const handleStopDrag = React.useCallback(() => {
+      setResizing(false);
+
+      if (document.activeElement instanceof HTMLElement) {
+        document.activeElement.blur();
+      }
+
+      if (isSmallerThanMinimum) {
+        const isSmallerThanCollapsePoint = width < minWidth / 2;
+
+        if (isSmallerThanCollapsePoint) {
+          setAnimating(false);
+          setCollapsing(true);
+          ui.collapseSidebar();
+        } else {
+          setWidth(minWidth);
+          setAnimating(true);
+        }
       } else {
         setWidth(width);
       }
-    },
-    [theme, offset, minWidth, maxWidth, setWidth]
-  );
+    }, [ui, isSmallerThanMinimum, minWidth, width, setWidth]);
 
-  const handleStopDrag = React.useCallback(() => {
-    setResizing(false);
-
-    if (document.activeElement instanceof HTMLElement) {
-      document.activeElement.blur();
-    }
-
-    if (isSmallerThanMinimum) {
-      const isSmallerThanCollapsePoint = width < minWidth / 2;
-
-      if (isSmallerThanCollapsePoint) {
+    const handleMouseDown = React.useCallback(
+      (event) => {
+        setOffset(event.pageX - width);
+        setResizing(true);
         setAnimating(false);
-        setCollapsing(true);
-        ui.collapseSidebar();
-      } else {
-        setWidth(minWidth);
-        setAnimating(true);
+      },
+      [width]
+    );
+
+    React.useEffect(() => {
+      if (isAnimating) {
+        setTimeout(() => setAnimating(false), ANIMATION_MS);
       }
-    } else {
-      setWidth(width);
-    }
-  }, [ui, isSmallerThanMinimum, minWidth, width, setWidth]);
+    }, [isAnimating]);
 
-  const handleMouseDown = React.useCallback(
-    (event) => {
-      setOffset(event.pageX - width);
-      setResizing(true);
-      setAnimating(false);
-    },
-    [width]
-  );
+    React.useEffect(() => {
+      if (isCollapsing) {
+        setTimeout(() => {
+          setWidth(minWidth);
+          setCollapsing(false);
+        }, ANIMATION_MS);
+      }
+    }, [setWidth, minWidth, isCollapsing]);
 
-  React.useEffect(() => {
-    if (isAnimating) {
-      setTimeout(() => setAnimating(false), ANIMATION_MS);
-    }
-  }, [isAnimating]);
+    React.useEffect(() => {
+      if (isResizing) {
+        document.addEventListener("mousemove", handleDrag);
+        document.addEventListener("mouseup", handleStopDrag);
+      }
 
-  React.useEffect(() => {
-    if (isCollapsing) {
-      setTimeout(() => {
-        setWidth(minWidth);
-        setCollapsing(false);
-      }, ANIMATION_MS);
-    }
-  }, [setWidth, minWidth, isCollapsing]);
+      return () => {
+        document.removeEventListener("mousemove", handleDrag);
+        document.removeEventListener("mouseup", handleStopDrag);
+      };
+    }, [isResizing, handleDrag, handleStopDrag]);
 
-  React.useEffect(() => {
-    if (isResizing) {
-      document.addEventListener("mousemove", handleDrag);
-      document.addEventListener("mouseup", handleStopDrag);
-    }
+    const handleReset = React.useCallback(() => {
+      ui.setSidebarWidth(theme.sidebarWidth);
+    }, [ui, theme.sidebarWidth]);
 
-    return () => {
-      document.removeEventListener("mousemove", handleDrag);
-      document.removeEventListener("mouseup", handleStopDrag);
-    };
-  }, [isResizing, handleDrag, handleStopDrag]);
+    React.useEffect(() => {
+      ui.setSidebarResizing(isResizing);
+    }, [ui, isResizing]);
 
-  const handleReset = React.useCallback(() => {
-    ui.setSidebarWidth(theme.sidebarWidth);
-  }, [ui, theme.sidebarWidth]);
+    React.useEffect(() => {
+      if (location !== previousLocation) {
+        ui.hideMobileSidebar();
+      }
+    }, [ui, location, previousLocation]);
 
-  React.useEffect(() => {
-    ui.setSidebarResizing(isResizing);
-  }, [ui, isResizing]);
+    const style = React.useMemo(
+      () => ({
+        width: `${width}px`,
+      }),
+      [width]
+    );
 
-  React.useEffect(() => {
-    if (location !== previousLocation) {
-      ui.hideMobileSidebar();
-    }
-  }, [ui, location, previousLocation]);
+    const toggleStyle = React.useMemo(
+      () => ({
+        right: "auto",
+        marginLeft: `${collapsed ? theme.sidebarCollapsedWidth : width}px`,
+      }),
+      [width, theme.sidebarCollapsedWidth, collapsed]
+    );
 
-  const style = React.useMemo(
-    () => ({
-      width: `${width}px`,
-    }),
-    [width]
-  );
+    return (
+      <>
+        <Container
+          ref={ref}
+          style={style}
+          $isAnimating={isAnimating}
+          $isSmallerThanMinimum={isSmallerThanMinimum}
+          $mobileSidebarVisible={ui.mobileSidebarVisible}
+          $collapsed={collapsed}
+          column
+        >
+          {ui.mobileSidebarVisible && (
+            <Portal>
+              <Backdrop onClick={ui.toggleMobileSidebar} />
+            </Portal>
+          )}
+          {children}
 
-  const toggleStyle = React.useMemo(
-    () => ({
-      right: "auto",
-      marginLeft: `${collapsed ? theme.sidebarCollapsedWidth : width}px`,
-    }),
-    [width, theme.sidebarCollapsedWidth, collapsed]
-  );
-
-  return (
-    <>
-      <Container
-        ref={ref}
-        style={style}
-        $isAnimating={isAnimating}
-        $isSmallerThanMinimum={isSmallerThanMinimum}
-        $mobileSidebarVisible={ui.mobileSidebarVisible}
-        $collapsed={collapsed}
-        column
-      >
-        {ui.mobileSidebarVisible && (
-          <Portal>
-            <Backdrop onClick={ui.toggleMobileSidebar} />
-          </Portal>
-        )}
-        {children}
-
-        {user && (
-          <AccountMenu>
-            {(props: HeaderButtonProps) => (
-              <HeaderButton
-                {...props}
-                showMoreMenu
-                title={user.name}
-                image={
-                  <StyledAvatar
-                    alt={user.name}
-                    model={user}
-                    size={24}
-                    showBorder={false}
-                  />
-                }
-              >
-                <NotificationsPopover>
-                  {(rest: HeaderButtonProps) => (
-                    <HeaderButton {...rest} image={<NotificationIcon />} />
-                  )}
-                </NotificationsPopover>
-              </HeaderButton>
-            )}
-          </AccountMenu>
-        )}
-        <ResizeBorder
-          onMouseDown={handleMouseDown}
-          onDoubleClick={ui.sidebarIsClosed ? undefined : handleReset}
-        />
-        {ui.sidebarIsClosed && (
-          <Toggle
-            onClick={ui.toggleCollapsedSidebar}
-            direction={"right"}
-            aria-label={t("Expand")}
+          {user && (
+            <AccountMenu>
+              {(props: HeaderButtonProps) => (
+                <HeaderButton
+                  {...props}
+                  showMoreMenu
+                  title={user.name}
+                  image={
+                    <StyledAvatar
+                      alt={user.name}
+                      model={user}
+                      size={24}
+                      showBorder={false}
+                    />
+                  }
+                />
+              )}
+            </AccountMenu>
+          )}
+          <ResizeBorder
+            onMouseDown={handleMouseDown}
+            onDoubleClick={ui.sidebarIsClosed ? undefined : handleReset}
           />
-        )}
-      </Container>
-      <Toggle
-        style={toggleStyle}
-        onClick={ui.toggleCollapsedSidebar}
-        direction={ui.sidebarIsClosed ? "right" : "left"}
-        aria-label={ui.sidebarIsClosed ? t("Expand") : t("Collapse")}
-      />
-    </>
-  );
-});
+          {ui.sidebarIsClosed && (
+            <Toggle
+              onClick={ui.toggleCollapsedSidebar}
+              direction={"right"}
+              aria-label={t("Expand")}
+            />
+          )}
+        </Container>
+        <Toggle
+          style={toggleStyle}
+          onClick={ui.toggleCollapsedSidebar}
+          direction={ui.sidebarIsClosed ? "right" : "left"}
+          aria-label={ui.sidebarIsClosed ? t("Expand") : t("Collapse")}
+        />
+      </>
+    );
+  }
+);
 
 const StyledAvatar = styled(Avatar)`
   margin-left: 4px;

app/components/Sidebar/components/CollectionLink.tsx

@@ -37,14 +37,14 @@ const CollectionLink: React.FC<Props> = ({
   expanded,
   onDisclosureClick,
   isDraggingAnyCollection,
-}: Props) => {
+}) => {
   const itemRef = React.useRef<
     NavigationNode & { depth: number; active: boolean; collectionId: string }
   >();
   const { dialogs, documents, collections } = useStores();
   const [menuOpen, handleMenuOpen, handleMenuClose] = useBoolean();
   const [isEditing, setIsEditing] = React.useState(false);
-  const can = usePolicy(collection);
+  const canUpdate = usePolicy(collection).update;
   const { t } = useTranslation();
   const history = useHistory();
   const inStarredSection = useStarredContext();
@@ -105,7 +105,7 @@ const CollectionLink: React.FC<Props> = ({
         }
       }
     },
-    canDrop: () => can.createDocument,
+    canDrop: () => canUpdate,
     collect: (monitor) => ({
       isOver: !!monitor.isOver({
         shallow: true,
@@ -118,10 +118,6 @@ const CollectionLink: React.FC<Props> = ({
     setIsEditing(isEditing);
   }, []);
 
-  const handlePrefetch = React.useCallback(() => {
-    void collection.fetchDocuments();
-  }, [collection]);
-
   const context = useActionContext({
     activeCollectionId: collection.id,
     inStarredSection,
@@ -138,7 +134,6 @@ const CollectionLink: React.FC<Props> = ({
             }}
             expanded={expanded}
             onDisclosureClick={onDisclosureClick}
-            onClickIntent={handlePrefetch}
             icon={
               <CollectionIcon collection={collection} expanded={expanded} />
             }
@@ -152,7 +147,7 @@ const CollectionLink: React.FC<Props> = ({
                 title={collection.name}
                 onSubmit={handleTitleChange}
                 onEditing={handleTitleEditing}
-                canUpdate={can.update}
+                canUpdate={canUpdate}
               />
             }
             exact={false}

app/components/Sidebar/components/CollectionLinkChildren.tsx

@@ -2,11 +2,8 @@ import { observer } from "mobx-react";
 import * as React from "react";
 import { useDrop } from "react-dnd";
 import { useTranslation } from "react-i18next";
-import styled from "styled-components";
 import Collection from "~/models/Collection";
 import Document from "~/models/Document";
-import DocumentsLoader from "~/components/DocumentsLoader";
-import { ResizingHeightContainer } from "~/components/ResizingHeightContainer";
 import usePolicy from "~/hooks/usePolicy";
 import useStores from "~/hooks/useStores";
 import useToasts from "~/hooks/useToasts";
@@ -14,7 +11,6 @@ import DocumentLink from "./DocumentLink";
 import DropCursor from "./DropCursor";
 import EmptyCollectionPlaceholder from "./EmptyCollectionPlaceholder";
 import Folder from "./Folder";
-import PlaceholderCollections from "./PlaceholderCollections";
 import { DragObject } from "./SidebarLink";
 import useCollectionDocuments from "./useCollectionDocuments";
 
@@ -57,7 +53,7 @@ function CollectionLinkChildren({
       if (!collection) {
         return;
       }
-      void documents.move(item.id, collection.id, undefined, 0);
+      documents.move(item.id, collection.id, undefined, 0);
     },
     collect: (monitor) => ({
       isOverReorder: !!monitor.isOver(),
@@ -67,40 +63,28 @@ function CollectionLinkChildren({
 
   return (
     <Folder expanded={expanded}>
-      {isDraggingAnyDocument && can.createDocument && manualSort && (
+      {isDraggingAnyDocument && can.update && manualSort && (
         <DropCursor
           isActiveDrop={isOverReorder}
           innerRef={dropToReorder}
           position="top"
         />
       )}
-      <DocumentsLoader collection={collection} enabled={expanded}>
-        {!childDocuments && (
-          <ResizingHeightContainer hideOverflow>
-            <Loading />
-          </ResizingHeightContainer>
-        )}
-        {childDocuments?.map((node, index) => (
-          <DocumentLink
-            key={node.id}
-            node={node}
-            collection={collection}
-            activeDocument={documents.active}
-            prefetchDocument={prefetchDocument}
-            isDraft={node.isDraft}
-            depth={2}
-            index={index}
-          />
-        ))}
-        {childDocuments?.length === 0 && <EmptyCollectionPlaceholder />}
-      </DocumentsLoader>
+      {childDocuments.map((node, index) => (
+        <DocumentLink
+          key={node.id}
+          node={node}
+          collection={collection}
+          activeDocument={documents.active}
+          prefetchDocument={prefetchDocument}
+          isDraft={node.isDraft}
+          depth={2}
+          index={index}
+        />
+      ))}
+      {childDocuments.length === 0 && <EmptyCollectionPlaceholder />}
     </Folder>
   );
 }
 
-const Loading = styled(PlaceholderCollections)`
-  margin-left: 44px;
-  min-height: 90px;
-`;
-
 export default observer(CollectionLinkChildren);

app/components/Sidebar/components/Collections.tsx

@@ -23,20 +23,13 @@ function Collections() {
   const { t } = useTranslation();
   const orderedCollections = collections.orderedData;
 
-  const params = React.useMemo(
-    () => ({
-      limit: 100,
-    }),
-    []
-  );
-
   const [
     { isCollectionDropping, isDraggingAnyCollection },
     dropToReorderCollection,
   ] = useDrop({
     accept: "collection",
     drop: async (item: DragObject) => {
-      void collections.move(
+      collections.move(
         item.id,
         fractionalIndex(null, orderedCollections[0].index)
       );
@@ -54,7 +47,7 @@ function Collections() {
         <Relative>
           <PaginatedList
             fetch={collections.fetchPage}
-            options={params}
+            options={{ limit: 25 }}
             aria-label={t("Collections")}
             items={collections.orderedData}
             loading={<PlaceholderCollections />}

app/components/Sidebar/components/DocumentLink.tsx

@@ -67,9 +67,9 @@ function InnerDocumentLink(
 
   React.useEffect(() => {
     if (isActiveDocument && hasChildDocuments) {
-      void fetchChildDocuments(node.id);
+      fetchChildDocuments(node.id);
     }
-  }, [fetchChildDocuments, node.id, hasChildDocuments, isActiveDocument]);
+  }, [fetchChildDocuments, node, hasChildDocuments, isActiveDocument]);
 
   const pathToNode = React.useMemo(
     () => collection?.pathToDocument(node.id).map((entry) => entry.id),
@@ -114,8 +114,8 @@ function InnerDocumentLink(
     [expanded]
   );
 
-  const handlePrefetch = React.useCallback(() => {
-    void prefetchDocument?.(node.id);
+  const handleMouseEnter = React.useCallback(() => {
+    prefetchDocument?.(node.id);
   }, [prefetchDocument, node]);
 
   const handleTitleChange = React.useCallback(
@@ -125,6 +125,7 @@ function InnerDocumentLink(
       }
       await documents.update({
         id: document.id,
+        text: document.text,
         title,
       });
     },
@@ -241,11 +242,11 @@ function InnerDocumentLink(
       }
 
       if (expanded) {
-        void documents.move(item.id, collection.id, node.id, 0);
+        documents.move(item.id, collection.id, node.id, 0);
         return;
       }
 
-      void documents.move(item.id, collection.id, parentId, index + 1);
+      documents.move(item.id, collection.id, parentId, index + 1);
     },
     collect: (monitor) => ({
       isOverReorder: monitor.isOver(),
@@ -316,7 +317,7 @@ function InnerDocumentLink(
               <SidebarLink
                 expanded={hasChildren ? isExpanded : undefined}
                 onDisclosureClick={handleDisclosureClick}
-                onClickIntent={handlePrefetch}
+                onMouseEnter={handleMouseEnter}
                 to={{
                   pathname: node.url,
                   state: {
@@ -334,9 +335,7 @@ function InnerDocumentLink(
                   />
                 }
                 isActive={(match, location: Location<{ starred?: boolean }>) =>
-                  ((document && location.pathname.endsWith(document.urlId)) ||
-                    !!match) &&
-                  location.state?.starred === inStarredSection
+                  !!match && location.state?.starred === inStarredSection
                 }
                 isActiveDrop={isOverReparent && canDropToReparent}
                 depth={depth}

app/components/Sidebar/components/DraggableCollectionLink.tsx

@@ -51,7 +51,7 @@ function DraggableCollectionLink({
   ] = useDrop({
     accept: "collection",
     drop: (item: DragObject) => {
-      void collections.move(
+      collections.move(
         item.id,
         fractionalIndex(collection.index, belowCollectionIndex)
       );

app/components/Sidebar/components/DropToImport.tsx

@@ -26,14 +26,10 @@ function DropToImport({ disabled, children, collectionId, documentId }: Props) {
     collectionId,
     documentId
   );
-  invariant(
-    collectionId || documentId,
-    "Must provide either collectionId or documentId"
-  );
-
-  const canCollection = usePolicy(collectionId);
-  const canDocument = usePolicy(documentId);
+  const targetId = collectionId || documentId;
+  invariant(targetId, "Must provide either collectionId or documentId");
 
+  const can = usePolicy(targetId);
   const handleRejection = React.useCallback(() => {
     showToast(
       t("Document not supported – try Markdown, Plain text, HTML, or Word"),
@@ -43,11 +39,7 @@ function DropToImport({ disabled, children, collectionId, documentId }: Props) {
     );
   }, [t, showToast]);
 
-  if (
-    disabled ||
-    (collectionId && !canCollection.createDocument) ||
-    (documentId && !canDocument.createChildDocument)
-  ) {
+  if (disabled || !can.update) {
     return children;
   }
 

app/components/Sidebar/components/EditableTitle.tsx

@@ -121,7 +121,7 @@ const Input = styled.input`
   border: 0;
   padding: 5px 6px;
   margin: -4px;
-  height: 30px;
+  height: 32px;
 
   &:focus {
     outline-color: ${s("accent")};

app/components/Sidebar/components/Folder.tsx

@@ -3,10 +3,9 @@ import styled from "styled-components";
 
 type Props = {
   expanded: boolean;
-  children?: React.ReactNode;
 };
 
-const Folder: React.FC<Props> = ({ expanded, children }: Props) => {
+const Folder: React.FC<Props> = ({ expanded, children }) => {
   const [openedOnce, setOpenedOnce] = React.useState(expanded);
 
   // allows us to avoid rendering all children when the folder hasn't been opened

app/components/Sidebar/components/Header.tsx

@@ -9,13 +9,12 @@ type Props = {
   /** Unique header id – if passed the header will become toggleable */
   id?: string;
   title: React.ReactNode;
-  children?: React.ReactNode;
 };
 
 /**
  * Toggleable sidebar header
  */
-export const Header: React.FC<Props> = ({ id, title, children }: Props) => {
+export const Header: React.FC<Props> = ({ id, title, children }) => {
   const [firstRender, setFirstRender] = React.useState(true);
   const [expanded, setExpanded] = usePersistedState<boolean>(
     `sidebar-header-${id}`,