upload images and store them as base64

.dockerignore

@@ -2,9 +2,6 @@
 !public/
 !src/
 !.npmrc
-!.eslintrc.json
-!.prettierrc
 !package-lock.json
 !package.json
 !tsconfig.json
-!.env

.env

@@ -1,5 +0,0 @@
-REACT_APP_BACKEND_V1_GET_URL=https://json.excalidraw.com/api/v1/
-REACT_APP_BACKEND_V2_GET_URL=https://json.excalidraw.com/api/v2/
-REACT_APP_BACKEND_V2_POST_URL=https://json.excalidraw.com/api/v2/post/
-REACT_APP_SOCKET_SERVER_URL=https://portal.excalidraw.com
-REACT_APP_FIREBASE_CONFIG='{"apiKey":"AIzaSyAd15pYlMci_xIp9ko6wkEsDzAAA0Dn0RU","authDomain":"excalidraw-room-persistence.firebaseapp.com","databaseURL":"https://excalidraw-room-persistence.firebaseio.com","projectId":"excalidraw-room-persistence","storageBucket":"excalidraw-room-persistence.appspot.com","messagingSenderId":"654800341332","appId":"1:654800341332:web:4a692de832b55bd57ce0c1"}'

.env.production

@@ -1 +0,0 @@
-REACT_APP_GOOGLE_ANALYTICS_ID=UA-387204-13

.eslintignore

@@ -3,4 +3,3 @@ build/
 package-lock.json
 .vscode/
 firebase/
-dist/

.eslintrc.json

@@ -3,8 +3,6 @@
   "plugins": ["prettier"],
   "rules": {
     "curly": "warn",
-    "dot-notation": "warn",
-    "import/no-anonymous-default-export": "off",
     "no-console": [
       "warn",
       {
@@ -12,22 +10,7 @@
       }
     ],
     "no-else-return": "warn",
-    "no-lonely-if": "warn",
-    "no-restricted-syntax": [
-      "warn",
-      {
-        "message": "Use 't(...)' instead of literal text in JSX",
-        "selector": "JSXText[value=/\\w/]"
-      }
-    ],
-    "no-unneeded-ternary": "warn",
-    "no-unused-expressions": "warn",
-    "no-unused-vars": "warn",
     "no-useless-return": "warn",
-    "no-var": "warn",
-    "object-shorthand": "warn",
-    "one-var": ["warn", "never"],
-    "prefer-arrow-callback": "warn",
     "prefer-const": [
       "warn",
       {
@@ -35,6 +18,13 @@
       }
     ],
     "prefer-template": "warn",
-    "prettier/prettier": "warn"
+    "prettier/prettier": "warn",
+    "no-restricted-syntax": [
+      "warn",
+      {
+        "selector": "JSXText[value=/\\w/]",
+        "message": "Use 't(...)' instead of literal text in JSX"
+      }
+    ]
   }
 }

.gitattributes

@@ -1 +0,0 @@
-* text=auto eol=lf

.github/dependabot.yml

@@ -1,37 +0,0 @@
-version: 2
-updates:
-  - package-ecosystem: npm
-    directory: "/"
-    schedule:
-      interval: weekly
-      day: sunday
-      time: "01:00"
-    open-pull-requests-limit: 99
-    reviewers:
-      - lipis
-    assignees:
-      - lipis
-
-  - package-ecosystem: npm
-    directory: "/src/packages/excalidraw/"
-    schedule:
-      interval: weekly
-      day: sunday
-      time: "01:00"
-    open-pull-requests-limit: 99
-    reviewers:
-      - ad1992
-    assignees:
-      - ad1992
-
-  - package-ecosystem: npm
-    directory: "/src/packages/utils/"
-    schedule:
-      interval: weekly
-      day: sunday
-      time: "01:00"
-    open-pull-requests-limit: 99
-    reviewers:
-      - ad1992
-    assignees:
-      - ad1992

.github/workflows/build-docker.yml

@@ -1,14 +0,0 @@
-name: Build Docker image
-
-on:
-  push:
-    branches:
-      - master
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v1
-      - run: docker build -t excalidraw .

.github/workflows/build-packages.yml

@@ -1,33 +0,0 @@
-name: Build packages
-
-on:
-  push:
-    branches:
-      - master
-  pull_request:
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v1
-
-      - name: Setup Node.js 14.x
-        uses: actions/setup-node@v1
-        with:
-          node-version: 14.x
-
-      - name: Install dependencies
-        run: |
-          npm ci
-          npm ci --prefix src/packages/excalidraw
-          npm ci --prefix src/packages/utils
-
-      - name: Build @excalidraw/excalidraw
-        run: |
-          npm run pack --prefix src/packages/excalidraw
-
-      - name: Build @excalidraw/utils
-        run: |
-          npm run pack --prefix src/packages/utils

.github/workflows/cancel.yml

@@ -1,18 +0,0 @@
-name: Cancel previous runs
-
-on:
-  push:
-    branches:
-      - master
-  pull_request:
-
-jobs:
-  cancel:
-    runs-on: ubuntu-latest
-
-    timeout-minutes: 3
-    steps:
-      - uses: styfle/cancel-workflow-action@0.6.0
-        with:
-          workflow_id: 400555, 400556, 905313, 1451724, 1710116, 3185001, 3438604
-          access_token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/lint.yml

@@ -1,6 +1,10 @@
 name: Lint
 
-on: pull_request
+on:
+  push:
+    branches:
+      - master
+  pull_request:
 
 jobs:
   lint:
@@ -9,10 +13,10 @@ jobs:
     steps:
       - uses: actions/checkout@v1
 
-      - name: Setup Node.js 14.x
+      - name: Setup Node.js 12.x
         uses: actions/setup-node@v1
         with:
-          node-version: 14.x
+          node-version: 12.x
 
       - name: Install and lint
         run: |
@@ -20,3 +24,5 @@ jobs:
           npm run test:other
           npm run test:code
           npm run test:typecheck
+        env:
+          CI: true

.github/workflows/locales-coverage.yml

@@ -1,47 +0,0 @@
-name: Build locales coverage
-
-on:
-  push:
-    branches:
-      - "l10n_master"
-
-jobs:
-  locales:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v2
-        with:
-          token: ${{ secrets.PUSH_TRANSLATIONS_COVERAGE_PAT }}
-
-      - name: Setup Node.js 14.x
-        uses: actions/setup-node@v1
-        with:
-          node-version: 14.x
-
-      - name: Create report file
-        run: |
-          npm run locales-coverage
-          FILE_CHANGED=$(git diff src/locales/percentages.json)
-          if [ ! -z "${FILE_CHANGED}" ]; then
-            git config --global user.name 'Excalidraw Bot'
-            git config --global user.email 'bot@excalidraw.com'
-            git add src/locales/percentages.json
-            git commit -am "Auto commit: Calculate translation coverage"
-            git push
-          fi
-      - name: Construct comment body
-        id: getCommentBody
-        run: |
-          body=$(npm run locales-coverage:description | grep '^[^>]')
-          body="${body//'%'/'%25'}"
-          body="${body//$'\n'/'%0A'}"
-          body="${body//$'\r'/'%0D'}"
-          echo ::set-output name=body::$body
-
-      - name: Update description with coverage
-        uses: kt3k/update-pr-description@v1.0.1
-        with:
-          pr_body: ${{ steps.getCommentBody.outputs.body }}
-          pr_title: "chore: Update translations from Crowdin"
-          github_token: ${{ secrets.PUSH_TRANSLATIONS_COVERAGE_PAT }}

.github/workflows/publish-docker.yml

@@ -1,20 +0,0 @@
-name: Publish Docker
-
-on:
-  push:
-    branches:
-      - master
-
-jobs:
-  publish-docker:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v2
-      - uses: docker/build-push-action@v1
-        with:
-          username: ${{ secrets.DOCKER_USERNAME }}
-          password: ${{ secrets.DOCKER_PASSWORD }}
-          repository: excalidraw/excalidraw
-          tag_with_ref: true
-          tag_with_sha: true

.github/workflows/semantic-pr-title.yml

@@ -1,17 +0,0 @@
-name: "Semantic PR title"
-
-on:
-  pull_request_target:
-    types:
-      - opened
-      - edited
-      - synchronize
-
-jobs:
-  main:
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: amannn/action-semantic-pull-request@v3.0.0
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/sentry-production.yml

@@ -8,14 +8,13 @@ on:
 jobs:
   release:
     runs-on: ubuntu-latest
-
     steps:
       - uses: actions/checkout@v1.0.0
 
-      - name: Setup Node.js 14.x
+      - name: Setup Node.js 12.x
         uses: actions/setup-node@v1
         with:
-          node-version: 14.x
+          node-version: 12.x
 
       - name: Install and build
         run: |

.github/workflows/test.yml

@@ -1,6 +1,10 @@
 name: Tests
 
-on: pull_request
+on:
+  push:
+    branches:
+      - master
+  pull_request:
 
 jobs:
   test:
@@ -9,12 +13,14 @@ jobs:
     steps:
       - uses: actions/checkout@v1
 
-      - name: Setup Node.js 14.x
+      - name: Setup Node.js 12.x
         uses: actions/setup-node@v1
         with:
-          node-version: 14.x
+          node-version: 12.x
 
       - name: Install and test
         run: |
           npm ci
           npm run test:app
+        env:
+          CI: true

.gitignore

@@ -1,18 +1,10 @@
-.DS_Store
-.env.development.local
-.env.local
-.env.production.local
-.env.test.local
-.envrc
-.eslintcache
-.idea
-.vercel
-.vscode
 *.log
-*.tgz
+.DS_Store
+.envrc
+.now
+.vscode
 build
-dist
-firebase
+firebase/
 logs
 node_modules
 npm-debug.log*
@@ -20,3 +12,4 @@ static
 yarn-debug.log*
 yarn-error.log*
 yarn.lock
+.idea

.lintstagedrc.js

@@ -1,7 +1,7 @@
 const { CLIEngine } = require("eslint");
 
 // see https://github.com/okonet/lint-staged#how-can-i-ignore-files-from-eslintignore-
-// for explanation
+//  for explanation
 const cli = new CLIEngine({});
 
 module.exports = {

.prettierrc

@@ -1,4 +1,3 @@
 {
-  "proseWrap": "never",
   "trailingComma": "all"
 }

CHANGELOG.md

@@ -1,3 +0,0 @@
-## 2020-10-13
-
-- Added ability to embed scene source into exported PNG/SVG files so you can import the scene from them (open via `Load` button or drag & drop). #2219

CONTRIBUTING.md

@@ -8,7 +8,8 @@
 1. Run `npm install` to install dependencies
 1. Create a branch for your PR with `git checkout -b your-branch-name`
 
-> To keep `master` branch pointing to remote repository and make pull requests from branches on your fork. To do this, run:
+> To keep `master` branch pointing to remote repository and make
+> pull requests from branches on your fork. To do this, run:
 >
 > ```sh
 > git remote add upstream https://github.com/excalidraw/excalidraw.git
@@ -24,41 +25,3 @@
 1. Tap on `Fork Sandbox`
 1. Write your code
 1. Commit and PR automatically
-
-## Pull Request Guidelines
-
-Don't worry if you get any of the below wrong, or if you don't know how. We'll gladly help out.
-
-### Title
-
-Make sure the title starts with a semantic prefix:
-
-- **feat**: A new feature
-- **fix**: A bug fix
-- **improvement**: An improvement to a current feature
-- **docs**: Documentation only changes
-- **style**: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
-- **refactor**: A code change that neither fixes a bug nor adds a feature
-- **perf**: A code change that improves performance
-- **test**: Adding missing tests or correcting existing tests
-- **build**: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
-- **ci**: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
-- **chore**: Other changes that don't modify src or test files
-- **revert**: Reverts a previous commit
-
-### Changelog
-
-Add a brief description of your pull request to the changelog located here: [`src/packages/excalidraw/CHANGELOG.md`](src/packages/excalidraw/CHANGELOG.md)
-
-Notes:
-
-- Make sure to prepend to the section corresponding with the semantic prefix you selected in the title
-- Link to your pull request - this will require updating the CHANGELOG _after_ creating the pull request
-
-### Testing
-
-Once you submit your pull request it will automatically be tested. Be sure to check the results of the test and fix any issues that arise.
-
-It's also a good idea to consider if your change should include additional tests. This is highly recommended for new features or bug-fixes. For example, it's good practice to create a test for each bug you fix which ensures that we don't regress the code in the future.
-
-Finally - always manually test your changes using the convenient staging environment deployed for each pull request. As much as local development attempts to replicate production, there can still be subtle differences in behavior. For larger features consider testing your change in multiple browsers as well.

Dockerfile

@@ -1,17 +1,16 @@
 FROM node:14-alpine AS build
 
-WORKDIR /opt/node_app
+ENV NODE_ENV=production
+WORKDIR /usr/src/app
 
 COPY package.json package-lock.json ./
-RUN npm i --no-optional
-
-ARG NODE_ENV=production
+RUN npm install
 
 COPY . .
-RUN npm run build:app:docker
+RUN npm run build:app
 
 FROM nginx:1.17-alpine
 
-COPY --from=build /opt/node_app/build /usr/share/nginx/html
+COPY --from=build /usr/src/app/build /usr/share/nginx/html
 
 HEALTHCHECK CMD wget -q -O /dev/null http://localhost || exit 1

README.md

@@ -1,13 +1,13 @@
 <div align="center" style="display:flex;flex-direction:column;">
   <a href="https://excalidraw.com">
-    <img width="540" src="./public/og-image-sm.png" alt="Excalidraw logo: Sketch handrawn like diagrams." />
+    <img src="./public/og-image.png" alt="Excalidraw logo: Sketch handrawn like diagrams." />
   </a>
-  <h3>Virtual whiteboard for sketching hand-drawn like diagrams.<br>Collaborative and end to end encrypted.</h3>
+  <h3>Excalidraw is a whiteboard tool that lets you easily sketch diagrams with a hand-drawn feel.</h3>
   <p>
     <a href="https://twitter.com/Excalidraw">
       <img alt="Follow Excalidraw on Twitter" src="https://img.shields.io/twitter/follow/excalidraw.svg?label=follow+excalidraw&style=social&logo=twitter">
     </a>
-    <a target="_blank" href="https://crowdin.com/project/excalidraw">
+    <a title="Crowdin" target="_blank" href="https://crowdin.com/project/excalidraw">
       <img src="https://badges.crowdin.net/excalidraw/localized.svg">
     </a>
   </p>
@@ -15,53 +15,11 @@
 
 ## Try it now
 
-Go to [excalidraw.com](https://excalidraw.com) to start sketching.
+Go to https://excalidraw.com to start sketching.
 
-Read the latest news and updates on our [blog](https://blog.excalidraw.com). A good start is to see all the updates of [One Year of Excalidraw](https://blog.excalidraw.com/one-year-of-excalidraw/).
+Read our [blog](https://blog.excalidraw.com) and follow the [guides](https://howto.excalidraw.com) to learn more about Excalidraw and how to use it effectively.
 
-## Documentation
-
-### Shortcuts
-
-You can almost do anything with shortcuts. Click on the help icon on the bottom right corner to see them all.
-
-### Curved lines and arrows
-
-Choose line or arrow and click click click instead of drag.
-
-### Charts
-
-You can easily create charts by copy pasting data from Excel or just plain comma separated text.
-
-### Translating
-
-To translate Excalidraw into other languages, please visit [our Crowdin page](https://crowdin.com/project/excalidraw). To add a new language, [open an issue](https://github.com/excalidraw/excalidraw/issues/new) so we can get things set up on our end first.
-
-Translations will be available on the app if they exceed a certain threshold of completion (currently 85%).
-
-### Create a collaboration session manually
-
-In order to create a session manually you just need to generate a link of this form:
-
-```
-https://excalidraw.com/#room=[0-9a-f]{20},[a-zA-Z0-9_-]{22}
-```
-
-#### Example
-
-```
-https://excalidraw.com/#room=91bd46ae3aa84dff9d20,pfLqgEoY1c2ioq8LmGwsFA
-```
-
-The first set of digits is the room. This is visible from the server that’s going to dispatch messages to everyone that knows this number.
-
-The second set of digits is the encryption key. The Excalidraw server doesn’t know about it. This is what all the participants use to encrypt/decrypt the messages.
-
-## Shape libraries
-
-Find a growing list of libraries containing assets for your drawings at [libraries.excalidraw.com](https://libraries.excalidraw.com).
-
-## Developement
+## Run the code
 
 ### Code Sandbox
 
@@ -90,38 +48,78 @@ git clone https://github.com/excalidraw/excalidraw.git
 | `npm run test:update` | Update test snapshots             |
 | `npm run test:code`   | Test for formatting with Prettier |
 
-#### Docker Compose
+### Docker Installation
 
-You can use docker-compose to work on excalidraw locally if you don't want to setup a Node.js env.
+A production-ready version for deploying to e.g. Kubernetes or OpenShift can be built using Docker.
+
+#### Docker Compose
 
 ```sh
 docker-compose up --build -d
 ```
 
-### Self hosting
-
-We publish a Docker image with the Excalidraw client at [excalidraw/excalidraw](https://hub.docker.com/r/excalidraw/excalidraw). You can use it to self host your own client under your own domain, on Kubernetes, AWS ECS, etc.
+#### Native Docker
 
 ```sh
 docker build -t excalidraw/excalidraw .
 docker run --rm -dit --name excalidraw -p 5000:80 excalidraw/excalidraw:latest
 ```
 
-The Docker image is free of analytics and other tracking libraries.
-
-**At the moment, self-hosting your own instance doesn't support sharing or collaboration features.**
-
-We are working towards providing a full-fledged solution for self hosting your own Excalidraw.
+After building the image and running the container, open <http://localhost:5000> to see the application.
 
 ## Contributing
 
 Pull requests are welcome. For major changes, please [open an issue](https://github.com/excalidraw/excalidraw/issues/new) first to discuss what you would like to change.
 
-## Notable used tools
+## Translating
 
-- [Create React App](https://github.com/facebook/create-react-app)
+To translate Excalidraw into other languages, please visit [our Crowdin page](https://crowdin.com/project/excalidraw). To add a new language, [open an issue](https://github.com/excalidraw/excalidraw/issues/new) so we can get things set up on our end first.
+
+## Excalidraw is built using these awesome tools
+
+- [React](https://reactjs.org)
 - [Rough.js](https://roughjs.com)
-- [TypeScript](https://www.typescriptlang.org)
+- [TypeScript](https://typescriptlang.org)
 - [Vercel](https://vercel.com)
 
 And the main source of inspiration for starting the project is the awesome [Zwibbler](https://zwibbler.com/demo/) app.
+
+## Testimonials
+
+<a href="https://twitter.com/Lissy_Sykes/status/1213813117177729026"><img width="398" src="https://user-images.githubusercontent.com/197597/71783813-dbf8a600-2fa0-11ea-9c0d-bb3cc45969e6.png"></a>
+<a href="https://twitter.com/dan_abramov/status/1213762494428262400"><img width="398" src="https://user-images.githubusercontent.com/197597/71783990-4d395880-2fa3-11ea-9ad7-186138db5003.png"></a>
+
+<a href="https://twitter.com/kyehohenberger/status/1214288572037025792"><img width="423" src="https://user-images.githubusercontent.com/197597/71851802-34f13880-308c-11ea-9416-191099e6349c.png"></a>
+<a href="https://twitter.com/lucasazzola/status/1215126440330416128"><img width="429" src="https://user-images.githubusercontent.com/197597/72039003-48e99580-3258-11ea-8daa-85dd055f2a82.png">
+
+<a href="https://twitter.com/jordwalke/status/1214858186789806080"><img width="434" src="https://user-images.githubusercontent.com/197597/72036874-07a1b780-3251-11ea-99e8-6bafd93483a0.png"></a>
+
+## Contributors
+
+### Code Contributors
+
+This project exists thanks to all the people who contribute. [[Contribute](CONTRIBUTING.md)].
+<a href="https://github.com/excalidraw/excalidraw/graphs/contributors"><img src="https://opencollective.com/excalidraw/contributors.svg?width=890&button=false" /></a>
+
+### Financial Contributors
+
+Become a financial contributor and help us sustain our community. [[Contribute](https://opencollective.com/excalidraw/contribute)]
+
+#### Individuals
+
+<a href="https://opencollective.com/excalidraw"><img src="https://opencollective.com/excalidraw/individuals.svg?width=890"></a>
+
+#### Organizations
+
+Support this project with your organization. Your logo will show up here with a link to your website. [[Contribute](https://opencollective.com/excalidraw/contribute)]
+
+<a href="https://opencollective.com/excalidraw/organization/0/website"><img src="https://opencollective.com/excalidraw/organization/0/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/1/website"><img src="https://opencollective.com/excalidraw/organization/1/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/2/website"><img src="https://opencollective.com/excalidraw/organization/2/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/3/website"><img src="https://opencollective.com/excalidraw/organization/3/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/4/website"><img src="https://opencollective.com/excalidraw/organization/4/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/5/website"><img src="https://opencollective.com/excalidraw/organization/5/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/6/website"><img src="https://opencollective.com/excalidraw/organization/6/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/7/website"><img src="https://opencollective.com/excalidraw/organization/7/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/8/website"><img src="https://opencollective.com/excalidraw/organization/8/avatar.svg"></a>
+<a href="https://opencollective.com/excalidraw/organization/9/website"><img src="https://opencollective.com/excalidraw/organization/9/avatar.svg"></a>

docker-compose.yml

@@ -1,25 +1,9 @@
-version: "3.8"
+version: "3"
 
 services:
   excalidraw:
-    build:
-      context: .
-      args:
-        - NODE_ENV=development
+    build: .
     container_name: excalidraw
     ports:
-      - "3000:80"
+      - "5000:80"
     restart: on-failure
-    stdin_open: true
-    healthcheck:
-      disable: true
-    environment:
-      - NODE_ENV=development
-    volumes:
-      - ./:/opt/node_app/app:delegated
-      - ./package.json:/opt/node_app/package.json
-      - ./package-lock.json:/opt/node_app/package-lock.json
-      - notused:/opt/node_app/app/node_modules
-
-volumes:
-  notused:

firebase-project/.firebaserc

@@ -1,5 +0,0 @@
-{
-  "projects": {
-    "default": "excalidraw-room-persistence"
-  }
-}

firebase-project/.gitignore

@@ -1,66 +0,0 @@
-# Logs
-logs
-*.log
-npm-debug.log*
-yarn-debug.log*
-yarn-error.log*
-firebase-debug.log*
-firebase-debug.*.log*
-
-# Firebase cache
-.firebase/
-
-# Firebase config
-
-# Uncomment this if you'd like others to create their own Firebase project.
-# For a team working on the same Firebase project(s), it is recommended to leave
-# it commented so all members can deploy to the same project(s) in .firebaserc.
-# .firebaserc
-
-# Runtime data
-pids
-*.pid
-*.seed
-*.pid.lock
-
-# Directory for instrumented libs generated by jscoverage/JSCover
-lib-cov
-
-# Coverage directory used by tools like istanbul
-coverage
-
-# nyc test coverage
-.nyc_output
-
-# Grunt intermediate storage (http://gruntjs.com/creating-plugins#storing-task-files)
-.grunt
-
-# Bower dependency directory (https://bower.io/)
-bower_components
-
-# node-waf configuration
-.lock-wscript
-
-# Compiled binary addons (http://nodejs.org/api/addons.html)
-build/Release
-
-# Dependency directories
-node_modules/
-
-# Optional npm cache directory
-.npm
-
-# Optional eslint cache
-.eslintcache
-
-# Optional REPL history
-.node_repl_history
-
-# Output of 'npm pack'
-*.tgz
-
-# Yarn Integrity file
-.yarn-integrity
-
-# dotenv environment variables file
-.env

firebase-project/firebase.json

@@ -1,6 +0,0 @@
-{
-  "firestore": {
-    "rules": "firestore.rules",
-    "indexes": "firestore.indexes.json"
-  }
-}

firebase-project/firestore.indexes.json

@@ -1,4 +0,0 @@
-{
-  "indexes": [],
-  "fieldOverrides": []
-}

firebase-project/firestore.rules

@@ -1,10 +0,0 @@
-rules_version = '2';
-service cloud.firestore {
-  match /databases/{database}/documents {
-    match /{document=**} {
-      allow get, write: if true;
-      // never set this to true, otherwise anyone can delete anyone else's drawing.
-      allow list: if false;
-    }
-  }
-}

now.json

@@ -21,5 +21,12 @@
         }
       ]
     }
+  ],
+  "redirects": [
+    {
+      "source": "/([^.]+)",
+      "destination": "/",
+      "statusCode": 301
+    }
   ]
 }

package.json

@@ -19,46 +19,41 @@
     ]
   },
   "dependencies": {
-    "@sentry/browser": "6.0.3",
-    "@sentry/integrations": "6.0.3",
-    "@testing-library/jest-dom": "5.11.9",
-    "@testing-library/react": "11.2.3",
-    "@types/jest": "26.0.20",
-    "@types/react": "17.0.0",
-    "@types/react-dom": "17.0.0",
-    "@types/socket.io-client": "1.4.35",
-    "browser-fs-access": "0.13.0",
-    "clsx": "1.1.1",
-    "firebase": "8.2.5",
-    "i18next-browser-languagedetector": "6.0.1",
+    "@sentry/browser": "5.15.5",
+    "@sentry/integrations": "5.15.5",
+    "@testing-library/jest-dom": "5.7.0",
+    "@testing-library/react": "10.0.4",
+    "@types/jest": "25.2.1",
+    "@types/nanoid": "2.1.0",
+    "@types/react": "16.9.35",
+    "@types/react-dom": "16.9.8",
+    "@types/socket.io-client": "1.4.32",
+    "browser-nativefs": "0.7.3",
+    "i18next-browser-languagedetector": "4.1.1",
     "lodash.throttle": "4.1.1",
-    "nanoid": "3.1.20",
+    "nanoid": "2.1.11",
     "node-sass": "4.14.1",
-    "open-color": "1.8.0",
-    "pako": "1.0.11",
-    "png-chunk-text": "1.0.0",
-    "png-chunks-encode": "1.0.0",
-    "png-chunks-extract": "1.0.0",
+    "open-color": "1.7.0",
     "points-on-curve": "0.2.0",
-    "pwacompat": "2.0.17",
-    "react": "17.0.1",
-    "react-dom": "17.0.1",
-    "react-scripts": "4.0.1",
+    "pwacompat": "2.0.12",
+    "react": "16.13.1",
+    "react-dom": "16.13.1",
+    "react-scripts": "3.4.1",
     "roughjs": "4.3.1",
-    "socket.io-client": "2.3.1",
-    "typescript": "4.1.3"
+    "socket.io-client": "2.3.0",
+    "typescript": "3.8.3"
   },
   "devDependencies": {
     "@types/lodash.throttle": "4.1.6",
-    "@types/pako": "1.0.1",
-    "eslint-config-prettier": "7.2.0",
-    "eslint-plugin-prettier": "3.3.1",
-    "firebase-tools": "9.2.2",
-    "husky": "4.3.8",
-    "jest-canvas-mock": "2.3.0",
-    "lint-staged": "10.5.3",
-    "pepjs": "0.5.3",
-    "prettier": "2.2.1",
+    "asar": "3.0.3",
+    "eslint": "6.8.0",
+    "eslint-config-prettier": "6.11.0",
+    "eslint-plugin-prettier": "3.1.3",
+    "husky": "4.2.5",
+    "jest-canvas-mock": "2.2.0",
+    "lint-staged": "10.2.2",
+    "pepjs": "0.5.2",
+    "prettier": "2.0.5",
     "rewire": "5.0.0"
   },
   "engines": {
@@ -72,33 +67,28 @@
   },
   "jest": {
     "transformIgnorePatterns": [
-      "node_modules/(?!(roughjs|points-on-curve|path-data-parser|points-on-path|browser-fs-access)/)"
-    ],
-    "resetMocks": false
+      "node_modules/(?!(roughjs|points-on-curve|path-data-parser|points-on-path|browser-nativefs)/)"
+    ]
   },
-  "name": "excalidraw",
   "private": true,
   "scripts": {
+    "build": "npm run build:app && npm run build:zip",
     "build-node": "node ./scripts/build-node.js",
-    "build:app:docker": "REACT_APP_DISABLE_SENTRY=true react-scripts build",
-    "build:app": "REACT_APP_GIT_SHA=$VERCEL_GIT_COMMIT_SHA react-scripts build",
-    "build:version": "node ./scripts/build-version.js",
-    "build": "npm run build:app && npm run build:version",
+    "build:app": "REACT_APP_GIT_SHA=$NOW_GITHUB_COMMIT_SHA react-scripts build",
+    "build:zip": "node ./scripts/build-version.js",
     "eject": "react-scripts eject",
+    "fix": "npm run fix:other && npm run fix:code",
     "fix:code": "npm run test:code -- --fix",
     "fix:other": "npm run prettier -- --write",
-    "fix": "npm run fix:other && npm run fix:code",
-    "locales-coverage": "node scripts/build-locales-coverage.js",
-    "locales-coverage:description": "node scripts/locales-coverage-description.js",
     "prettier": "prettier \"**/*.{css,scss,json,md,html,yml}\" --ignore-path=.eslintignore",
     "start": "react-scripts start",
+    "test": "npm run test:app",
     "test:all": "npm run test:typecheck && npm run test:code && npm run test:other && npm run test:app -- --watchAll=false",
-    "test:app": "react-scripts test --passWithNoTests",
+    "test:update": "npm run test:app -- --updateSnapshot --watchAll=false",
+    "test:app": "react-scripts test --env=jsdom --passWithNoTests",
     "test:code": "eslint --max-warnings=0 --ignore-path .gitignore --ext .js,.ts,.tsx .",
     "test:debug": "react-scripts --inspect-brk test --runInBand --no-cache",
     "test:other": "npm run prettier -- --list-different",
-    "test:typecheck": "tsc",
-    "test:update": "npm run test:app -- --updateSnapshot --watchAll=false",
-    "test": "npm run test:app"
+    "test:typecheck": "tsc"
   }
 }

public/index.html

@@ -13,6 +13,18 @@
 
     <meta name="theme-color" content="#000" />
 
+    <!-- Origin Trial token for the Native File System API v1 https://developers.chrome.com/origintrials/#/view_trial/3868592079911256065 (Chrome 78–81) -->
+    <meta
+      http-equiv="origin-trial"
+      content="AoGjY+6r8OQZ5c0AXpK+bbca0pJdCTSHWFqSFNulxiW4OwFBB63kHdDHNo433GeuEOir8IvSovR0LOZLfPnEDAUAAABceyJvcmlnaW4iOiJodHRwczovL3d3dy5leGNhbGlkcmF3LmNvbTo0NDMiLCJmZWF0dXJlIjoiTmF0aXZlRmlsZVN5c3RlbSIsImV4cGlyeSI6MTU4OTMyNzk5OX0="
+    />
+
+    <!-- Origin Trial token for the Native File System API v2 https://developers.chrome.com/origintrials/#/view_trial/4019462667428167681 (Chrome 83–85) -->
+    <meta
+      http-equiv="origin-trial"
+      content="AgMee3sqSZkE0QaZP8f/F9OJj5iSLdnNMRGttIDlOQy552MI4GoL41jyCAHOYsQ8UWM1kPdrb6PVmbSllX/JqwEAAABZeyJvcmlnaW4iOiJodHRwczovL2V4Y2FsaWRyYXcuY29tOjQ0MyIsImZlYXR1cmUiOiJOYXRpdmVGaWxlU3lzdGVtMiIsImV4cGlyeSI6MTU5MDU3MzM5MX0="
+    />
+
     <!-- General tags -->
     <meta
       name="description"
@@ -54,10 +66,7 @@
     <!-- OG tags require absolute url for images -->
     <meta name="twitter:image" content="https://excalidraw.com/og-image.png" />
     <link rel="shortcut icon" href="favicon.ico" type="image/x-icon" />
-
-    <!-- Excalidraw version -->
-    <meta name="version" content="{version}" />
-
+    <link rel="stylesheet" href="fonts.css" type="text/css" />
     <link
       rel="preload"
       href="FG_Virgil.woff2"
@@ -74,7 +83,7 @@
     />
 
     <link
-      href="%REACT_APP_SOCKET_SERVER_URL%/socket.io"
+      href="https://excalidraw-socket.herokuapp.com/socket.io"
       rel="preconnect"
       crossorigin="anonymous"
     />
@@ -82,51 +91,12 @@
     <link
       rel="manifest"
       href="manifest.json"
-      style="--pwacompat-splash-font: 24px Virgil"
+      style="--pwacompat-splash-font: 24px Virgil;"
     />
 
-    <link rel="stylesheet" href="fonts.css" type="text/css" />
-
-    <% if (process.env.REACT_APP_GOOGLE_ANALYTICS_ID) { %>
-    <script
-      async
-      src="https://www.googletagmanager.com/gtag/js?id=%REACT_APP_GOOGLE_ANALYTICS_ID%"
-    ></script>
-    <script>
-      window.dataLayer = window.dataLayer || [];
-      function gtag() {
-        dataLayer.push(arguments);
-      }
-      gtag("js", new Date());
-      gtag("config", "%REACT_APP_GOOGLE_ANALYTICS_ID%");
-    </script>
-    <% } %>
-
-    <!-- FIXME: remove this when we update CRA (fix SW caching) -->
     <style>
-      body {
-        margin: 0;
-        --ui-font: system-ui, BlinkMacSystemFont, -apple-system, Segoe UI,
-          Roboto, Helvetica, Arial, sans-serif;
-        font-family: var(--ui-font);
-        -webkit-text-size-adjust: 100%;
-        -webkit-user-select: none;
-        user-select: none;
-        width: 100vw;
-        height: 100vh;
-      }
-
-      .visually-hidden {
-        position: absolute !important;
-        height: 1px;
-        width: 1px;
-        overflow: hidden;
-        clip: rect(1px, 1px, 1px, 1px);
-        white-space: nowrap; /* added line */
-      }
-
       .LoadingMessage {
-        position: absolute;
+        position: fixed;
         top: 0;
         right: 0;
         bottom: 0;
@@ -137,19 +107,40 @@
         justify-content: center;
         pointer-events: none;
       }
-
       .LoadingMessage span {
-        background-color: var(--button-gray-1);
+        background-color: rgba(255, 255, 255, 0.8);
         border-radius: 5px;
         padding: 0.8em 1.2em;
-        color: var(--popup-text-color);
         font-size: 1.3em;
       }
+      .visually-hidden {
+        position: absolute !important;
+        height: 1px;
+        width: 1px;
+        overflow: hidden;
+        clip: rect(1px 1px 1px 1px); /* IE6, IE7 */
+        clip: rect(1px, 1px, 1px, 1px);
+        white-space: nowrap; /* added line */
+      }
     </style>
+    <script
+      async
+      src="https://www.googletagmanager.com/gtag/js?id=UA-387204-13"
+    ></script>
+    <script>
+      window.dataLayer = window.dataLayer || [];
+      function gtag() {
+        dataLayer.push(arguments);
+      }
+      gtag("js", new Date());
+      gtag("config", "UA-387204-13");
+    </script>
   </head>
 
   <body>
-    <noscript> You need to enable JavaScript to run this app. </noscript>
+    <noscript>
+      You need to enable JavaScript to run this app.
+    </noscript>
     <header>
       <h1 class="visually-hidden">Excalidraw</h1>
     </header>

public/manifest.json

@@ -14,17 +14,8 @@
       "sizes": "256x256"
     }
   ],
-  "start_url": "/",
+  "start_url": ".",
   "display": "standalone",
   "theme_color": "#000000",
-  "background_color": "#ffffff",
-  "file_handlers": [
-    {
-      "action": "/",
-      "accept": {
-        "application/vnd.excalidraw+json": [".excalidraw"]
-      }
-    }
-  ],
-  "capture_links": "new_client"
+  "background_color": "#ffffff"
 }

public/robots.txt

@@ -1,3 +0,0 @@
-user-agent: *
-Allow: /$
-Disallow: /

scripts/build-locales-coverage.js

@@ -1,32 +0,0 @@
-const { readdirSync, writeFileSync } = require("fs");
-const files = readdirSync(`${__dirname}/../src/locales`);
-
-const flatten = (object) =>
-  Object.keys(object).reduce(
-    (initial, current) => ({ ...initial, ...object[current] }),
-    {},
-  );
-
-const locales = files.filter(
-  (file) => file !== "README.md" && file !== "percentages.json",
-);
-
-const percentages = {};
-
-for (let index = 0; index < locales.length; index++) {
-  const currentLocale = locales[index];
-  const data = flatten(require(`${__dirname}/../src/locales/${currentLocale}`));
-
-  const allKeys = Object.keys(data);
-  const translatedKeys = allKeys.filter((item) => data[item] !== "");
-
-  const percentage = (100 * translatedKeys.length) / allKeys.length;
-
-  percentages[currentLocale.replace(".json", "")] = parseInt(percentage);
-}
-
-writeFileSync(
-  `${__dirname}/../src/locales/percentages.json`,
-  `${JSON.stringify(percentages, null, 2)}\n`,
-  "utf8",
-);

scripts/build-node.js

@@ -9,9 +9,9 @@
 //   node build/static/js/build-node.js
 //   open test.png
 
-const rewire = require("rewire");
-const defaults = rewire("react-scripts/scripts/build.js");
-const config = defaults.__get__("config");
+var rewire = require("rewire");
+var defaults = rewire("react-scripts/scripts/build.js");
+var config = defaults.__get__("config");
 
 // Disable multiple chunks
 config.optimization.runtimeChunk = false;
@@ -29,7 +29,7 @@ config.entry = "./src/index-node";
 // By default, webpack is going to replace the require of the canvas.node file
 // to just a string with the path of the canvas.node file. We need to tell
 // webpack to avoid rewriting that dependency.
-config.externals = (context, request, callback) => {
+config.externals = function (context, request, callback) {
   if (/\.node$/.test(request)) {
     return callback(
       null,

scripts/build-version.js

@@ -2,60 +2,36 @@
 
 const fs = require("fs");
 const path = require("path");
-const versionFile = path.join("build", "version.json");
-const indexFile = path.join("build", "index.html");
+const asar = require("asar");
 
-const versionDate = (date) => date.toISOString().replace(".000", "");
+const zero = (digit) => `0${digit}`.slice(-2);
 
-const commitHash = () => {
-  try {
-    return require("child_process")
-      .execSync("git rev-parse --short HEAD")
-      .toString()
-      .trim();
-  } catch {
-    return "none";
-  }
+const versionDate = (date) => {
+  const date_ = `${date.getFullYear()}-${zero(date.getMonth() + 1)}-${zero(
+    date.getDate(),
+  )}`;
+  const time = `${zero(date.getHours())}-${zero(date.getMinutes())}-${zero(
+    date.getSeconds(),
+  )}`;
+  return `${date_}-${time}`;
 };
 
-const commitDate = (hash) => {
-  try {
-    const unix = require("child_process")
-      .execSync(`git show -s --format=%ct ${hash}`)
-      .toString()
-      .trim();
-    const date = new Date(parseInt(unix) * 1000);
-    return versionDate(date);
-  } catch {
-    return versionDate(new Date());
-  }
-};
-
-const getFullVersion = () => {
-  const hash = commitHash();
-  return `${commitDate(hash)}-${hash}`;
-};
+const now = new Date();
 
 const data = JSON.stringify(
   {
-    version: getFullVersion(),
+    asar: `excalidraw.asar`,
+    version: versionDate(now),
   },
   undefined,
   2,
 );
 
-fs.writeFileSync(versionFile, data);
+fs.writeFileSync(path.join("build", "version.json"), data);
 
-// https://stackoverflow.com/a/14181136/8418
-fs.readFile(indexFile, "utf8", (error, data) => {
-  if (error) {
-    return console.error(error);
-  }
-  const result = data.replace(/{version}/g, getFullVersion());
+(async () => {
+  const src = "build/";
+  const dest = path.join("build", `excalidraw.asar`);
 
-  fs.writeFile(indexFile, result, "utf8", (error) => {
-    if (error) {
-      return console.error(error);
-    }
-  });
-});
+  await asar.createPackage(src, dest);
+})();

scripts/locales-coverage-description.js

@@ -1,161 +0,0 @@
-const fs = require("fs");
-
-const THRESSHOLD = 85;
-
-const crowdinMap = {
-  "ar-SA": "en-ar",
-  "bg-BG": "en-bg",
-  "ca-ES": "en-ca",
-  "de-DE": "en-de",
-  "el-GR": "en-el",
-  "es-ES": "en-es",
-  "fa-IR": "en-fa",
-  "fi-FI": "en-fi",
-  "fr-FR": "en-fr",
-  "he-IL": "en-he",
-  "hi-IN": "en-hi",
-  "hu-HU": "en-hu",
-  "id-ID": "en-id",
-  "it-IT": "en-it",
-  "ja-JP": "en-ja",
-  "kab-KAB": "en-kab",
-  "ko-KR": "en-ko",
-  "my-MM": "en-my",
-  "nb-NO": "en-nb",
-  "nl-NL": "en-nl",
-  "nn-NO": "en-nnno",
-  "pa-IN": "en-pain",
-  "pl-PL": "en-pl",
-  "pt-BR": "en-ptbr",
-  "pt-PT": "en-pt",
-  "ro-RO": "en-ro",
-  "ru-RU": "en-ru",
-  "sk-SK": "en-sk",
-  "sv-SE": "en-sv",
-  "tr-TR": "en-tr",
-  "uk-UA": "en-uk",
-  "zh-CN": "en-zhcn",
-  "zh-TW": "en-zhtw",
-};
-
-const flags = {
-  "ar-SA": "🇸🇦",
-  "bg-BG": "🇧🇬",
-  "ca-ES": "🏳",
-  "de-DE": "🇩🇪",
-  "el-GR": "🇬🇷",
-  "es-ES": "🇪🇸",
-  "fa-IR": "🇮🇷",
-  "fi-FI": "🇫🇮",
-  "fr-FR": "🇫🇷",
-  "he-IL": "🇮🇱",
-  "hi-IN": "🇮🇳",
-  "hu-HU": "🇭🇺",
-  "id-ID": "🇮🇩",
-  "it-IT": "🇮🇹",
-  "ja-JP": "🇯🇵",
-  "kab-KAB": "🏳",
-  "ko-KR": "🇰🇷",
-  "my-MM": "🇲🇲",
-  "nb-NO": "🇳🇴",
-  "nl-NL": "🇳🇱",
-  "nn-NO": "🇳🇴",
-  "pa-IN": "🇮🇳",
-  "pl-PL": "🇵🇱",
-  "pt-BR": "🇧🇷",
-  "pt-PT": "🇵🇹",
-  "ro-RO": "🇷🇴",
-  "ru-RU": "🇷🇺",
-  "sk-SK": "🇸🇰",
-  "sv-SE": "🇸🇪",
-  "tr-TR": "🇹🇷",
-  "uk-UA": "🇺🇦",
-  "zh-CN": "🇨🇳",
-  "zh-TW": "🇹🇼",
-};
-
-const languages = {
-  "ar-SA": "العربية",
-  "bg-BG": "Български",
-  "ca-ES": "Català",
-  "de-DE": "Deutsch",
-  "el-GR": "Ελληνικά",
-  "es-ES": "Español",
-  "fa-IR": "فارسی",
-  "fi-FI": "Suomi",
-  "fr-FR": "Français",
-  "he-IL": "עברית",
-  "hi-IN": "हिन्दी",
-  "hu-HU": "Magyar",
-  "id-ID": "Bahasa Indonesia",
-  "it-IT": "Italiano",
-  "ja-JP": "日本語",
-  "kab-KAB": "Taqbaylit",
-  "ko-KR": "한국어",
-  "my-MM": "Burmese",
-  "nb-NO": "Norsk bokmål",
-  "nl-NL": "Nederlands",
-  "nn-NO": "Norsk nynorsk",
-  "pa-IN": "ਪੰਜਾਬੀ",
-  "pl-PL": "Polski",
-  "pt-BR": "Português Brasileiro",
-  "pt-PT": "Português",
-  "ro-RO": "Română",
-  "ru-RU": "Русский",
-  "sk-SK": "Slovenčina",
-  "sv-SE": "Svenska",
-  "tr-TR": "Türkçe",
-  "uk-UA": "Українська",
-  "zh-CN": "简体中文",
-  "zh-TW": "繁體中文",
-};
-
-const percentages = fs.readFileSync(
-  `${__dirname}/../src/locales/percentages.json`,
-);
-const rowData = JSON.parse(percentages);
-
-const coverages = Object.entries(rowData)
-  .sort(([, a], [, b]) => b - a)
-  .reduce((r, [k, v]) => ({ ...r, [k]: v }), {});
-
-const boldIf = (text, condition) => (condition ? `**${text}**` : text);
-
-const printHeader = () => {
-  let result = "| | Flag | Locale | % |\n";
-  result += "| :--: | :--: | -- | :--: |";
-  return result;
-};
-
-const printRow = (id, locale, coverage) => {
-  const isOver = coverage >= THRESSHOLD;
-  let result = `| ${isOver ? id : "..."} | `;
-  result += `${locale in flags ? flags[locale] : ""} | `;
-  const language = locale in languages ? languages[locale] : locale;
-  if (locale in crowdinMap && crowdinMap[locale]) {
-    result += `[${boldIf(
-      language,
-      isOver,
-    )}](https://crowdin.com/translate/excalidraw/10/${crowdinMap[locale]}) | `;
-  } else {
-    result += `${boldIf(language, isOver)} | `;
-  }
-  result += `${coverage === 100 ? "💯" : boldIf(coverage, isOver)} |`;
-  return result;
-};
-
-console.info(
-  `Each language must be at least **${THRESSHOLD}%** translated in order to appear on Excalidraw. Join us on [Crowdin](https://crowdin.com/project/excalidraw) and help us translate your own language. **Can't find yours yet?** Open an [issue](https://github.com/excalidraw/excalidraw/issues/new) and we'll add it to the list.`,
-);
-console.info("\n\r");
-console.info(printHeader());
-let index = 1;
-for (const coverage in coverages) {
-  if (coverage === "en") {
-    continue;
-  }
-  console.info(printRow(index, coverage, coverages[coverage]));
-  index++;
-}
-console.info("\n\r");
-console.info("\\* Languages in **bold** are going to appear on production.");

src/actions/actionAddToLibrary.ts

@@ -1,21 +0,0 @@
-import { register } from "./register";
-import { getSelectedElements } from "../scene";
-import { getNonDeletedElements } from "../element";
-import { deepCopyElement } from "../element/newElement";
-import { Library } from "../data/library";
-
-export const actionAddToLibrary = register({
-  name: "addToLibrary",
-  perform: (elements, appState) => {
-    const selectedElements = getSelectedElements(
-      getNonDeletedElements(elements),
-      appState,
-    );
-
-    Library.loadLibrary().then((items) => {
-      Library.saveLibrary([...items, selectedElements.map(deepCopyElement)]);
-    });
-    return false;
-  },
-  contextItemLabel: "labels.addToLibrary",
-});

src/actions/actionAlign.tsx

@@ -1,207 +0,0 @@
-import React from "react";
-import { alignElements, Alignment } from "../align";
-import {
-  AlignBottomIcon,
-  AlignLeftIcon,
-  AlignRightIcon,
-  AlignTopIcon,
-  CenterHorizontallyIcon,
-  CenterVerticallyIcon,
-} from "../components/icons";
-import { ToolButton } from "../components/ToolButton";
-import { getElementMap, getNonDeletedElements } from "../element";
-import { ExcalidrawElement } from "../element/types";
-import { t } from "../i18n";
-import { KEYS } from "../keys";
-import { getSelectedElements, isSomeElementSelected } from "../scene";
-import { AppState } from "../types";
-import { getShortcutKey } from "../utils";
-import { register } from "./register";
-
-const enableActionGroup = (
-  elements: readonly ExcalidrawElement[],
-  appState: AppState,
-) => getSelectedElements(getNonDeletedElements(elements), appState).length > 1;
-
-const alignSelectedElements = (
-  elements: readonly ExcalidrawElement[],
-  appState: Readonly<AppState>,
-  alignment: Alignment,
-) => {
-  const selectedElements = getSelectedElements(
-    getNonDeletedElements(elements),
-    appState,
-  );
-
-  const updatedElements = alignElements(selectedElements, alignment);
-
-  const updatedElementsMap = getElementMap(updatedElements);
-
-  return elements.map((element) => updatedElementsMap[element.id] || element);
-};
-
-export const actionAlignTop = register({
-  name: "alignTop",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: alignSelectedElements(elements, appState, {
-        position: "start",
-        axis: "y",
-      }),
-      commitToHistory: true,
-    };
-  },
-  keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] && event.shiftKey && event.key === KEYS.ARROW_UP,
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<AlignTopIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.alignTop")} — ${getShortcutKey(
-        "CtrlOrCmd+Shift+Up",
-      )}`}
-      aria-label={t("labels.alignTop")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});
-
-export const actionAlignBottom = register({
-  name: "alignBottom",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: alignSelectedElements(elements, appState, {
-        position: "end",
-        axis: "y",
-      }),
-      commitToHistory: true,
-    };
-  },
-  keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] && event.shiftKey && event.key === KEYS.ARROW_DOWN,
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<AlignBottomIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.alignBottom")} — ${getShortcutKey(
-        "CtrlOrCmd+Shift+Down",
-      )}`}
-      aria-label={t("labels.alignBottom")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});
-
-export const actionAlignLeft = register({
-  name: "alignLeft",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: alignSelectedElements(elements, appState, {
-        position: "start",
-        axis: "x",
-      }),
-      commitToHistory: true,
-    };
-  },
-  keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] && event.shiftKey && event.key === KEYS.ARROW_LEFT,
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<AlignLeftIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.alignLeft")} — ${getShortcutKey(
-        "CtrlOrCmd+Shift+Left",
-      )}`}
-      aria-label={t("labels.alignLeft")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});
-
-export const actionAlignRight = register({
-  name: "alignRight",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: alignSelectedElements(elements, appState, {
-        position: "end",
-        axis: "x",
-      }),
-      commitToHistory: true,
-    };
-  },
-  keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] && event.shiftKey && event.key === KEYS.ARROW_RIGHT,
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<AlignRightIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.alignRight")} — ${getShortcutKey(
-        "CtrlOrCmd+Shift+Right",
-      )}`}
-      aria-label={t("labels.alignRight")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});
-
-export const actionAlignVerticallyCentered = register({
-  name: "alignVerticallyCentered",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: alignSelectedElements(elements, appState, {
-        position: "center",
-        axis: "y",
-      }),
-      commitToHistory: true,
-    };
-  },
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<CenterVerticallyIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={t("labels.centerVertically")}
-      aria-label={t("labels.centerVertically")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});
-
-export const actionAlignHorizontallyCentered = register({
-  name: "alignHorizontallyCentered",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: alignSelectedElements(elements, appState, {
-        position: "center",
-        axis: "x",
-      }),
-      commitToHistory: true,
-    };
-  },
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<CenterHorizontallyIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={t("labels.centerHorizontally")}
-      aria-label={t("labels.centerHorizontally")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});

src/actions/actionCanvas.tsx

@@ -1,21 +1,17 @@
 import React from "react";
-import { getDefaultAppState } from "../appState";
 import { ColorPicker } from "../components/ColorPicker";
-import { resetZoom, trash, zoomIn, zoomOut } from "../components/icons";
+import { getDefaultAppState } from "../appState";
+import { trash, zoomIn, zoomOut, resetZoom } from "../components/icons";
 import { ToolButton } from "../components/ToolButton";
-import { ZOOM_STEP } from "../constants";
-import { getCommonBounds, getNonDeletedElements } from "../element";
-import { newElementWith } from "../element/mutateElement";
-import { ExcalidrawElement } from "../element/types";
 import { t } from "../i18n";
-import useIsMobile from "../is-mobile";
-import { CODES, KEYS } from "../keys";
-import { getNormalizedZoom, getSelectedElements } from "../scene";
-import { centerScrollOn } from "../scene/scroll";
-import { getNewZoom } from "../scene/zoom";
-import { AppState, NormalizedZoomValue } from "../types";
+import { getNormalizedZoom, calculateScrollCenter } from "../scene";
+import { KEYS } from "../keys";
 import { getShortcutKey } from "../utils";
+import useIsMobile from "../is-mobile";
 import { register } from "./register";
+import { newElementWith } from "../element/mutateElement";
+import { AppState, FlooredNumber } from "../types";
+import { getCommonBounds } from "../element";
 
 export const actionChangeViewBackgroundColor = register({
   name: "changeViewBackgroundColor",
@@ -48,14 +44,7 @@ export const actionClearCanvas = register({
       ),
       appState: {
         ...getDefaultAppState(),
-        appearance: appState.appearance,
-        elementLocked: appState.elementLocked,
-        exportBackground: appState.exportBackground,
-        exportEmbedScene: appState.exportEmbedScene,
-        gridSize: appState.gridSize,
-        shouldAddWatermark: appState.shouldAddWatermark,
-        showStats: appState.showStats,
-        pasteDialog: appState.pasteDialog,
+        username: appState.username,
       },
       commitToHistory: true,
     };
@@ -69,6 +58,10 @@ export const actionClearCanvas = register({
       showAriaLabel={useIsMobile()}
       onClick={() => {
         if (window.confirm(t("alerts.clearReset"))) {
+          // TODO: Defined globally, since file handles aren't yet serializable.
+          // Once `FileSystemFileHandle` can be serialized, make this
+          // part of `AppState`.
+          (window as any).handle = null;
           updateData(null);
         }
       }}
@@ -76,19 +69,25 @@ export const actionClearCanvas = register({
   ),
 });
 
+const ZOOM_STEP = 0.1;
+
+const KEY_CODES = {
+  MINUS: "Minus",
+  EQUAL: "Equal",
+  ONE: "Digit1",
+  ZERO: "Digit0",
+  NUM_SUBTRACT: "NumpadSubtract",
+  NUM_ADD: "NumpadAdd",
+  NUM_ZERO: "Numpad0",
+};
+
 export const actionZoomIn = register({
   name: "zoomIn",
   perform: (_elements, appState) => {
-    const zoom = getNewZoom(
-      getNormalizedZoom(appState.zoom.value + ZOOM_STEP),
-      appState.zoom,
-      { left: appState.offsetLeft, top: appState.offsetTop },
-      { x: appState.width / 2, y: appState.height / 2 },
-    );
     return {
       appState: {
         ...appState,
-        zoom,
+        zoom: getNormalizedZoom(appState.zoom + ZOOM_STEP),
       },
       commitToHistory: false,
     };
@@ -105,24 +104,17 @@ export const actionZoomIn = register({
     />
   ),
   keyTest: (event) =>
-    (event.code === CODES.EQUAL || event.code === CODES.NUM_ADD) &&
+    (event.code === KEY_CODES.EQUAL || event.code === KEY_CODES.NUM_ADD) &&
     (event[KEYS.CTRL_OR_CMD] || event.shiftKey),
 });
 
 export const actionZoomOut = register({
   name: "zoomOut",
   perform: (_elements, appState) => {
-    const zoom = getNewZoom(
-      getNormalizedZoom(appState.zoom.value - ZOOM_STEP),
-      appState.zoom,
-      { left: appState.offsetLeft, top: appState.offsetTop },
-      { x: appState.width / 2, y: appState.height / 2 },
-    );
-
     return {
       appState: {
         ...appState,
-        zoom,
+        zoom: getNormalizedZoom(appState.zoom - ZOOM_STEP),
       },
       commitToHistory: false,
     };
@@ -139,7 +131,7 @@ export const actionZoomOut = register({
     />
   ),
   keyTest: (event) =>
-    (event.code === CODES.MINUS || event.code === CODES.NUM_SUBTRACT) &&
+    (event.code === KEY_CODES.MINUS || event.code === KEY_CODES.NUM_SUBTRACT) &&
     (event[KEYS.CTRL_OR_CMD] || event.shiftKey),
 });
 
@@ -149,15 +141,7 @@ export const actionResetZoom = register({
     return {
       appState: {
         ...appState,
-        zoom: getNewZoom(
-          1 as NormalizedZoomValue,
-          appState.zoom,
-          { left: appState.offsetLeft, top: appState.offsetTop },
-          {
-            x: appState.width / 2,
-            y: appState.height / 2,
-          },
-        ),
+        zoom: 1,
       },
       commitToHistory: false,
     };
@@ -174,86 +158,66 @@ export const actionResetZoom = register({
     />
   ),
   keyTest: (event) =>
-    (event.code === CODES.ZERO || event.code === CODES.NUM_ZERO) &&
+    (event.code === KEY_CODES.ZERO || event.code === KEY_CODES.NUM_ZERO) &&
     (event[KEYS.CTRL_OR_CMD] || event.shiftKey),
 });
 
-const zoomValueToFitBoundsOnViewport = (
-  bounds: [number, number, number, number],
-  viewportDimensions: { width: number; height: number },
-) => {
-  const [x1, y1, x2, y2] = bounds;
-  const commonBoundsWidth = x2 - x1;
-  const zoomValueForWidth = viewportDimensions.width / commonBoundsWidth;
-  const commonBoundsHeight = y2 - y1;
-  const zoomValueForHeight = viewportDimensions.height / commonBoundsHeight;
-  const smallestZoomValue = Math.min(zoomValueForWidth, zoomValueForHeight);
-  const zoomAdjustedToSteps =
-    Math.floor(smallestZoomValue / ZOOM_STEP) * ZOOM_STEP;
-  const clampedZoomValueToFitElements = Math.min(
-    Math.max(zoomAdjustedToSteps, ZOOM_STEP),
-    1,
-  );
-  return clampedZoomValueToFitElements as NormalizedZoomValue;
+const calculateZoom = (
+  commonBounds: number[],
+  currentZoom: number,
+  {
+    scrollX,
+    scrollY,
+  }: {
+    scrollX: FlooredNumber;
+    scrollY: FlooredNumber;
+  },
+): number => {
+  const { innerWidth, innerHeight } = window;
+  const [x, y] = commonBounds;
+  const zoomX = -innerWidth / (2 * scrollX + 2 * x - innerWidth);
+  const zoomY = -innerHeight / (2 * scrollY + 2 * y - innerHeight);
+  const margin = 0.01;
+  let newZoom;
+
+  if (zoomX < zoomY) {
+    newZoom = zoomX - margin;
+  } else if (zoomY <= zoomX) {
+    newZoom = zoomY - margin;
+  } else {
+    newZoom = currentZoom;
+  }
+
+  if (newZoom <= 0.1) {
+    return 0.1;
+  }
+  if (newZoom >= 1) {
+    return 1;
+  }
+
+  return newZoom;
 };
 
-const zoomToFitElements = (
-  elements: readonly ExcalidrawElement[],
-  appState: Readonly<AppState>,
-  zoomToSelection: boolean,
-) => {
-  const nonDeletedElements = getNonDeletedElements(elements);
-  const selectedElements = getSelectedElements(nonDeletedElements, appState);
-
-  const commonBounds =
-    zoomToSelection && selectedElements.length > 0
-      ? getCommonBounds(selectedElements)
-      : getCommonBounds(nonDeletedElements);
-
-  const zoomValue = zoomValueToFitBoundsOnViewport(commonBounds, {
-    width: appState.width,
-    height: appState.height,
-  });
-  const newZoom = getNewZoom(zoomValue, appState.zoom, {
-    left: appState.offsetLeft,
-    top: appState.offsetTop,
-  });
-
-  const [x1, y1, x2, y2] = commonBounds;
-  const centerX = (x1 + x2) / 2;
-  const centerY = (y1 + y2) / 2;
-  return {
-    appState: {
-      ...appState,
-      ...centerScrollOn({
-        scenePoint: { x: centerX, y: centerY },
-        viewportDimensions: {
-          width: appState.width,
-          height: appState.height,
-        },
-        zoom: newZoom,
-      }),
-      zoom: newZoom,
-    },
-    commitToHistory: false,
-  };
-};
-
-export const actionZoomToSelected = register({
-  name: "zoomToSelection",
-  perform: (elements, appState) => zoomToFitElements(elements, appState, true),
-  keyTest: (event) =>
-    event.code === CODES.TWO &&
-    event.shiftKey &&
-    !event.altKey &&
-    !event[KEYS.CTRL_OR_CMD],
-});
-
 export const actionZoomToFit = register({
   name: "zoomToFit",
-  perform: (elements, appState) => zoomToFitElements(elements, appState, false),
+  perform: (elements, appState) => {
+    const nonDeletedElements = elements.filter((element) => !element.isDeleted);
+    const scrollCenter = calculateScrollCenter(nonDeletedElements);
+    const commonBounds = getCommonBounds(nonDeletedElements);
+    const zoom = calculateZoom(commonBounds, appState.zoom, scrollCenter);
+
+    return {
+      appState: {
+        ...appState,
+        scrollX: scrollCenter.scrollX,
+        scrollY: scrollCenter.scrollY,
+        zoom,
+      },
+      commitToHistory: false,
+    };
+  },
   keyTest: (event) =>
-    event.code === CODES.ONE &&
+    event.code === KEY_CODES.ONE &&
     event.shiftKey &&
     !event.altKey &&
     !event[KEYS.CTRL_OR_CMD],

src/actions/actionClipboard.tsx

@@ -1,114 +0,0 @@
-import { CODES, KEYS } from "../keys";
-import { register } from "./register";
-import { copyToClipboard } from "../clipboard";
-import { actionDeleteSelected } from "./actionDeleteSelected";
-import { getSelectedElements } from "../scene/selection";
-import { exportCanvas } from "../data/index";
-import { getNonDeletedElements } from "../element";
-import { t } from "../i18n";
-
-export const actionCopy = register({
-  name: "copy",
-  perform: (elements, appState) => {
-    copyToClipboard(getNonDeletedElements(elements), appState);
-
-    return {
-      commitToHistory: false,
-    };
-  },
-  contextItemLabel: "labels.copy",
-  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.code === CODES.C,
-});
-
-export const actionCut = register({
-  name: "cut",
-  perform: (elements, appState, data, app) => {
-    actionCopy.perform(elements, appState, data, app);
-    return actionDeleteSelected.perform(elements, appState, data, app);
-  },
-  contextItemLabel: "labels.cut",
-  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.code === CODES.X,
-});
-
-export const actionCopyAsSvg = register({
-  name: "copyAsSvg",
-  perform: async (elements, appState, _data, app) => {
-    if (!app.canvas) {
-      return {
-        commitToHistory: false,
-      };
-    }
-    const selectedElements = getSelectedElements(
-      getNonDeletedElements(elements),
-      appState,
-    );
-    try {
-      await exportCanvas(
-        "clipboard-svg",
-        selectedElements.length
-          ? selectedElements
-          : getNonDeletedElements(elements),
-        appState,
-        app.canvas,
-        appState,
-      );
-      return {
-        commitToHistory: false,
-      };
-    } catch (error) {
-      console.error(error);
-      return {
-        appState: {
-          ...appState,
-          errorMessage: error.message,
-        },
-        commitToHistory: false,
-      };
-    }
-  },
-  contextItemLabel: "labels.copyAsSvg",
-});
-
-export const actionCopyAsPng = register({
-  name: "copyAsPng",
-  perform: async (elements, appState, _data, app) => {
-    if (!app.canvas) {
-      return {
-        commitToHistory: false,
-      };
-    }
-    const selectedElements = getSelectedElements(
-      getNonDeletedElements(elements),
-      appState,
-    );
-    try {
-      await exportCanvas(
-        "clipboard",
-        selectedElements.length
-          ? selectedElements
-          : getNonDeletedElements(elements),
-        appState,
-        app.canvas,
-        appState,
-      );
-      return {
-        appState: {
-          ...appState,
-          toastMessage: t("toast.copyToClipboardAsPng"),
-        },
-        commitToHistory: false,
-      };
-    } catch (error) {
-      console.error(error);
-      return {
-        appState: {
-          ...appState,
-          errorMessage: error.message,
-        },
-        commitToHistory: false,
-      };
-    }
-  },
-  contextItemLabel: "labels.copyAsPng",
-  keyTest: (event) => event.code === CODES.C && event.altKey && event.shiftKey,
-});

src/actions/actionDeleteSelected.tsx

@@ -1,4 +1,4 @@
-import { isSomeElementSelected } from "../scene";
+import { deleteSelectedElements, isSomeElementSelected } from "../scene";
 import { KEYS } from "../keys";
 import { ToolButton } from "../components/ToolButton";
 import React from "react";
@@ -6,122 +6,14 @@ import { trash } from "../components/icons";
 import { t } from "../i18n";
 import { register } from "./register";
 import { getNonDeletedElements } from "../element";
-import { ExcalidrawElement } from "../element/types";
-import { AppState } from "../types";
-import { newElementWith } from "../element/mutateElement";
-import { getElementsInGroup } from "../groups";
-import { LinearElementEditor } from "../element/linearElementEditor";
-import { fixBindingsAfterDeletion } from "../element/binding";
-
-const deleteSelectedElements = (
-  elements: readonly ExcalidrawElement[],
-  appState: AppState,
-) => {
-  return {
-    elements: elements.map((el) => {
-      if (appState.selectedElementIds[el.id]) {
-        return newElementWith(el, { isDeleted: true });
-      }
-      return el;
-    }),
-    appState: {
-      ...appState,
-      selectedElementIds: {},
-    },
-  };
-};
-
-const handleGroupEditingState = (
-  appState: AppState,
-  elements: readonly ExcalidrawElement[],
-): AppState => {
-  if (appState.editingGroupId) {
-    const siblingElements = getElementsInGroup(
-      getNonDeletedElements(elements),
-      appState.editingGroupId!,
-    );
-    if (siblingElements.length) {
-      return {
-        ...appState,
-        selectedElementIds: { [siblingElements[0].id]: true },
-      };
-    }
-  }
-  return appState;
-};
 
 export const actionDeleteSelected = register({
   name: "deleteSelectedElements",
   perform: (elements, appState) => {
-    if (appState.editingLinearElement) {
-      const {
-        elementId,
-        activePointIndex,
-        startBindingElement,
-        endBindingElement,
-      } = appState.editingLinearElement;
-      const element = LinearElementEditor.getElement(elementId);
-      if (!element) {
-        return false;
-      }
-      if (
-        // case: no point selected → delete whole element
-        activePointIndex == null ||
-        activePointIndex === -1 ||
-        // case: deleting last remaining point
-        element.points.length < 2
-      ) {
-        const nextElements = elements.filter((el) => el.id !== element.id);
-        const nextAppState = handleGroupEditingState(appState, nextElements);
-
-        return {
-          elements: nextElements,
-          appState: {
-            ...nextAppState,
-            editingLinearElement: null,
-          },
-          commitToHistory: false,
-        };
-      }
-
-      // We cannot do this inside `movePoint` because it is also called
-      // when deleting the uncommitted point (which hasn't caused any binding)
-      const binding = {
-        startBindingElement:
-          activePointIndex === 0 ? null : startBindingElement,
-        endBindingElement:
-          activePointIndex === element.points.length - 1
-            ? null
-            : endBindingElement,
-      };
-
-      LinearElementEditor.movePoint(element, activePointIndex, "delete");
-
-      return {
-        elements,
-        appState: {
-          ...appState,
-          editingLinearElement: {
-            ...appState.editingLinearElement,
-            ...binding,
-            activePointIndex: activePointIndex > 0 ? activePointIndex - 1 : 0,
-          },
-        },
-        commitToHistory: true,
-      };
-    }
-
-    let {
+    const {
       elements: nextElements,
       appState: nextAppState,
     } = deleteSelectedElements(elements, appState);
-    fixBindingsAfterDeletion(
-      nextElements,
-      elements.filter(({ id }) => appState.selectedElementIds[id]),
-    );
-
-    nextAppState = handleGroupEditingState(nextAppState, nextElements);
-
     return {
       elements: nextElements,
       appState: {
@@ -136,6 +28,7 @@ export const actionDeleteSelected = register({
     };
   },
   contextItemLabel: "labels.delete",
+  contextMenuOrder: 3,
   keyTest: (event) => event.key === KEYS.BACKSPACE || event.key === KEYS.DELETE,
   PanelComponent: ({ elements, appState, updateData }) => (
     <ToolButton

src/actions/actionDistribute.tsx

@@ -1,91 +0,0 @@
-import React from "react";
-import {
-  DistributeHorizontallyIcon,
-  DistributeVerticallyIcon,
-} from "../components/icons";
-import { ToolButton } from "../components/ToolButton";
-import { distributeElements, Distribution } from "../disitrubte";
-import { getElementMap, getNonDeletedElements } from "../element";
-import { ExcalidrawElement } from "../element/types";
-import { t } from "../i18n";
-import { CODES } from "../keys";
-import { getSelectedElements, isSomeElementSelected } from "../scene";
-import { AppState } from "../types";
-import { getShortcutKey } from "../utils";
-import { register } from "./register";
-
-const enableActionGroup = (
-  elements: readonly ExcalidrawElement[],
-  appState: AppState,
-) => getSelectedElements(getNonDeletedElements(elements), appState).length > 1;
-
-const distributeSelectedElements = (
-  elements: readonly ExcalidrawElement[],
-  appState: Readonly<AppState>,
-  distribution: Distribution,
-) => {
-  const selectedElements = getSelectedElements(
-    getNonDeletedElements(elements),
-    appState,
-  );
-
-  const updatedElements = distributeElements(selectedElements, distribution);
-
-  const updatedElementsMap = getElementMap(updatedElements);
-
-  return elements.map((element) => updatedElementsMap[element.id] || element);
-};
-
-export const distributeHorizontally = register({
-  name: "distributeHorizontally",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: distributeSelectedElements(elements, appState, {
-        space: "between",
-        axis: "x",
-      }),
-      commitToHistory: true,
-    };
-  },
-  keyTest: (event) => event.altKey && event.code === CODES.H,
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<DistributeHorizontallyIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.distributeHorizontally")} — ${getShortcutKey(
-        "Alt+H",
-      )}`}
-      aria-label={t("labels.distributeHorizontally")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});
-
-export const distributeVertically = register({
-  name: "distributeVertically",
-  perform: (elements, appState) => {
-    return {
-      appState,
-      elements: distributeSelectedElements(elements, appState, {
-        space: "between",
-        axis: "y",
-      }),
-      commitToHistory: true,
-    };
-  },
-  keyTest: (event) => event.altKey && event.code === CODES.V,
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<DistributeVerticallyIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.distributeVertically")} — ${getShortcutKey("Alt+V")}`}
-      aria-label={t("labels.distributeVertically")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    />
-  ),
-});

src/actions/actionDuplicateSelection.tsx

@@ -8,63 +8,32 @@ import { ToolButton } from "../components/ToolButton";
 import { clone } from "../components/icons";
 import { t } from "../i18n";
 import { getShortcutKey } from "../utils";
-import { LinearElementEditor } from "../element/linearElementEditor";
-import { mutateElement } from "../element/mutateElement";
-import {
-  selectGroupsForSelectedElements,
-  getSelectedGroupForElement,
-  getElementsInGroup,
-} from "../groups";
-import { AppState } from "../types";
-import { fixBindingsAfterDuplication } from "../element/binding";
-import { ActionResult } from "./types";
-import { GRID_SIZE } from "../constants";
 
 export const actionDuplicateSelection = register({
   name: "duplicateSelection",
   perform: (elements, appState) => {
-    // duplicate point if selected while editing multi-point element
-    if (appState.editingLinearElement) {
-      const { activePointIndex, elementId } = appState.editingLinearElement;
-      const element = LinearElementEditor.getElement(elementId);
-      if (!element || activePointIndex === null) {
-        return false;
-      }
-      const { points } = element;
-      const selectedPoint = points[activePointIndex];
-      const nextPoint = points[activePointIndex + 1];
-      mutateElement(element, {
-        points: [
-          ...points.slice(0, activePointIndex + 1),
-          nextPoint
-            ? [
-                (selectedPoint[0] + nextPoint[0]) / 2,
-                (selectedPoint[1] + nextPoint[1]) / 2,
-              ]
-            : [selectedPoint[0] + 30, selectedPoint[1] + 30],
-          ...points.slice(activePointIndex + 1),
-        ],
-      });
-      return {
-        appState: {
-          ...appState,
-          editingLinearElement: {
-            ...appState.editingLinearElement,
-            activePointIndex: activePointIndex + 1,
-          },
-        },
-        elements,
-        commitToHistory: true,
-      };
-    }
-
     return {
-      ...duplicateElements(elements, appState),
+      appState,
+      elements: elements.reduce(
+        (acc: Array<ExcalidrawElement>, element: ExcalidrawElement) => {
+          if (appState.selectedElementIds[element.id]) {
+            const newElement = duplicateElement(element, {
+              x: element.x + 10,
+              y: element.y + 10,
+            });
+            appState.selectedElementIds[newElement.id] = true;
+            delete appState.selectedElementIds[element.id];
+            return acc.concat([element, newElement]);
+          }
+          return acc.concat(element);
+        },
+        [],
+      ),
       commitToHistory: true,
     };
   },
   contextItemLabel: "labels.duplicateSelection",
-  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.key === KEYS.D,
+  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.key === "d",
   PanelComponent: ({ elements, appState, updateData }) => (
     <ToolButton
       type="button"
@@ -78,74 +47,3 @@ export const actionDuplicateSelection = register({
     />
   ),
 });
-
-const duplicateElements = (
-  elements: readonly ExcalidrawElement[],
-  appState: AppState,
-): Partial<ActionResult> => {
-  const groupIdMap = new Map();
-  const newElements: ExcalidrawElement[] = [];
-  const oldElements: ExcalidrawElement[] = [];
-  const oldIdToDuplicatedId = new Map();
-
-  const duplicateAndOffsetElement = (element: ExcalidrawElement) => {
-    const newElement = duplicateElement(
-      appState.editingGroupId,
-      groupIdMap,
-      element,
-      {
-        x: element.x + GRID_SIZE / 2,
-        y: element.y + GRID_SIZE / 2,
-      },
-    );
-    oldIdToDuplicatedId.set(element.id, newElement.id);
-    oldElements.push(element);
-    newElements.push(newElement);
-    return newElement;
-  };
-
-  const finalElements: ExcalidrawElement[] = [];
-
-  let index = 0;
-  while (index < elements.length) {
-    const element = elements[index];
-    if (appState.selectedElementIds[element.id]) {
-      if (element.groupIds.length) {
-        const groupId = getSelectedGroupForElement(appState, element);
-        // if group selected, duplicate it atomically
-        if (groupId) {
-          const groupElements = getElementsInGroup(elements, groupId);
-          finalElements.push(
-            ...groupElements,
-            ...groupElements.map((element) =>
-              duplicateAndOffsetElement(element),
-            ),
-          );
-          index = index + groupElements.length;
-          continue;
-        }
-      }
-      finalElements.push(element, duplicateAndOffsetElement(element));
-    } else {
-      finalElements.push(element);
-    }
-    index++;
-  }
-
-  fixBindingsAfterDuplication(finalElements, oldElements, oldIdToDuplicatedId);
-
-  return {
-    elements: finalElements,
-    appState: selectGroupsForSelectedElements(
-      {
-        ...appState,
-        selectedGroupIds: {},
-        selectedElementIds: newElements.reduce((acc, element) => {
-          acc[element.id] = true;
-          return acc;
-        }, {} as any),
-      },
-      getNonDeletedElements(finalElements),
-    ),
-  };
-};

src/actions/actionExport.tsx

@@ -1,20 +1,16 @@
 import React from "react";
-import { trackEvent } from "../analytics";
-import { load, questionCircle, save, saveAs } from "../components/icons";
 import { ProjectName } from "../components/ProjectName";
+import { saveAsJSON, loadFromJSON } from "../data";
+import { load, save } from "../components/icons";
 import { ToolButton } from "../components/ToolButton";
-import "../components/ToolIcon.scss";
-import { Tooltip } from "../components/Tooltip";
-import { loadFromJSON, saveAsJSON } from "../data";
 import { t } from "../i18n";
 import useIsMobile from "../is-mobile";
-import { KEYS } from "../keys";
 import { register } from "./register";
+import { KEYS } from "../keys";
 
 export const actionChangeProjectName = register({
   name: "changeProjectName",
   perform: (_elements, appState, value) => {
-    trackEvent("change", "title");
     return { appState: { ...appState, name: value }, commitToHistory: false };
   },
   PanelComponent: ({ appState, updateData }) => (
@@ -46,33 +42,6 @@ export const actionChangeExportBackground = register({
   ),
 });
 
-export const actionChangeExportEmbedScene = register({
-  name: "changeExportEmbedScene",
-  perform: (_elements, appState, value) => {
-    return {
-      appState: { ...appState, exportEmbedScene: value },
-      commitToHistory: false,
-    };
-  },
-  PanelComponent: ({ appState, updateData }) => (
-    <label style={{ display: "flex" }}>
-      <input
-        type="checkbox"
-        checked={appState.exportEmbedScene}
-        onChange={(event) => updateData(event.target.checked)}
-      />{" "}
-      {t("labels.exportEmbedScene")}
-      <Tooltip
-        label={t("labels.exportEmbedScene_details")}
-        position="above"
-        long={true}
-      >
-        <div className="TooltipIcon">{questionCircle}</div>
-      </Tooltip>
-    </label>
-  ),
-});
-
 export const actionChangeShouldAddWatermark = register({
   name: "changeShouldAddWatermark",
   perform: (_elements, appState, value) => {
@@ -95,19 +64,13 @@ export const actionChangeShouldAddWatermark = register({
 
 export const actionSaveScene = register({
   name: "saveScene",
-  perform: async (elements, appState, value) => {
-    try {
-      const { fileHandle } = await saveAsJSON(elements, appState);
-      return { commitToHistory: false, appState: { ...appState, fileHandle } };
-    } catch (error) {
-      if (error?.name !== "AbortError") {
-        console.error(error);
-      }
-      return { commitToHistory: false };
-    }
+  perform: (elements, appState, value) => {
+    saveAsJSON(elements, appState).catch((error) => console.error(error));
+    return { commitToHistory: false };
+  },
+  keyTest: (event) => {
+    return event.key === "s" && event[KEYS.CTRL_OR_CMD];
   },
-  keyTest: (event) =>
-    event.key === KEYS.S && event[KEYS.CTRL_OR_CMD] && !event.shiftKey,
   PanelComponent: ({ updateData }) => (
     <ToolButton
       type="button"
@@ -120,72 +83,42 @@ export const actionSaveScene = register({
   ),
 });
 
-export const actionSaveAsScene = register({
-  name: "saveAsScene",
-  perform: async (elements, appState, value) => {
-    try {
-      const { fileHandle } = await saveAsJSON(elements, {
-        ...appState,
-        fileHandle: null,
-      });
-      return { commitToHistory: false, appState: { ...appState, fileHandle } };
-    } catch (error) {
-      if (error?.name !== "AbortError") {
-        console.error(error);
-      }
-      return { commitToHistory: false };
-    }
-  },
-  keyTest: (event) =>
-    event.key === KEYS.S && event.shiftKey && event[KEYS.CTRL_OR_CMD],
-  PanelComponent: ({ updateData }) => (
-    <ToolButton
-      type="button"
-      icon={saveAs}
-      title={t("buttons.saveAs")}
-      aria-label={t("buttons.saveAs")}
-      showAriaLabel={useIsMobile()}
-      hidden={
-        !("chooseFileSystemEntries" in window || "showOpenFilePicker" in window)
-      }
-      onClick={() => updateData(null)}
-    />
-  ),
-});
-
 export const actionLoadScene = register({
   name: "loadScene",
-  perform: async (elements, appState) => {
-    try {
-      const {
-        elements: loadedElements,
-        appState: loadedAppState,
-      } = await loadFromJSON(appState);
-      return {
-        elements: loadedElements,
-        appState: loadedAppState,
-        commitToHistory: true,
-      };
-    } catch (error) {
-      if (error?.name === "AbortError") {
-        return false;
-      }
-      return {
-        elements,
-        appState: { ...appState, errorMessage: error.message },
-        commitToHistory: false,
-      };
-    }
+  perform: (
+    elements,
+    appState,
+    { elements: loadedElements, appState: loadedAppState, error },
+  ) => {
+    return {
+      elements: loadedElements,
+      appState: {
+        ...loadedAppState,
+        errorMessage: error,
+      },
+      commitToHistory: false,
+    };
   },
-  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.key === KEYS.O,
-  PanelComponent: ({ updateData, appState }) => (
+  PanelComponent: ({ updateData }) => (
     <ToolButton
       type="button"
       icon={load}
       title={t("buttons.load")}
       aria-label={t("buttons.load")}
       showAriaLabel={useIsMobile()}
-      onClick={updateData}
+      onClick={() => {
+        loadFromJSON()
+          .then(({ elements, appState }) => {
+            updateData({ elements: elements, appState: appState });
+          })
+          .catch((error) => {
+            // if user cancels, ignore the error
+            if (error.name === "AbortError") {
+              return;
+            }
+            updateData({ error: error.message });
+          });
+      }}
     />
   ),
 });

src/actions/actionFinalize.tsx

@@ -8,47 +8,10 @@ import { t } from "../i18n";
 import { register } from "./register";
 import { mutateElement } from "../element/mutateElement";
 import { isPathALoop } from "../math";
-import { LinearElementEditor } from "../element/linearElementEditor";
-import Scene from "../scene/Scene";
-import {
-  maybeBindLinearElement,
-  bindOrUnbindLinearElement,
-} from "../element/binding";
-import { isBindingElement } from "../element/typeChecks";
 
 export const actionFinalize = register({
   name: "finalize",
   perform: (elements, appState) => {
-    if (appState.editingLinearElement) {
-      const {
-        elementId,
-        startBindingElement,
-        endBindingElement,
-      } = appState.editingLinearElement;
-      const element = LinearElementEditor.getElement(elementId);
-
-      if (element) {
-        if (isBindingElement(element)) {
-          bindOrUnbindLinearElement(
-            element,
-            startBindingElement,
-            endBindingElement,
-          );
-        }
-        return {
-          elements:
-            element.points.length < 2 || isInvisiblySmallElement(element)
-              ? elements.filter((el) => el.id !== element.id)
-              : undefined,
-          appState: {
-            ...appState,
-            editingLinearElement: null,
-          },
-          commitToHistory: true,
-        };
-      }
-    }
-
     let newElements = elements;
     if (window.document.activeElement instanceof HTMLElement) {
       window.document.activeElement.blur();
@@ -83,17 +46,16 @@ export const actionFinalize = register({
       // If the multi point line closes the loop,
       // set the last point to first point.
       // This ensures that loop remains closed at different scales.
-      const isLoop = isPathALoop(multiPointElement.points);
       if (
         multiPointElement.type === "line" ||
         multiPointElement.type === "draw"
       ) {
-        if (isLoop) {
+        if (isPathALoop(multiPointElement.points)) {
           const linePoints = multiPointElement.points;
           const firstPoint = linePoints[0];
           mutateElement(multiPointElement, {
-            points: linePoints.map((point, index) =>
-              index === linePoints.length - 1
+            points: linePoints.map((point, i) =>
+              i === linePoints.length - 1
                 ? ([firstPoint[0], firstPoint[1]] as const)
                 : point,
             ),
@@ -101,31 +63,11 @@ export const actionFinalize = register({
         }
       }
 
-      if (
-        isBindingElement(multiPointElement) &&
-        !isLoop &&
-        multiPointElement.points.length > 1
-      ) {
-        const [x, y] = LinearElementEditor.getPointAtIndexGlobalCoordinates(
-          multiPointElement,
-          -1,
-        );
-        maybeBindLinearElement(
-          multiPointElement,
-          appState,
-          Scene.getScene(multiPointElement)!,
-          { x, y },
-        );
-      }
-
-      if (!appState.elementLocked && appState.elementType !== "draw") {
+      if (!appState.elementLocked) {
         appState.selectedElementIds[multiPointElement.id] = true;
       }
     }
-    if (
-      (!appState.elementLocked && appState.elementType !== "draw") ||
-      !multiPointElement
-    ) {
+    if (!appState.elementLocked || !multiPointElement) {
       resetCursor();
     }
     return {
@@ -133,32 +75,27 @@ export const actionFinalize = register({
       appState: {
         ...appState,
         elementType:
-          (appState.elementLocked || appState.elementType === "draw") &&
-          multiPointElement
+          appState.elementLocked && multiPointElement
             ? appState.elementType
             : "selection",
         draggingElement: null,
         multiElement: null,
         editingElement: null,
-        startBoundElement: null,
-        suggestedBindings: [],
         selectedElementIds:
-          multiPointElement &&
-          !appState.elementLocked &&
-          appState.elementType !== "draw"
+          multiPointElement && !appState.elementLocked
             ? {
                 ...appState.selectedElementIds,
                 [multiPointElement.id]: true,
               }
             : appState.selectedElementIds,
       },
-      commitToHistory: appState.elementType === "draw",
+      commitToHistory: false,
     };
   },
   keyTest: (event, appState) =>
     (event.key === KEYS.ESCAPE &&
-      (appState.editingLinearElement !== null ||
-        (!appState.draggingElement && appState.multiElement === null))) ||
+      !appState.draggingElement &&
+      appState.multiElement === null) ||
     ((event.key === KEYS.ESCAPE || event.key === KEYS.ENTER) &&
       appState.multiElement !== null),
   PanelComponent: ({ appState, updateData }) => (

src/actions/actionGroup.tsx

@@ -1,191 +0,0 @@
-import React from "react";
-import { CODES, KEYS } from "../keys";
-import { t } from "../i18n";
-import { getShortcutKey } from "../utils";
-import { register } from "./register";
-import { UngroupIcon, GroupIcon } from "../components/icons";
-import { newElementWith } from "../element/mutateElement";
-import { getSelectedElements, isSomeElementSelected } from "../scene";
-import {
-  getSelectedGroupIds,
-  selectGroup,
-  selectGroupsForSelectedElements,
-  getElementsInGroup,
-  addToGroup,
-  removeFromSelectedGroups,
-  isElementInGroup,
-} from "../groups";
-import { getNonDeletedElements } from "../element";
-import { randomId } from "../random";
-import { ToolButton } from "../components/ToolButton";
-import { ExcalidrawElement } from "../element/types";
-import { AppState } from "../types";
-
-const allElementsInSameGroup = (elements: readonly ExcalidrawElement[]) => {
-  if (elements.length >= 2) {
-    const groupIds = elements[0].groupIds;
-    for (const groupId of groupIds) {
-      if (
-        elements.reduce(
-          (acc, element) => acc && isElementInGroup(element, groupId),
-          true,
-        )
-      ) {
-        return true;
-      }
-    }
-  }
-  return false;
-};
-
-const enableActionGroup = (
-  elements: readonly ExcalidrawElement[],
-  appState: AppState,
-) => {
-  const selectedElements = getSelectedElements(
-    getNonDeletedElements(elements),
-    appState,
-  );
-  return (
-    selectedElements.length >= 2 && !allElementsInSameGroup(selectedElements)
-  );
-};
-
-export const actionGroup = register({
-  name: "group",
-  perform: (elements, appState) => {
-    const selectedElements = getSelectedElements(
-      getNonDeletedElements(elements),
-      appState,
-    );
-    if (selectedElements.length < 2) {
-      // nothing to group
-      return { appState, elements, commitToHistory: false };
-    }
-    // if everything is already grouped into 1 group, there is nothing to do
-    const selectedGroupIds = getSelectedGroupIds(appState);
-    if (selectedGroupIds.length === 1) {
-      const selectedGroupId = selectedGroupIds[0];
-      const elementIdsInGroup = new Set(
-        getElementsInGroup(elements, selectedGroupId).map(
-          (element) => element.id,
-        ),
-      );
-      const selectedElementIds = new Set(
-        selectedElements.map((element) => element.id),
-      );
-      const combinedSet = new Set([
-        ...Array.from(elementIdsInGroup),
-        ...Array.from(selectedElementIds),
-      ]);
-      if (combinedSet.size === elementIdsInGroup.size) {
-        // no incremental ids in the selected ids
-        return { appState, elements, commitToHistory: false };
-      }
-    }
-    const newGroupId = randomId();
-    const updatedElements = elements.map((element) => {
-      if (!appState.selectedElementIds[element.id]) {
-        return element;
-      }
-      return newElementWith(element, {
-        groupIds: addToGroup(
-          element.groupIds,
-          newGroupId,
-          appState.editingGroupId,
-        ),
-      });
-    });
-    // keep the z order within the group the same, but move them
-    // to the z order of the highest element in the layer stack
-    const elementsInGroup = getElementsInGroup(updatedElements, newGroupId);
-    const lastElementInGroup = elementsInGroup[elementsInGroup.length - 1];
-    const lastGroupElementIndex = updatedElements.lastIndexOf(
-      lastElementInGroup,
-    );
-    const elementsAfterGroup = updatedElements.slice(lastGroupElementIndex + 1);
-    const elementsBeforeGroup = updatedElements
-      .slice(0, lastGroupElementIndex)
-      .filter(
-        (updatedElement) => !isElementInGroup(updatedElement, newGroupId),
-      );
-    const updatedElementsInOrder = [
-      ...elementsBeforeGroup,
-      ...elementsInGroup,
-      ...elementsAfterGroup,
-    ];
-
-    return {
-      appState: selectGroup(
-        newGroupId,
-        { ...appState, selectedGroupIds: {} },
-        getNonDeletedElements(updatedElementsInOrder),
-      ),
-      elements: updatedElementsInOrder,
-      commitToHistory: true,
-    };
-  },
-  contextItemLabel: "labels.group",
-  contextItemPredicate: (elements, appState) =>
-    enableActionGroup(elements, appState),
-  keyTest: (event) =>
-    !event.shiftKey && event[KEYS.CTRL_OR_CMD] && event.code === CODES.G,
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      hidden={!enableActionGroup(elements, appState)}
-      type="button"
-      icon={<GroupIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.group")} — ${getShortcutKey("CtrlOrCmd+G")}`}
-      aria-label={t("labels.group")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    ></ToolButton>
-  ),
-});
-
-export const actionUngroup = register({
-  name: "ungroup",
-  perform: (elements, appState) => {
-    const groupIds = getSelectedGroupIds(appState);
-    if (groupIds.length === 0) {
-      return { appState, elements, commitToHistory: false };
-    }
-    const nextElements = elements.map((element) => {
-      const nextGroupIds = removeFromSelectedGroups(
-        element.groupIds,
-        appState.selectedGroupIds,
-      );
-      if (nextGroupIds.length === element.groupIds.length) {
-        return element;
-      }
-      return newElementWith(element, {
-        groupIds: nextGroupIds,
-      });
-    });
-    return {
-      appState: selectGroupsForSelectedElements(
-        { ...appState, selectedGroupIds: {} },
-        getNonDeletedElements(nextElements),
-      ),
-      elements: nextElements,
-      commitToHistory: true,
-    };
-  },
-  keyTest: (event) =>
-    event.shiftKey && event[KEYS.CTRL_OR_CMD] && event.code === CODES.G,
-  contextItemLabel: "labels.ungroup",
-  contextItemPredicate: (elements, appState) =>
-    getSelectedGroupIds(appState).length > 0,
-
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <ToolButton
-      type="button"
-      hidden={getSelectedGroupIds(appState).length === 0}
-      icon={<UngroupIcon appearance={appState.appearance} />}
-      onClick={() => updateData(null)}
-      title={`${t("labels.ungroup")} — ${getShortcutKey("CtrlOrCmd+Shift+G")}`}
-      aria-label={t("labels.ungroup")}
-      visible={isSomeElementSelected(getNonDeletedElements(elements), appState)}
-    ></ToolButton>
-  ),
-});

src/actions/actionHistory.tsx

@@ -3,18 +3,20 @@ import React from "react";
 import { undo, redo } from "../components/icons";
 import { ToolButton } from "../components/ToolButton";
 import { t } from "../i18n";
-import { SceneHistory, HistoryEntry } from "../history";
+import { SceneHistory } from "../history";
 import { ExcalidrawElement } from "../element/types";
 import { AppState } from "../types";
-import { isWindows, KEYS } from "../keys";
+import { KEYS } from "../keys";
 import { getElementMap } from "../element";
 import { newElementWith } from "../element/mutateElement";
-import { fixBindingsAfterDeletion } from "../element/binding";
 
 const writeData = (
   prevElements: readonly ExcalidrawElement[],
   appState: AppState,
-  updater: () => HistoryEntry | null,
+  updater: () => {
+    elements: ExcalidrawElement[];
+    appState: AppState;
+  } | null,
 ): ActionResult => {
   const commitToHistory = false;
   if (
@@ -31,44 +33,40 @@ const writeData = (
     const prevElementMap = getElementMap(prevElements);
     const nextElements = data.elements;
     const nextElementMap = getElementMap(nextElements);
-
-    const deletedElements = prevElements.filter(
-      (prevElement) => !nextElementMap.hasOwnProperty(prevElement.id),
-    );
-    const elements = nextElements
-      .map((nextElement) =>
-        newElementWith(
-          prevElementMap[nextElement.id] || nextElement,
-          nextElement,
-        ),
-      )
-      .concat(
-        deletedElements.map((prevElement) =>
-          newElementWith(prevElement, { isDeleted: true }),
-        ),
-      );
-    fixBindingsAfterDeletion(elements, deletedElements);
-
     return {
-      elements,
+      elements: nextElements
+        .map((nextElement) =>
+          newElementWith(
+            prevElementMap[nextElement.id] || nextElement,
+            nextElement,
+          ),
+        )
+        .concat(
+          prevElements
+            .filter(
+              (prevElement) => !nextElementMap.hasOwnProperty(prevElement.id),
+            )
+            .map((prevElement) =>
+              newElementWith(prevElement, { isDeleted: true }),
+            ),
+        ),
       appState: { ...appState, ...data.appState },
       commitToHistory,
-      syncHistory: true,
     };
   }
   return { commitToHistory };
 };
 
+const testUndo = (shift: boolean) => (event: KeyboardEvent) =>
+  event[KEYS.CTRL_OR_CMD] && /z/i.test(event.key) && event.shiftKey === shift;
+
 type ActionCreator = (history: SceneHistory) => Action;
 
 export const createUndoAction: ActionCreator = (history) => ({
   name: "undo",
   perform: (elements, appState) =>
     writeData(elements, appState, () => history.undoOnce()),
-  keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] &&
-    event.key.toLowerCase() === KEYS.Z &&
-    !event.shiftKey,
+  keyTest: testUndo(false),
   PanelComponent: ({ updateData }) => (
     <ToolButton
       type="button"
@@ -84,11 +82,7 @@ export const createRedoAction: ActionCreator = (history) => ({
   name: "redo",
   perform: (elements, appState) =>
     writeData(elements, appState, () => history.redoOnce()),
-  keyTest: (event) =>
-    (event[KEYS.CTRL_OR_CMD] &&
-      event.shiftKey &&
-      event.key.toLowerCase() === KEYS.Z) ||
-    (isWindows && event.ctrlKey && !event.shiftKey && event.key === KEYS.Y),
+  keyTest: testUndo(true),
   PanelComponent: ({ updateData }) => (
     <ToolButton
       type="button"

src/actions/actionMenu.tsx

@@ -5,7 +5,7 @@ import { t } from "../i18n";
 import { showSelectedShapeActions, getNonDeletedElements } from "../element";
 import { register } from "./register";
 import { allowFullScreen, exitFullScreen, isFullScreen } from "../utils";
-import { CODES, KEYS } from "../keys";
+import { KEYS } from "../keys";
 import { HelpIcon } from "../components/HelpIcon";
 
 export const actionToggleCanvasMenu = register({
@@ -65,7 +65,7 @@ export const actionFullScreen = register({
       commitToHistory: false,
     };
   },
-  keyTest: (event) => event.code === CODES.F && !event[KEYS.CTRL_OR_CMD],
+  keyTest: (event) => event.keyCode === KEYS.F_KEY_CODE,
 });
 
 export const actionShortcuts = register({
@@ -74,13 +74,13 @@ export const actionShortcuts = register({
     return {
       appState: {
         ...appState,
-        showHelpDialog: !appState.showHelpDialog,
+        showShortcutsDialog: true,
       },
       commitToHistory: false,
     };
   },
   PanelComponent: ({ updateData }) => (
-    <HelpIcon title={t("helpDialog.title")} onClick={updateData} />
+    <HelpIcon title={t("shortcutsDialog.title")} onClick={updateData} />
   ),
   keyTest: (event) => event.key === KEYS.QUESTION_MARK,
 });

src/actions/actionNavigate.tsx

@@ -1,58 +0,0 @@
-import React from "react";
-import { getClientColors, getClientInitials } from "../clients";
-import { Avatar } from "../components/Avatar";
-import { centerScrollOn } from "../scene/scroll";
-import { Collaborator } from "../types";
-import { register } from "./register";
-
-export const actionGoToCollaborator = register({
-  name: "goToCollaborator",
-  perform: (_elements, appState, value) => {
-    const point = value as Collaborator["pointer"];
-    if (!point) {
-      return { appState, commitToHistory: false };
-    }
-
-    return {
-      appState: {
-        ...appState,
-        ...centerScrollOn({
-          scenePoint: point,
-          viewportDimensions: {
-            width: appState.width,
-            height: appState.height,
-          },
-          zoom: appState.zoom,
-        }),
-        // Close mobile menu
-        openMenu: appState.openMenu === "canvas" ? null : appState.openMenu,
-      },
-      commitToHistory: false,
-    };
-  },
-  PanelComponent: ({ appState, updateData, id }) => {
-    const clientId = id;
-    if (!clientId) {
-      return null;
-    }
-
-    const collaborator = appState.collaborators.get(clientId);
-
-    if (!collaborator) {
-      return null;
-    }
-
-    const { background, stroke } = getClientColors(clientId);
-    const shortName = getClientInitials(collaborator.username);
-
-    return (
-      <Avatar
-        color={background}
-        border={stroke}
-        onClick={() => updateData(collaborator.pointer)}
-      >
-        {shortName}
-      </Avatar>
-    );
-  },
-});

src/actions/actionProperties.tsx

@@ -1,53 +1,25 @@
 import React from "react";
-import { AppState } from "../../src/types";
-import { ButtonIconSelect } from "../components/ButtonIconSelect";
-import { ButtonSelect } from "../components/ButtonSelect";
-import { ColorPicker } from "../components/ColorPicker";
-import { IconPicker } from "../components/IconPicker";
 import {
-  ArrowheadArrowIcon,
-  ArrowheadBarIcon,
-  ArrowheadDotIcon,
-  ArrowheadNoneIcon,
-  EdgeRoundIcon,
-  EdgeSharpIcon,
-  FillCrossHatchIcon,
-  FillHachureIcon,
-  FillSolidIcon,
-  SloppinessArchitectIcon,
-  SloppinessArtistIcon,
-  SloppinessCartoonistIcon,
-  StrokeStyleDashedIcon,
-  StrokeStyleDottedIcon,
-  StrokeStyleSolidIcon,
-  StrokeWidthIcon,
-} from "../components/icons";
-import { DEFAULT_FONT_FAMILY, DEFAULT_FONT_SIZE } from "../constants";
-import {
-  getNonDeletedElements,
-  isTextElement,
-  redrawTextBoundingBox,
-} from "../element";
-import { newElementWith } from "../element/mutateElement";
-import { isLinearElement, isLinearElementType } from "../element/typeChecks";
-import {
-  Arrowhead,
   ExcalidrawElement,
-  ExcalidrawLinearElement,
   ExcalidrawTextElement,
-  FontFamily,
   TextAlign,
 } from "../element/types";
-import { getLanguage, t } from "../i18n";
-import { randomInteger } from "../random";
 import {
-  canChangeSharpness,
-  canHaveArrowheads,
   getCommonAttributeOfSelectedElements,
-  getTargetElements,
   isSomeElementSelected,
 } from "../scene";
+import { ButtonSelect } from "../components/ButtonSelect";
+import {
+  isTextElement,
+  redrawTextBoundingBox,
+  getNonDeletedElements,
+} from "../element";
+import { ColorPicker } from "../components/ColorPicker";
+import { AppState } from "../../src/types";
+import { t } from "../i18n";
+import { DEFAULT_FONT } from "../appState";
 import { register } from "./register";
+import { newElementWith } from "../element/mutateElement";
 
 const changeProperty = (
   elements: readonly ExcalidrawElement[],
@@ -164,23 +136,11 @@ export const actionChangeFillStyle = register({
   PanelComponent: ({ elements, appState, updateData }) => (
     <fieldset>
       <legend>{t("labels.fill")}</legend>
-      <ButtonIconSelect
+      <ButtonSelect
         options={[
-          {
-            value: "hachure",
-            text: t("labels.hachure"),
-            icon: <FillHachureIcon appearance={appState.appearance} />,
-          },
-          {
-            value: "cross-hatch",
-            text: t("labels.crossHatch"),
-            icon: <FillCrossHatchIcon appearance={appState.appearance} />,
-          },
-          {
-            value: "solid",
-            text: t("labels.solid"),
-            icon: <FillSolidIcon appearance={appState.appearance} />,
-          },
+          { value: "hachure", text: t("labels.hachure") },
+          { value: "cross-hatch", text: t("labels.crossHatch") },
+          { value: "solid", text: t("labels.solid") },
         ]}
         group="fill"
         value={getFormValue(
@@ -213,39 +173,12 @@ export const actionChangeStrokeWidth = register({
   PanelComponent: ({ elements, appState, updateData }) => (
     <fieldset>
       <legend>{t("labels.strokeWidth")}</legend>
-      <ButtonIconSelect
+      <ButtonSelect
         group="stroke-width"
         options={[
-          {
-            value: 1,
-            text: t("labels.thin"),
-            icon: (
-              <StrokeWidthIcon
-                appearance={appState.appearance}
-                strokeWidth={2}
-              />
-            ),
-          },
-          {
-            value: 2,
-            text: t("labels.bold"),
-            icon: (
-              <StrokeWidthIcon
-                appearance={appState.appearance}
-                strokeWidth={6}
-              />
-            ),
-          },
-          {
-            value: 4,
-            text: t("labels.extraBold"),
-            icon: (
-              <StrokeWidthIcon
-                appearance={appState.appearance}
-                strokeWidth={10}
-              />
-            ),
-          },
+          { value: 1, text: t("labels.thin") },
+          { value: 2, text: t("labels.bold") },
+          { value: 4, text: t("labels.extraBold") },
         ]}
         value={getFormValue(
           elements,
@@ -265,7 +198,6 @@ export const actionChangeSloppiness = register({
     return {
       elements: changeProperty(elements, appState, (el) =>
         newElementWith(el, {
-          seed: randomInteger(),
           roughness: value,
         }),
       ),
@@ -276,24 +208,12 @@ export const actionChangeSloppiness = register({
   PanelComponent: ({ elements, appState, updateData }) => (
     <fieldset>
       <legend>{t("labels.sloppiness")}</legend>
-      <ButtonIconSelect
+      <ButtonSelect
         group="sloppiness"
         options={[
-          {
-            value: 0,
-            text: t("labels.architect"),
-            icon: <SloppinessArchitectIcon appearance={appState.appearance} />,
-          },
-          {
-            value: 1,
-            text: t("labels.artist"),
-            icon: <SloppinessArtistIcon appearance={appState.appearance} />,
-          },
-          {
-            value: 2,
-            text: t("labels.cartoonist"),
-            icon: <SloppinessCartoonistIcon appearance={appState.appearance} />,
-          },
+          { value: 0, text: t("labels.architect") },
+          { value: 1, text: t("labels.artist") },
+          { value: 2, text: t("labels.cartoonist") },
         ]}
         value={getFormValue(
           elements,
@@ -307,53 +227,6 @@ export const actionChangeSloppiness = register({
   ),
 });
 
-export const actionChangeStrokeStyle = register({
-  name: "changeStrokeStyle",
-  perform: (elements, appState, value) => {
-    return {
-      elements: changeProperty(elements, appState, (el) =>
-        newElementWith(el, {
-          strokeStyle: value,
-        }),
-      ),
-      appState: { ...appState, currentItemStrokeStyle: value },
-      commitToHistory: true,
-    };
-  },
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <fieldset>
-      <legend>{t("labels.strokeStyle")}</legend>
-      <ButtonIconSelect
-        group="strokeStyle"
-        options={[
-          {
-            value: "solid",
-            text: t("labels.strokeStyle_solid"),
-            icon: <StrokeStyleSolidIcon appearance={appState.appearance} />,
-          },
-          {
-            value: "dashed",
-            text: t("labels.strokeStyle_dashed"),
-            icon: <StrokeStyleDashedIcon appearance={appState.appearance} />,
-          },
-          {
-            value: "dotted",
-            text: t("labels.strokeStyle_dotted"),
-            icon: <StrokeStyleDottedIcon appearance={appState.appearance} />,
-          },
-        ]}
-        value={getFormValue(
-          elements,
-          appState,
-          (element) => element.strokeStyle,
-          appState.currentItemStrokeStyle,
-        )}
-        onChange={(value) => updateData(value)}
-      />
-    </fieldset>
-  ),
-});
-
 export const actionChangeOpacity = register({
   name: "changeOpacity",
   perform: (elements, appState, value) => {
@@ -410,7 +283,7 @@ export const actionChangeFontSize = register({
       elements: changeProperty(elements, appState, (el) => {
         if (isTextElement(el)) {
           const element: ExcalidrawTextElement = newElementWith(el, {
-            fontSize: value,
+            font: `${value}px ${el.font.split("px ")[1]}`,
           });
           redrawTextBoundingBox(element);
           return element;
@@ -420,7 +293,9 @@ export const actionChangeFontSize = register({
       }),
       appState: {
         ...appState,
-        currentItemFontSize: value,
+        currentItemFont: `${value}px ${
+          appState.currentItemFont.split("px ")[1]
+        }`,
       },
       commitToHistory: true,
     };
@@ -439,8 +314,8 @@ export const actionChangeFontSize = register({
         value={getFormValue(
           elements,
           appState,
-          (element) => isTextElement(element) && element.fontSize,
-          appState.currentItemFontSize || DEFAULT_FONT_SIZE,
+          (element) => isTextElement(element) && +element.font.split("px ")[0],
+          +(appState.currentItemFont || DEFAULT_FONT).split("px ")[0],
         )}
         onChange={(value) => updateData(value)}
       />
@@ -455,7 +330,7 @@ export const actionChangeFontFamily = register({
       elements: changeProperty(elements, appState, (el) => {
         if (isTextElement(el)) {
           const element: ExcalidrawTextElement = newElementWith(el, {
-            fontFamily: value,
+            font: `${el.font.split("px ")[0]}px ${value}`,
           });
           redrawTextBoundingBox(element);
           return element;
@@ -465,35 +340,33 @@ export const actionChangeFontFamily = register({
       }),
       appState: {
         ...appState,
-        currentItemFontFamily: value,
+        currentItemFont: `${
+          appState.currentItemFont.split("px ")[0]
+        }px ${value}`,
       },
       commitToHistory: true,
     };
   },
-  PanelComponent: ({ elements, appState, updateData }) => {
-    const options: { value: FontFamily; text: string }[] = [
-      { value: 1, text: t("labels.handDrawn") },
-      { value: 2, text: t("labels.normal") },
-      { value: 3, text: t("labels.code") },
-    ];
-
-    return (
-      <fieldset>
-        <legend>{t("labels.fontFamily")}</legend>
-        <ButtonSelect<FontFamily | false>
-          group="font-family"
-          options={options}
-          value={getFormValue(
-            elements,
-            appState,
-            (element) => isTextElement(element) && element.fontFamily,
-            appState.currentItemFontFamily || DEFAULT_FONT_FAMILY,
-          )}
-          onChange={(value) => updateData(value)}
-        />
-      </fieldset>
-    );
-  },
+  PanelComponent: ({ elements, appState, updateData }) => (
+    <fieldset>
+      <legend>{t("labels.fontFamily")}</legend>
+      <ButtonSelect
+        group="font-family"
+        options={[
+          { value: "Virgil", text: t("labels.handDrawn") },
+          { value: "Helvetica", text: t("labels.normal") },
+          { value: "Cascadia", text: t("labels.code") },
+        ]}
+        value={getFormValue(
+          elements,
+          appState,
+          (element) => isTextElement(element) && element.font.split("px ")[1],
+          (appState.currentItemFont || DEFAULT_FONT).split("px ")[1],
+        )}
+        onChange={(value) => updateData(value)}
+      />
+    </fieldset>
+  ),
 });
 
 export const actionChangeTextAlign = register({
@@ -539,225 +412,3 @@ export const actionChangeTextAlign = register({
     </fieldset>
   ),
 });
-
-export const actionChangeSharpness = register({
-  name: "changeSharpness",
-  perform: (elements, appState, value) => {
-    const targetElements = getTargetElements(
-      getNonDeletedElements(elements),
-      appState,
-    );
-    const shouldUpdateForNonLinearElements = targetElements.length
-      ? targetElements.every((el) => !isLinearElement(el))
-      : !isLinearElementType(appState.elementType);
-    const shouldUpdateForLinearElements = targetElements.length
-      ? targetElements.every(isLinearElement)
-      : isLinearElementType(appState.elementType);
-    return {
-      elements: changeProperty(elements, appState, (el) =>
-        newElementWith(el, {
-          strokeSharpness: value,
-        }),
-      ),
-      appState: {
-        ...appState,
-        currentItemStrokeSharpness: shouldUpdateForNonLinearElements
-          ? value
-          : appState.currentItemStrokeSharpness,
-        currentItemLinearStrokeSharpness: shouldUpdateForLinearElements
-          ? value
-          : appState.currentItemLinearStrokeSharpness,
-      },
-      commitToHistory: true,
-    };
-  },
-  PanelComponent: ({ elements, appState, updateData }) => (
-    <fieldset>
-      <legend>{t("labels.edges")}</legend>
-      <ButtonIconSelect
-        group="edges"
-        options={[
-          {
-            value: "sharp",
-            text: t("labels.sharp"),
-            icon: <EdgeSharpIcon appearance={appState.appearance} />,
-          },
-          {
-            value: "round",
-            text: t("labels.round"),
-            icon: <EdgeRoundIcon appearance={appState.appearance} />,
-          },
-        ]}
-        value={getFormValue(
-          elements,
-          appState,
-          (element) => element.strokeSharpness,
-          (canChangeSharpness(appState.elementType) &&
-            (isLinearElementType(appState.elementType)
-              ? appState.currentItemLinearStrokeSharpness
-              : appState.currentItemStrokeSharpness)) ||
-            null,
-        )}
-        onChange={(value) => updateData(value)}
-      />
-    </fieldset>
-  ),
-});
-
-export const actionChangeArrowhead = register({
-  name: "changeArrowhead",
-  perform: (
-    elements,
-    appState,
-    value: { position: "start" | "end"; type: Arrowhead },
-  ) => {
-    return {
-      elements: changeProperty(elements, appState, (el) => {
-        if (isLinearElement(el)) {
-          const { position, type } = value;
-
-          if (position === "start") {
-            const element: ExcalidrawLinearElement = newElementWith(el, {
-              startArrowhead: type,
-            });
-            return element;
-          } else if (position === "end") {
-            const element: ExcalidrawLinearElement = newElementWith(el, {
-              endArrowhead: type,
-            });
-            return element;
-          }
-        }
-
-        return el;
-      }),
-      appState: {
-        ...appState,
-        [value.position === "start"
-          ? "currentItemStartArrowhead"
-          : "currentItemEndArrowhead"]: value.type,
-      },
-      commitToHistory: true,
-    };
-  },
-  PanelComponent: ({ elements, appState, updateData }) => {
-    const isRTL = getLanguage().rtl;
-
-    return (
-      <fieldset>
-        <legend>{t("labels.arrowheads")}</legend>
-        <div className="iconSelectList">
-          <IconPicker
-            label="arrowhead_start"
-            options={[
-              {
-                value: null,
-                text: t("labels.arrowhead_none"),
-                icon: <ArrowheadNoneIcon appearance={appState.appearance} />,
-                keyBinding: "q",
-              },
-              {
-                value: "arrow",
-                text: t("labels.arrowhead_arrow"),
-                icon: (
-                  <ArrowheadArrowIcon
-                    appearance={appState.appearance}
-                    flip={!isRTL}
-                  />
-                ),
-                keyBinding: "w",
-              },
-              {
-                value: "bar",
-                text: t("labels.arrowhead_bar"),
-                icon: (
-                  <ArrowheadBarIcon
-                    appearance={appState.appearance}
-                    flip={!isRTL}
-                  />
-                ),
-                keyBinding: "e",
-              },
-              {
-                value: "dot",
-                text: t("labels.arrowhead_dot"),
-                icon: (
-                  <ArrowheadDotIcon
-                    appearance={appState.appearance}
-                    flip={!isRTL}
-                  />
-                ),
-                keyBinding: "r",
-              },
-            ]}
-            value={getFormValue<Arrowhead | null>(
-              elements,
-              appState,
-              (element) =>
-                isLinearElement(element) && canHaveArrowheads(element.type)
-                  ? element.startArrowhead
-                  : appState.currentItemStartArrowhead,
-              appState.currentItemStartArrowhead,
-            )}
-            onChange={(value) => updateData({ position: "start", type: value })}
-          />
-          <IconPicker
-            label="arrowhead_end"
-            group="arrowheads"
-            options={[
-              {
-                value: null,
-                text: t("labels.arrowhead_none"),
-                keyBinding: "q",
-                icon: <ArrowheadNoneIcon appearance={appState.appearance} />,
-              },
-              {
-                value: "arrow",
-                text: t("labels.arrowhead_arrow"),
-                keyBinding: "w",
-                icon: (
-                  <ArrowheadArrowIcon
-                    appearance={appState.appearance}
-                    flip={isRTL}
-                  />
-                ),
-              },
-              {
-                value: "bar",
-                text: t("labels.arrowhead_bar"),
-                keyBinding: "e",
-                icon: (
-                  <ArrowheadBarIcon
-                    appearance={appState.appearance}
-                    flip={isRTL}
-                  />
-                ),
-              },
-              {
-                value: "dot",
-                text: t("labels.arrowhead_dot"),
-                keyBinding: "r",
-                icon: (
-                  <ArrowheadDotIcon
-                    appearance={appState.appearance}
-                    flip={isRTL}
-                  />
-                ),
-              },
-            ]}
-            value={getFormValue<Arrowhead | null>(
-              elements,
-              appState,
-              (element) =>
-                isLinearElement(element) && canHaveArrowheads(element.type)
-                  ? element.endArrowhead
-                  : appState.currentItemEndArrowhead,
-              appState.currentItemEndArrowhead,
-            )}
-            onChange={(value) => updateData({ position: "end", type: value })}
-          />
-        </div>
-      </fieldset>
-    );
-  },
-});

src/actions/actionSelectAll.ts

@@ -1,31 +1,22 @@
 import { KEYS } from "../keys";
 import { register } from "./register";
-import { selectGroupsForSelectedElements } from "../groups";
-import { getNonDeletedElements } from "../element";
 
 export const actionSelectAll = register({
   name: "selectAll",
   perform: (elements, appState) => {
-    if (appState.editingLinearElement) {
-      return false;
-    }
     return {
-      appState: selectGroupsForSelectedElements(
-        {
-          ...appState,
-          editingGroupId: null,
-          selectedElementIds: elements.reduce((map, element) => {
-            if (!element.isDeleted) {
-              map[element.id] = true;
-            }
-            return map;
-          }, {} as any),
-        },
-        getNonDeletedElements(elements),
-      ),
+      appState: {
+        ...appState,
+        selectedElementIds: elements.reduce((map, element) => {
+          if (!element.isDeleted) {
+            map[element.id] = true;
+          }
+          return map;
+        }, {} as any),
+      },
       commitToHistory: true,
     };
   },
   contextItemLabel: "labels.selectAll",
-  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.key === KEYS.A,
+  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.key === "a",
 });

src/actions/actionStyles.ts

@@ -3,18 +3,12 @@ import {
   isExcalidrawElement,
   redrawTextBoundingBox,
 } from "../element";
-import { CODES, KEYS } from "../keys";
-import { t } from "../i18n";
+import { KEYS } from "../keys";
+import { DEFAULT_FONT, DEFAULT_TEXT_ALIGN } from "../appState";
 import { register } from "./register";
 import { mutateElement, newElementWith } from "../element/mutateElement";
-import {
-  DEFAULT_FONT_SIZE,
-  DEFAULT_FONT_FAMILY,
-  DEFAULT_TEXT_ALIGN,
-} from "../constants";
 
-// `copiedStyles` is exported only for tests.
-export let copiedStyles: string = "{}";
+let copiedStyles: string = "{}";
 
 export const actionCopyStyles = register({
   name: "copyStyles",
@@ -24,16 +18,13 @@ export const actionCopyStyles = register({
       copiedStyles = JSON.stringify(element);
     }
     return {
-      appState: {
-        ...appState,
-        toastMessage: t("toast.copyStyles"),
-      },
       commitToHistory: false,
     };
   },
   contextItemLabel: "labels.copyStyles",
   keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] && event.altKey && event.code === CODES.C,
+    event[KEYS.CTRL_OR_CMD] && event.shiftKey && event.key === "C",
+  contextMenuOrder: 0,
 });
 
 export const actionPasteStyles = register({
@@ -50,15 +41,13 @@ export const actionPasteStyles = register({
             backgroundColor: pastedElement?.backgroundColor,
             strokeWidth: pastedElement?.strokeWidth,
             strokeColor: pastedElement?.strokeColor,
-            strokeStyle: pastedElement?.strokeStyle,
             fillStyle: pastedElement?.fillStyle,
             opacity: pastedElement?.opacity,
             roughness: pastedElement?.roughness,
           });
           if (isTextElement(newElement)) {
             mutateElement(newElement, {
-              fontSize: pastedElement?.fontSize || DEFAULT_FONT_SIZE,
-              fontFamily: pastedElement?.fontFamily || DEFAULT_FONT_FAMILY,
+              font: pastedElement?.font || DEFAULT_FONT,
               textAlign: pastedElement?.textAlign || DEFAULT_TEXT_ALIGN,
             });
             redrawTextBoundingBox(newElement);
@@ -72,5 +61,6 @@ export const actionPasteStyles = register({
   },
   contextItemLabel: "labels.pasteStyles",
   keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] && event.altKey && event.code === CODES.V,
+    event[KEYS.CTRL_OR_CMD] && event.shiftKey && event.key === "V",
+  contextMenuOrder: 1,
 });

src/actions/actionToggleGridMode.tsx

@@ -1,22 +0,0 @@
-import { CODES, KEYS } from "../keys";
-import { register } from "./register";
-import { GRID_SIZE } from "../constants";
-import { AppState } from "../types";
-import { trackEvent } from "../analytics";
-
-export const actionToggleGridMode = register({
-  name: "gridMode",
-  perform(elements, appState) {
-    trackEvent("view", "mode", "grid");
-    return {
-      appState: {
-        ...appState,
-        gridSize: this.checked!(appState) ? null : GRID_SIZE,
-      },
-      commitToHistory: false,
-    };
-  },
-  checked: (appState: AppState) => appState.gridSize !== null,
-  contextItemLabel: "labels.gridMode",
-  keyTest: (event) => event[KEYS.CTRL_OR_CMD] && event.code === CODES.QUOTE,
-});

src/actions/actionToggleStats.tsx

@@ -1,16 +0,0 @@
-import { register } from "./register";
-
-export const actionToggleStats = register({
-  name: "stats",
-  perform(elements, appState) {
-    return {
-      appState: {
-        ...appState,
-        showStats: !this.checked!(appState),
-      },
-      commitToHistory: false,
-    };
-  },
-  checked: (appState) => appState.showStats,
-  contextItemLabel: "stats.title",
-});

src/actions/actionToggleViewMode.tsx

@@ -1,22 +0,0 @@
-import { CODES, KEYS } from "../keys";
-import { register } from "./register";
-import { trackEvent } from "../analytics";
-
-export const actionToggleViewMode = register({
-  name: "viewMode",
-  perform(elements, appState) {
-    trackEvent("view", "mode", "view");
-    return {
-      appState: {
-        ...appState,
-        viewModeEnabled: !this.checked!(appState),
-        selectedElementIds: {},
-      },
-      commitToHistory: false,
-    };
-  },
-  checked: (appState) => appState.viewModeEnabled,
-  contextItemLabel: "labels.viewMode",
-  keyTest: (event) =>
-    !event[KEYS.CTRL_OR_CMD] && event.altKey && event.code === CODES.R,
-});

src/actions/actionToggleZenMode.tsx

@@ -1,22 +0,0 @@
-import { CODES, KEYS } from "../keys";
-import { register } from "./register";
-import { trackEvent } from "../analytics";
-
-export const actionToggleZenMode = register({
-  name: "zenMode",
-  perform(elements, appState) {
-    trackEvent("view", "mode", "zen");
-
-    return {
-      appState: {
-        ...appState,
-        zenModeEnabled: !this.checked!(appState),
-      },
-      commitToHistory: false,
-    };
-  },
-  checked: (appState) => appState.zenModeEnabled,
-  contextItemLabel: "buttons.zenMode",
-  keyTest: (event) =>
-    !event[KEYS.CTRL_OR_CMD] && event.altKey && event.code === CODES.Z,
-});

src/actions/actionZindex.tsx

@@ -5,22 +5,79 @@ import {
   moveAllLeft,
   moveAllRight,
 } from "../zindex";
-import { KEYS, isDarwin, CODES } from "../keys";
+import { KEYS, isDarwin } from "../keys";
 import { t } from "../i18n";
 import { getShortcutKey } from "../utils";
 import { register } from "./register";
 import {
-  SendBackwardIcon,
-  BringToFrontIcon,
-  SendToBackIcon,
-  BringForwardIcon,
+  sendBackward,
+  bringToFront,
+  sendToBack,
+  bringForward,
 } from "../components/icons";
+import { ExcalidrawElement } from "../element/types";
+import { AppState } from "../types";
+
+function getElementIndices(
+  direction: "left" | "right",
+  elements: readonly ExcalidrawElement[],
+  appState: AppState,
+) {
+  const selectedIndices: number[] = [];
+  let deletedIndicesCache: number[] = [];
+
+  function cb(element: ExcalidrawElement, index: number) {
+    if (element.isDeleted) {
+      // we want to build an array of deleted elements that are preceeding
+      //  a selected element so that we move them together
+      deletedIndicesCache.push(index);
+    } else {
+      if (appState.selectedElementIds[element.id]) {
+        selectedIndices.push(...deletedIndicesCache, index);
+      }
+      // always empty cache of deleted elements after either pushing a group
+      //  of selected/deleted elements, of after encountering non-deleted elem
+      deletedIndicesCache = [];
+    }
+  }
+
+  // sending back → select contiguous deleted elements that are to the left of
+  //  selected element(s)
+  if (direction === "left") {
+    let i = -1;
+    const len = elements.length;
+    while (++i < len) {
+      cb(elements[i], i);
+    }
+    // moving to front → loop from right to left so that we don't need to
+    //  backtrack when gathering deleted elements
+  } else {
+    let i = elements.length;
+    while (--i > -1) {
+      cb(elements[i], i);
+    }
+  }
+  // sort in case we were gathering indexes from right to left
+  return selectedIndices.sort();
+}
+
+function moveElements(
+  func: typeof moveOneLeft,
+  elements: readonly ExcalidrawElement[],
+  appState: AppState,
+) {
+  const _elements = elements.slice();
+  const direction =
+    func === moveOneLeft || func === moveAllLeft ? "left" : "right";
+  const indices = getElementIndices(direction, _elements, appState);
+  return func(_elements, indices);
+}
 
 export const actionSendBackward = register({
   name: "sendBackward",
   perform: (elements, appState) => {
     return {
-      elements: moveOneLeft(elements, appState),
+      elements: moveElements(moveOneLeft, elements, appState),
       appState,
       commitToHistory: true,
     };
@@ -28,17 +85,15 @@ export const actionSendBackward = register({
   contextItemLabel: "labels.sendBackward",
   keyPriority: 40,
   keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] &&
-    !event.shiftKey &&
-    event.code === CODES.BRACKET_LEFT,
-  PanelComponent: ({ updateData, appState }) => (
+    event[KEYS.CTRL_OR_CMD] && !event.shiftKey && event.code === "BracketLeft",
+  PanelComponent: ({ updateData }) => (
     <button
       type="button"
       className="zIndexButton"
       onClick={() => updateData(null)}
       title={`${t("labels.sendBackward")} — ${getShortcutKey("CtrlOrCmd+[")}`}
     >
-      <SendBackwardIcon appearance={appState.appearance} />
+      {sendBackward}
     </button>
   ),
 });
@@ -47,7 +102,7 @@ export const actionBringForward = register({
   name: "bringForward",
   perform: (elements, appState) => {
     return {
-      elements: moveOneRight(elements, appState),
+      elements: moveElements(moveOneRight, elements, appState),
       appState,
       commitToHistory: true,
     };
@@ -55,17 +110,15 @@ export const actionBringForward = register({
   contextItemLabel: "labels.bringForward",
   keyPriority: 40,
   keyTest: (event) =>
-    event[KEYS.CTRL_OR_CMD] &&
-    !event.shiftKey &&
-    event.code === CODES.BRACKET_RIGHT,
-  PanelComponent: ({ updateData, appState }) => (
+    event[KEYS.CTRL_OR_CMD] && !event.shiftKey && event.code === "BracketRight",
+  PanelComponent: ({ updateData }) => (
     <button
       type="button"
       className="zIndexButton"
       onClick={() => updateData(null)}
       title={`${t("labels.bringForward")} — ${getShortcutKey("CtrlOrCmd+]")}`}
     >
-      <BringForwardIcon appearance={appState.appearance} />
+      {bringForward}
     </button>
   ),
 });
@@ -74,21 +127,20 @@ export const actionSendToBack = register({
   name: "sendToBack",
   perform: (elements, appState) => {
     return {
-      elements: moveAllLeft(elements, appState),
+      elements: moveElements(moveAllLeft, elements, appState),
       appState,
       commitToHistory: true,
     };
   },
   contextItemLabel: "labels.sendToBack",
-  keyTest: (event) =>
-    isDarwin
-      ? event[KEYS.CTRL_OR_CMD] &&
-        event.altKey &&
-        event.code === CODES.BRACKET_LEFT
+  keyTest: (event) => {
+    return isDarwin
+      ? event[KEYS.CTRL_OR_CMD] && event.altKey && event.code === "BracketLeft"
       : event[KEYS.CTRL_OR_CMD] &&
-        event.shiftKey &&
-        event.code === CODES.BRACKET_LEFT,
-  PanelComponent: ({ updateData, appState }) => (
+          event.shiftKey &&
+          event.code === "BracketLeft";
+  },
+  PanelComponent: ({ updateData }) => (
     <button
       type="button"
       className="zIndexButton"
@@ -99,7 +151,7 @@ export const actionSendToBack = register({
           : getShortcutKey("CtrlOrCmd+Shift+[")
       }`}
     >
-      <SendToBackIcon appearance={appState.appearance} />
+      {sendToBack}
     </button>
   ),
 });
@@ -108,21 +160,20 @@ export const actionBringToFront = register({
   name: "bringToFront",
   perform: (elements, appState) => {
     return {
-      elements: moveAllRight(elements, appState),
+      elements: moveElements(moveAllRight, elements, appState),
       appState,
       commitToHistory: true,
     };
   },
   contextItemLabel: "labels.bringToFront",
-  keyTest: (event) =>
-    isDarwin
-      ? event[KEYS.CTRL_OR_CMD] &&
-        event.altKey &&
-        event.code === CODES.BRACKET_RIGHT
+  keyTest: (event) => {
+    return isDarwin
+      ? event[KEYS.CTRL_OR_CMD] && event.altKey && event.code === "BracketRight"
       : event[KEYS.CTRL_OR_CMD] &&
-        event.shiftKey &&
-        event.code === CODES.BRACKET_RIGHT,
-  PanelComponent: ({ updateData, appState }) => (
+          event.shiftKey &&
+          event.code === "BracketRight";
+  },
+  PanelComponent: ({ updateData }) => (
     <button
       type="button"
       className="zIndexButton"
@@ -133,7 +184,7 @@ export const actionBringToFront = register({
           : getShortcutKey("CtrlOrCmd+Shift+]")
       }`}
     >
-      <BringToFrontIcon appearance={appState.appearance} />
+      {bringToFront}
     </button>
   ),
 });

src/actions/index.ts

@@ -34,7 +34,6 @@ export {
   actionChangeProjectName,
   actionChangeExportBackground,
   actionSaveScene,
-  actionSaveAsScene,
   actionLoadScene,
 } from "./actionExport";
 
@@ -45,35 +44,3 @@ export {
   actionFullScreen,
   actionShortcuts,
 } from "./actionMenu";
-
-export { actionGroup, actionUngroup } from "./actionGroup";
-
-export { actionGoToCollaborator } from "./actionNavigate";
-
-export { actionAddToLibrary } from "./actionAddToLibrary";
-
-export {
-  actionAlignTop,
-  actionAlignBottom,
-  actionAlignLeft,
-  actionAlignRight,
-  actionAlignVerticallyCentered,
-  actionAlignHorizontallyCentered,
-} from "./actionAlign";
-
-export {
-  distributeHorizontally,
-  distributeVertically,
-} from "./actionDistribute";
-
-export {
-  actionCopy,
-  actionCut,
-  actionCopyAsPng,
-  actionCopyAsSvg,
-} from "./actionClipboard";
-
-export { actionToggleGridMode } from "./actionToggleGridMode";
-export { actionToggleZenMode } from "./actionToggleZenMode";
-
-export { actionToggleStats } from "./actionToggleStats";

src/actions/manager.tsx

@@ -3,43 +3,33 @@ import {
   Action,
   ActionsManagerInterface,
   UpdaterFn,
+  ActionFilterFn,
   ActionName,
-  ActionResult,
 } from "./types";
 import { ExcalidrawElement } from "../element/types";
-import { AppState, ExcalidrawProps } from "../types";
-
-// This is the <App> component, but for now we don't care about anything but its
-// `canvas` state.
-type App = { canvas: HTMLCanvasElement | null; props: ExcalidrawProps };
+import { AppState } from "../types";
+import { t } from "../i18n";
+import { globalSceneState } from "../scene";
 
 export class ActionManager implements ActionsManagerInterface {
   actions = {} as ActionsManagerInterface["actions"];
 
-  updater: (actionResult: ActionResult | Promise<ActionResult>) => void;
+  updater: UpdaterFn;
+
+  getAppState: () => AppState;
 
-  getAppState: () => Readonly<AppState>;
   getElementsIncludingDeleted: () => readonly ExcalidrawElement[];
-  app: App;
 
   constructor(
     updater: UpdaterFn,
     getAppState: () => AppState,
-    getElementsIncludingDeleted: () => readonly ExcalidrawElement[],
-    app: App,
+    getElementsIncludingDeleted: () => ReturnType<
+      typeof globalSceneState["getElementsIncludingDeleted"]
+    >,
   ) {
-    this.updater = (actionResult) => {
-      if (actionResult && "then" in actionResult) {
-        actionResult.then((actionResult) => {
-          return updater(actionResult);
-        });
-      } else {
-        return updater(actionResult);
-      }
-    };
+    this.updater = updater;
     this.getAppState = getAppState;
     this.getElementsIncludingDeleted = getElementsIncludingDeleted;
-    this.app = app;
   }
 
   registerAction(action: Action) {
@@ -66,12 +56,6 @@ export class ActionManager implements ActionsManagerInterface {
     if (data.length === 0) {
       return false;
     }
-    const { viewModeEnabled } = this.getAppState();
-    if (viewModeEnabled) {
-      if (data[0].name !== "viewMode") {
-        return false;
-      }
-    }
 
     event.preventDefault();
     this.updater(
@@ -79,7 +63,6 @@ export class ActionManager implements ActionsManagerInterface {
         this.getElementsIncludingDeleted(),
         this.getAppState(),
         null,
-        this.app,
       ),
     );
     return true;
@@ -91,16 +74,34 @@ export class ActionManager implements ActionsManagerInterface {
         this.getElementsIncludingDeleted(),
         this.getAppState(),
         null,
-        this.app,
       ),
     );
   }
 
-  // Id is an attribute that we can use to pass in data like keys.
-  // This is needed for dynamically generated action components
-  // like the user list. We can use this key to extract more
-  // data from app state. This is an alternative to generic prop hell!
-  renderAction = (name: ActionName, id?: string) => {
+  getContextMenuItems(actionFilter: ActionFilterFn = (action) => action) {
+    return Object.values(this.actions)
+      .filter(actionFilter)
+      .filter((action) => "contextItemLabel" in action)
+      .sort(
+        (a, b) =>
+          (a.contextMenuOrder !== undefined ? a.contextMenuOrder : 999) -
+          (b.contextMenuOrder !== undefined ? b.contextMenuOrder : 999),
+      )
+      .map((action) => ({
+        label: action.contextItemLabel ? t(action.contextItemLabel) : "",
+        action: () => {
+          this.updater(
+            action.perform(
+              this.getElementsIncludingDeleted(),
+              this.getAppState(),
+              null,
+            ),
+          );
+        },
+      }));
+  }
+
+  renderAction = (name: ActionName) => {
     if (this.actions[name] && "PanelComponent" in this.actions[name]) {
       const action = this.actions[name];
       const PanelComponent = action.PanelComponent!;
@@ -110,7 +111,6 @@ export class ActionManager implements ActionsManagerInterface {
             this.getElementsIncludingDeleted(),
             this.getAppState(),
             formState,
-            this.app,
           ),
         );
       };
@@ -120,7 +120,6 @@ export class ActionManager implements ActionsManagerInterface {
           elements={this.getElementsIncludingDeleted()}
           appState={this.getAppState()}
           updateData={updateData}
-          id={id}
         />
       );
     }

src/actions/register.ts

@@ -2,7 +2,7 @@ import { Action } from "./types";
 
 export let actions: readonly Action[] = [];
 
-export const register = (action: Action): Action => {
+export function register(action: Action): Action {
   actions = actions.concat(action);
   return action;
-};
+}

src/actions/shortcuts.ts

@@ -1,67 +0,0 @@
-import { t } from "../i18n";
-import { isDarwin } from "../keys";
-import { getShortcutKey } from "../utils";
-
-export type ShortcutName =
-  | "cut"
-  | "copy"
-  | "paste"
-  | "copyStyles"
-  | "pasteStyles"
-  | "selectAll"
-  | "deleteSelectedElements"
-  | "duplicateSelection"
-  | "sendBackward"
-  | "bringForward"
-  | "sendToBack"
-  | "bringToFront"
-  | "copyAsPng"
-  | "copyAsSvg"
-  | "group"
-  | "ungroup"
-  | "gridMode"
-  | "zenMode"
-  | "stats"
-  | "addToLibrary"
-  | "viewMode";
-
-const shortcutMap: Record<ShortcutName, string[]> = {
-  cut: [getShortcutKey("CtrlOrCmd+X")],
-  copy: [getShortcutKey("CtrlOrCmd+C")],
-  paste: [getShortcutKey("CtrlOrCmd+V")],
-  copyStyles: [getShortcutKey("CtrlOrCmd+Alt+C")],
-  pasteStyles: [getShortcutKey("CtrlOrCmd+Alt+V")],
-  selectAll: [getShortcutKey("CtrlOrCmd+A")],
-  deleteSelectedElements: [getShortcutKey("Del")],
-  duplicateSelection: [
-    getShortcutKey("CtrlOrCmd+D"),
-    getShortcutKey(`Alt+${t("helpDialog.drag")}`),
-  ],
-  sendBackward: [getShortcutKey("CtrlOrCmd+[")],
-  bringForward: [getShortcutKey("CtrlOrCmd+]")],
-  sendToBack: [
-    isDarwin
-      ? getShortcutKey("CtrlOrCmd+Alt+[")
-      : getShortcutKey("CtrlOrCmd+Shift+["),
-  ],
-  bringToFront: [
-    isDarwin
-      ? getShortcutKey("CtrlOrCmd+Alt+]")
-      : getShortcutKey("CtrlOrCmd+Shift+]"),
-  ],
-  copyAsPng: [getShortcutKey("Shift+Alt+C")],
-  copyAsSvg: [],
-  group: [getShortcutKey("CtrlOrCmd+G")],
-  ungroup: [getShortcutKey("CtrlOrCmd+Shift+G")],
-  gridMode: [getShortcutKey("CtrlOrCmd+'")],
-  zenMode: [getShortcutKey("Alt+Z")],
-  stats: [],
-  addToLibrary: [],
-  viewMode: [getShortcutKey("Alt+R")],
-};
-
-export const getShortcutFromShortcutName = (name: ShortcutName) => {
-  const shortcuts = shortcutMap[name];
-  // if multiple shortcuts availiable, take the first one
-  return shortcuts && shortcuts.length > 0 ? shortcuts[0] : "";
-};

src/actions/types.ts

@@ -2,32 +2,22 @@ import React from "react";
 import { ExcalidrawElement } from "../element/types";
 import { AppState } from "../types";
 
-/** if false, the action should be prevented */
-export type ActionResult =
-  | {
-      elements?: readonly ExcalidrawElement[] | null;
-      appState?: MarkOptional<AppState, "offsetTop" | "offsetLeft"> | null;
-      commitToHistory: boolean;
-      syncHistory?: boolean;
-    }
-  | false;
+export type ActionResult = {
+  elements?: readonly ExcalidrawElement[] | null;
+  appState?: AppState | null;
+  commitToHistory: boolean;
+};
 
 type ActionFn = (
   elements: readonly ExcalidrawElement[],
-  appState: Readonly<AppState>,
+  appState: AppState,
   formData: any,
-  app: { canvas: HTMLCanvasElement | null },
-) => ActionResult | Promise<ActionResult>;
+) => ActionResult;
 
-export type UpdaterFn = (res: ActionResult) => void;
+export type UpdaterFn = (res: ActionResult, commitToHistory?: boolean) => void;
 export type ActionFilterFn = (action: Action) => void;
 
 export type ActionName =
-  | "copy"
-  | "cut"
-  | "paste"
-  | "copyAsPng"
-  | "copyAsSvg"
   | "sendBackward"
   | "bringForward"
   | "sendToBack"
@@ -35,16 +25,11 @@ export type ActionName =
   | "copyStyles"
   | "selectAll"
   | "pasteStyles"
-  | "gridMode"
-  | "zenMode"
-  | "stats"
   | "changeStrokeColor"
   | "changeBackgroundColor"
   | "changeFillStyle"
   | "changeStrokeWidth"
   | "changeSloppiness"
-  | "changeStrokeStyle"
-  | "changeArrowhead"
   | "changeOpacity"
   | "changeFontSize"
   | "toggleCanvasMenu"
@@ -54,10 +39,8 @@ export type ActionName =
   | "finalize"
   | "changeProjectName"
   | "changeExportBackground"
-  | "changeExportEmbedScene"
   | "changeShouldAddWatermark"
   | "saveScene"
-  | "saveAsScene"
   | "loadScene"
   | "duplicateSelection"
   | "deleteSelectedElements"
@@ -67,25 +50,10 @@ export type ActionName =
   | "zoomOut"
   | "resetZoom"
   | "zoomToFit"
-  | "zoomToSelection"
   | "changeFontFamily"
   | "changeTextAlign"
   | "toggleFullScreen"
-  | "toggleShortcuts"
-  | "group"
-  | "ungroup"
-  | "goToCollaborator"
-  | "addToLibrary"
-  | "changeSharpness"
-  | "alignTop"
-  | "alignBottom"
-  | "alignLeft"
-  | "alignRight"
-  | "alignVerticallyCentered"
-  | "alignHorizontallyCentered"
-  | "distributeHorizontally"
-  | "distributeVertically"
-  | "viewMode";
+  | "toggleShortcuts";
 
 export interface Action {
   name: ActionName;
@@ -93,7 +61,6 @@ export interface Action {
     elements: readonly ExcalidrawElement[];
     appState: AppState;
     updateData: (formData?: any) => void;
-    id?: string;
   }>;
   perform: ActionFn;
   keyPriority?: number;
@@ -103,16 +70,17 @@ export interface Action {
     elements: readonly ExcalidrawElement[],
   ) => boolean;
   contextItemLabel?: string;
-  contextItemPredicate?: (
-    elements: readonly ExcalidrawElement[],
-    appState: AppState,
-  ) => boolean;
-  checked?: (appState: Readonly<AppState>) => boolean;
+  contextMenuOrder?: number;
 }
 
 export interface ActionsManagerInterface {
-  actions: Record<ActionName, Action>;
+  actions: {
+    [actionName in ActionName]: Action;
+  };
   registerAction: (action: Action) => void;
   handleKeyDown: (event: KeyboardEvent) => boolean;
+  getContextMenuItems: (
+    actionFilter: ActionFilterFn,
+  ) => { label: string; action: () => void }[];
   renderAction: (name: ActionName) => React.ReactElement | null;
 }

src/align.ts

@@ -1,95 +0,0 @@
-import { ExcalidrawElement } from "./element/types";
-import { newElementWith } from "./element/mutateElement";
-import { getCommonBounds } from "./element";
-
-interface Box {
-  minX: number;
-  minY: number;
-  maxX: number;
-  maxY: number;
-}
-
-export interface Alignment {
-  position: "start" | "center" | "end";
-  axis: "x" | "y";
-}
-
-export const alignElements = (
-  selectedElements: ExcalidrawElement[],
-  alignment: Alignment,
-): ExcalidrawElement[] => {
-  const groups: ExcalidrawElement[][] = getMaximumGroups(selectedElements);
-
-  const selectionBoundingBox = getCommonBoundingBox(selectedElements);
-
-  return groups.flatMap((group) => {
-    const translation = calculateTranslation(
-      group,
-      selectionBoundingBox,
-      alignment,
-    );
-    return group.map((element) =>
-      newElementWith(element, {
-        x: element.x + translation.x,
-        y: element.y + translation.y,
-      }),
-    );
-  });
-};
-
-export const getMaximumGroups = (
-  elements: ExcalidrawElement[],
-): ExcalidrawElement[][] => {
-  const groups: Map<String, ExcalidrawElement[]> = new Map<
-    String,
-    ExcalidrawElement[]
-  >();
-
-  elements.forEach((element: ExcalidrawElement) => {
-    const groupId =
-      element.groupIds.length === 0
-        ? element.id
-        : element.groupIds[element.groupIds.length - 1];
-
-    const currentGroupMembers = groups.get(groupId) || [];
-
-    groups.set(groupId, [...currentGroupMembers, element]);
-  });
-
-  return Array.from(groups.values());
-};
-
-const calculateTranslation = (
-  group: ExcalidrawElement[],
-  selectionBoundingBox: Box,
-  { axis, position }: Alignment,
-): { x: number; y: number } => {
-  const groupBoundingBox = getCommonBoundingBox(group);
-
-  const [min, max]: ["minX" | "minY", "maxX" | "maxY"] =
-    axis === "x" ? ["minX", "maxX"] : ["minY", "maxY"];
-
-  const noTranslation = { x: 0, y: 0 };
-  if (position === "start") {
-    return {
-      ...noTranslation,
-      [axis]: selectionBoundingBox[min] - groupBoundingBox[min],
-    };
-  } else if (position === "end") {
-    return {
-      ...noTranslation,
-      [axis]: selectionBoundingBox[max] - groupBoundingBox[max],
-    };
-  } // else if (position === "center") {
-  return {
-    ...noTranslation,
-    [axis]:
-      (selectionBoundingBox[min] + selectionBoundingBox[max]) / 2 -
-      (groupBoundingBox[min] + groupBoundingBox[max]) / 2,
-  };
-};
-
-const getCommonBoundingBox = (elements: ExcalidrawElement[]): Box => {
-  const [minX, minY, maxX, maxY] = getCommonBounds(elements);
-  return { minX, minY, maxX, maxY };
-};

src/analytics.ts

@@ -1,18 +0,0 @@
-export const trackEvent =
-  typeof process !== "undefined" &&
-  process.env?.REACT_APP_GOOGLE_ANALYTICS_ID &&
-  typeof window !== "undefined" &&
-  window.gtag
-    ? (category: string, name: string, label?: string, value?: number) => {
-        window.gtag("event", name, {
-          event_category: category,
-          event_label: label,
-          value,
-        });
-      }
-    : typeof process !== "undefined" && process.env?.JEST_WORKER_ID
-    ? (category: string, name: string, label?: string, value?: number) => {}
-    : (category: string, name: string, label?: string, value?: number) => {
-        // Uncomment the next line to track locally
-        // console.info("Track Event", category, name, label, value);
-      };

src/appState.ts

@@ -1,184 +1,96 @@
 import oc from "open-color";
-import {
-  DEFAULT_FONT_FAMILY,
-  DEFAULT_FONT_SIZE,
-  DEFAULT_TEXT_ALIGN,
-} from "./constants";
-import { t } from "./i18n";
-import { AppState, NormalizedZoomValue } from "./types";
+import { AppState, FlooredNumber } from "./types";
 import { getDateTime } from "./utils";
+import { t } from "./i18n";
 
-export const getDefaultAppState = (): Omit<
-  AppState,
-  "offsetTop" | "offsetLeft"
-> => {
+export const DEFAULT_FONT = "20px Virgil";
+export const DEFAULT_TEXT_ALIGN = "left";
+
+export function getDefaultAppState(): AppState {
   return {
-    appearance: "light",
-    collaborators: new Map(),
-    currentChartType: "bar",
-    currentItemBackgroundColor: "transparent",
-    currentItemEndArrowhead: "arrow",
-    currentItemFillStyle: "hachure",
-    currentItemFontFamily: DEFAULT_FONT_FAMILY,
-    currentItemFontSize: DEFAULT_FONT_SIZE,
-    currentItemLinearStrokeSharpness: "round",
-    currentItemOpacity: 100,
-    currentItemRoughness: 1,
-    currentItemStartArrowhead: null,
-    currentItemStrokeColor: oc.black,
-    currentItemStrokeSharpness: "sharp",
-    currentItemStrokeStyle: "solid",
-    currentItemStrokeWidth: 1,
-    currentItemTextAlign: DEFAULT_TEXT_ALIGN,
-    cursorButton: "up",
-    draggingElement: null,
-    editingElement: null,
-    editingGroupId: null,
-    editingLinearElement: null,
-    elementLocked: false,
-    elementType: "selection",
-    errorMessage: null,
-    exportBackground: true,
-    exportEmbedScene: false,
-    fileHandle: null,
-    gridSize: null,
-    height: window.innerHeight,
-    isBindingEnabled: true,
-    isLibraryOpen: false,
     isLoading: false,
+    errorMessage: null,
+    draggingElement: null,
+    resizingElement: null,
+    multiElement: null,
+    editingElement: null,
+    elementType: "selection",
+    elementLocked: false,
+    exportBackground: true,
+    shouldAddWatermark: false,
+    currentItemStrokeColor: oc.black,
+    currentItemBackgroundColor: "transparent",
+    currentItemFillStyle: "hachure",
+    currentItemStrokeWidth: 1,
+    currentItemRoughness: 1,
+    currentItemOpacity: 100,
+    currentItemFont: DEFAULT_FONT,
+    currentItemTextAlign: DEFAULT_TEXT_ALIGN,
+    viewBackgroundColor: oc.white,
+    scrollX: 0 as FlooredNumber,
+    scrollY: 0 as FlooredNumber,
+    cursorX: 0,
+    cursorY: 0,
+    cursorButton: "up",
+    scrolledOutside: false,
+    name: `${t("labels.untitled")}-${getDateTime()}`,
+    username: "",
+    isCollaborating: false,
     isResizing: false,
     isRotating: false,
-    lastPointerDownWith: "mouse",
-    multiElement: null,
-    name: `${t("labels.untitled")}-${getDateTime()}`,
-    openMenu: null,
-    pasteDialog: { shown: false, data: null },
-    previousSelectedElementIds: {},
-    resizingElement: null,
-    scrolledOutside: false,
-    scrollX: 0,
-    scrollY: 0,
-    selectedElementIds: {},
-    selectedGroupIds: {},
     selectionElement: null,
-    shouldAddWatermark: false,
+    zoom: 1,
+    openMenu: null,
+    lastPointerDownWith: "mouse",
+    selectedElementIds: {},
+    collaborators: new Map(),
     shouldCacheIgnoreZoom: false,
-    showHelpDialog: false,
-    showStats: false,
-    startBoundElement: null,
-    suggestedBindings: [],
-    toastMessage: null,
-    viewBackgroundColor: oc.white,
-    width: window.innerWidth,
+    showShortcutsDialog: false,
     zenModeEnabled: false,
-    zoom: { value: 1 as NormalizedZoomValue, translation: { x: 0, y: 0 } },
-    viewModeEnabled: false,
   };
-};
+}
 
-/**
- * Config containing all AppState keys. Used to determine whether given state
- *  prop should be stripped when exporting to given storage type.
- */
-const APP_STATE_STORAGE_CONF = (<
-  Values extends {
-    /** whether to keep when storing to browser storage (localStorage/IDB) */
-    browser: boolean;
-    /** whether to keep when exporting to file/database */
-    export: boolean;
-  },
-  T extends Record<keyof AppState, Values>
->(
-  config: { [K in keyof T]: K extends keyof AppState ? T[K] : never },
-) => config)({
-  appearance: { browser: true, export: false },
-  collaborators: { browser: false, export: false },
-  currentChartType: { browser: true, export: false },
-  currentItemBackgroundColor: { browser: true, export: false },
-  currentItemEndArrowhead: { browser: true, export: false },
-  currentItemFillStyle: { browser: true, export: false },
-  currentItemFontFamily: { browser: true, export: false },
-  currentItemFontSize: { browser: true, export: false },
-  currentItemLinearStrokeSharpness: { browser: true, export: false },
-  currentItemOpacity: { browser: true, export: false },
-  currentItemRoughness: { browser: true, export: false },
-  currentItemStartArrowhead: { browser: true, export: false },
-  currentItemStrokeColor: { browser: true, export: false },
-  currentItemStrokeSharpness: { browser: true, export: false },
-  currentItemStrokeStyle: { browser: true, export: false },
-  currentItemStrokeWidth: { browser: true, export: false },
-  currentItemTextAlign: { browser: true, export: false },
-  cursorButton: { browser: true, export: false },
-  draggingElement: { browser: false, export: false },
-  editingElement: { browser: false, export: false },
-  editingGroupId: { browser: true, export: false },
-  editingLinearElement: { browser: false, export: false },
-  elementLocked: { browser: true, export: false },
-  elementType: { browser: true, export: false },
-  errorMessage: { browser: false, export: false },
-  exportBackground: { browser: true, export: false },
-  exportEmbedScene: { browser: true, export: false },
-  fileHandle: { browser: false, export: false },
-  gridSize: { browser: true, export: true },
-  height: { browser: false, export: false },
-  isBindingEnabled: { browser: false, export: false },
-  isLibraryOpen: { browser: false, export: false },
-  isLoading: { browser: false, export: false },
-  isResizing: { browser: false, export: false },
-  isRotating: { browser: false, export: false },
-  lastPointerDownWith: { browser: true, export: false },
-  multiElement: { browser: false, export: false },
-  name: { browser: true, export: false },
-  offsetLeft: { browser: false, export: false },
-  offsetTop: { browser: false, export: false },
-  openMenu: { browser: true, export: false },
-  pasteDialog: { browser: false, export: false },
-  previousSelectedElementIds: { browser: true, export: false },
-  resizingElement: { browser: false, export: false },
-  scrolledOutside: { browser: true, export: false },
-  scrollX: { browser: true, export: false },
-  scrollY: { browser: true, export: false },
-  selectedElementIds: { browser: true, export: false },
-  selectedGroupIds: { browser: true, export: false },
-  selectionElement: { browser: false, export: false },
-  shouldAddWatermark: { browser: true, export: false },
-  shouldCacheIgnoreZoom: { browser: true, export: false },
-  showHelpDialog: { browser: false, export: false },
-  showStats: { browser: true, export: false },
-  startBoundElement: { browser: false, export: false },
-  suggestedBindings: { browser: false, export: false },
-  toastMessage: { browser: false, export: false },
-  viewBackgroundColor: { browser: true, export: true },
-  width: { browser: false, export: false },
-  zenModeEnabled: { browser: true, export: false },
-  zoom: { browser: true, export: false },
-  viewModeEnabled: { browser: false, export: false },
-});
+export function clearAppStateForLocalStorage(appState: AppState) {
+  const {
+    draggingElement,
+    resizingElement,
+    multiElement,
+    editingElement,
+    selectionElement,
+    isResizing,
+    isRotating,
+    collaborators,
+    isCollaborating,
+    isLoading,
+    errorMessage,
+    showShortcutsDialog,
+    ...exportedState
+  } = appState;
+  return exportedState;
+}
 
-const _clearAppStateForStorage = <ExportType extends "export" | "browser">(
-  appState: Partial<AppState>,
-  exportType: ExportType,
-) => {
-  type ExportableKeys = {
-    [K in keyof typeof APP_STATE_STORAGE_CONF]: typeof APP_STATE_STORAGE_CONF[K][ExportType] extends true
-      ? K
-      : never;
-  }[keyof typeof APP_STATE_STORAGE_CONF];
-  const stateForExport = {} as { [K in ExportableKeys]?: typeof appState[K] };
-  for (const key of Object.keys(appState) as (keyof typeof appState)[]) {
-    const propConfig = APP_STATE_STORAGE_CONF[key];
-    if (propConfig?.[exportType]) {
-      // @ts-ignore see https://github.com/microsoft/TypeScript/issues/31445
-      stateForExport[key] = appState[key];
-    }
-  }
-  return stateForExport;
-};
+export function clearAppStatePropertiesForHistory(
+  appState: AppState,
+): Partial<AppState> {
+  return {
+    selectedElementIds: appState.selectedElementIds,
+    exportBackground: appState.exportBackground,
+    shouldAddWatermark: appState.shouldAddWatermark,
+    currentItemStrokeColor: appState.currentItemStrokeColor,
+    currentItemBackgroundColor: appState.currentItemBackgroundColor,
+    currentItemFillStyle: appState.currentItemFillStyle,
+    currentItemStrokeWidth: appState.currentItemStrokeWidth,
+    currentItemRoughness: appState.currentItemRoughness,
+    currentItemOpacity: appState.currentItemOpacity,
+    currentItemFont: appState.currentItemFont,
+    currentItemTextAlign: appState.currentItemTextAlign,
+    viewBackgroundColor: appState.viewBackgroundColor,
+    name: appState.name,
+  };
+}
 
-export const clearAppStateForLocalStorage = (appState: Partial<AppState>) => {
-  return _clearAppStateForStorage(appState, "browser");
-};
-
-export const cleanAppStateForExport = (appState: Partial<AppState>) => {
-  return _clearAppStateForStorage(appState, "export");
-};
+export function cleanAppStateForExport(appState: AppState) {
+  return {
+    viewBackgroundColor: appState.viewBackgroundColor,
+  };
+}

src/charts.ts

@@ -1,479 +0,0 @@
-import colors from "./colors";
-import { DEFAULT_FONT_FAMILY, DEFAULT_FONT_SIZE, ENV } from "./constants";
-import { newElement, newLinearElement, newTextElement } from "./element";
-import { NonDeletedExcalidrawElement } from "./element/types";
-import { randomId } from "./random";
-
-export type ChartElements = readonly NonDeletedExcalidrawElement[];
-
-const BAR_WIDTH = 32;
-const BAR_GAP = 12;
-const BAR_HEIGHT = 256;
-const GRID_OPACITY = 50;
-
-export interface Spreadsheet {
-  title: string | null;
-  labels: string[] | null;
-  values: number[];
-}
-
-export const NOT_SPREADSHEET = "NOT_SPREADSHEET";
-export const VALID_SPREADSHEET = "VALID_SPREADSHEET";
-
-type ParseSpreadsheetResult =
-  | { type: typeof NOT_SPREADSHEET; reason: string }
-  | { type: typeof VALID_SPREADSHEET; spreadsheet: Spreadsheet };
-
-const tryParseNumber = (s: string): number | null => {
-  const match = /^[$€£¥₩]?([0-9,]+(\.[0-9]+)?)$/.exec(s);
-  if (!match) {
-    return null;
-  }
-  return parseFloat(match[1].replace(/,/g, ""));
-};
-
-const isNumericColumn = (lines: string[][], columnIndex: number) =>
-  lines.slice(1).every((line) => tryParseNumber(line[columnIndex]) !== null);
-
-const tryParseCells = (cells: string[][]): ParseSpreadsheetResult => {
-  const numCols = cells[0].length;
-
-  if (numCols > 2) {
-    return { type: NOT_SPREADSHEET, reason: "More than 2 columns" };
-  }
-
-  if (numCols === 1) {
-    if (!isNumericColumn(cells, 0)) {
-      return { type: NOT_SPREADSHEET, reason: "Value is not numeric" };
-    }
-
-    const hasHeader = tryParseNumber(cells[0][0]) === null;
-    const values = (hasHeader ? cells.slice(1) : cells).map((line) =>
-      tryParseNumber(line[0]),
-    );
-
-    if (values.length < 2) {
-      return { type: NOT_SPREADSHEET, reason: "Less than two rows" };
-    }
-
-    return {
-      type: VALID_SPREADSHEET,
-      spreadsheet: {
-        title: hasHeader ? cells[0][0] : null,
-        labels: null,
-        values: values as number[],
-      },
-    };
-  }
-
-  const valueColumnIndex = isNumericColumn(cells, 0) ? 0 : 1;
-
-  if (!isNumericColumn(cells, valueColumnIndex)) {
-    return { type: NOT_SPREADSHEET, reason: "Value is not numeric" };
-  }
-
-  const labelColumnIndex = (valueColumnIndex + 1) % 2;
-  const hasHeader = tryParseNumber(cells[0][valueColumnIndex]) === null;
-  const rows = hasHeader ? cells.slice(1) : cells;
-
-  if (rows.length < 2) {
-    return { type: NOT_SPREADSHEET, reason: "Less than 2 rows" };
-  }
-
-  return {
-    type: VALID_SPREADSHEET,
-    spreadsheet: {
-      title: hasHeader ? cells[0][valueColumnIndex] : null,
-      labels: rows.map((row) => row[labelColumnIndex]),
-      values: rows.map((row) => tryParseNumber(row[valueColumnIndex])!),
-    },
-  };
-};
-
-const transposeCells = (cells: string[][]) => {
-  const nextCells: string[][] = [];
-  for (let col = 0; col < cells[0].length; col++) {
-    const nextCellRow: string[] = [];
-    for (let row = 0; row < cells.length; row++) {
-      nextCellRow.push(cells[row][col]);
-    }
-    nextCells.push(nextCellRow);
-  }
-  return nextCells;
-};
-
-export const tryParseSpreadsheet = (text: string): ParseSpreadsheetResult => {
-  // Copy/paste from excel, spreadhseets, tsv, csv.
-  // For now we only accept 2 columns with an optional header
-
-  // Check for tab separated values
-  let lines = text
-    .trim()
-    .split("\n")
-    .map((line) => line.trim().split("\t"));
-
-  // Check for comma separated files
-  if (lines.length && lines[0].length !== 2) {
-    lines = text
-      .trim()
-      .split("\n")
-      .map((line) => line.trim().split(","));
-  }
-
-  if (lines.length === 0) {
-    return { type: NOT_SPREADSHEET, reason: "No values" };
-  }
-
-  const numColsFirstLine = lines[0].length;
-  const isSpreadsheet = lines.every((line) => line.length === numColsFirstLine);
-
-  if (!isSpreadsheet) {
-    return {
-      type: NOT_SPREADSHEET,
-      reason: "All rows don't have same number of columns",
-    };
-  }
-
-  const result = tryParseCells(lines);
-  if (result.type !== VALID_SPREADSHEET) {
-    const transposedResults = tryParseCells(transposeCells(lines));
-    if (transposedResults.type === VALID_SPREADSHEET) {
-      return transposedResults;
-    }
-  }
-  return result;
-};
-
-const bgColors = colors.elementBackground.slice(
-  2,
-  colors.elementBackground.length,
-);
-
-// Put all the common properties here so when the whole chart is selected
-// the properties dialog shows the correct selected values
-const commonProps = {
-  fillStyle: "hachure",
-  fontFamily: DEFAULT_FONT_FAMILY,
-  fontSize: DEFAULT_FONT_SIZE,
-  opacity: 100,
-  roughness: 1,
-  strokeColor: colors.elementStroke[0],
-  strokeSharpness: "sharp",
-  strokeStyle: "solid",
-  strokeWidth: 1,
-  verticalAlign: "middle",
-} as const;
-
-const getChartDimentions = (spreadsheet: Spreadsheet) => {
-  const chartWidth =
-    (BAR_WIDTH + BAR_GAP) * spreadsheet.values.length + BAR_GAP;
-  const chartHeight = BAR_HEIGHT + BAR_GAP * 2;
-  return { chartWidth, chartHeight };
-};
-
-const chartXLabels = (
-  spreadsheet: Spreadsheet,
-  x: number,
-  y: number,
-  groupId: string,
-  backgroundColor: string,
-): ChartElements => {
-  return (
-    spreadsheet.labels?.map((label, index) => {
-      return newTextElement({
-        groupIds: [groupId],
-        backgroundColor,
-        ...commonProps,
-        text: label.length > 8 ? `${label.slice(0, 5)}...` : label,
-        x: x + index * (BAR_WIDTH + BAR_GAP) + BAR_GAP * 2,
-        y: y + BAR_GAP / 2,
-        width: BAR_WIDTH,
-        angle: 5.87,
-        fontSize: 16,
-        textAlign: "center",
-        verticalAlign: "top",
-      });
-    }) || []
-  );
-};
-
-const chartYLabels = (
-  spreadsheet: Spreadsheet,
-  x: number,
-  y: number,
-  groupId: string,
-  backgroundColor: string,
-): ChartElements => {
-  const minYLabel = newTextElement({
-    groupIds: [groupId],
-    backgroundColor,
-    ...commonProps,
-    x: x - BAR_GAP,
-    y: y - BAR_GAP,
-    text: "0",
-    textAlign: "right",
-  });
-
-  const maxYLabel = newTextElement({
-    groupIds: [groupId],
-    backgroundColor,
-    ...commonProps,
-    x: x - BAR_GAP,
-    y: y - BAR_HEIGHT - minYLabel.height / 2,
-    text: Math.max(...spreadsheet.values).toLocaleString(),
-    textAlign: "right",
-  });
-
-  return [minYLabel, maxYLabel];
-};
-
-const chartLines = (
-  spreadsheet: Spreadsheet,
-  x: number,
-  y: number,
-  groupId: string,
-  backgroundColor: string,
-): ChartElements => {
-  const { chartWidth, chartHeight } = getChartDimentions(spreadsheet);
-  const xLine = newLinearElement({
-    backgroundColor,
-    groupIds: [groupId],
-    ...commonProps,
-    type: "line",
-    x,
-    y,
-    startArrowhead: null,
-    endArrowhead: null,
-    width: chartWidth,
-    points: [
-      [0, 0],
-      [chartWidth, 0],
-    ],
-  });
-
-  const yLine = newLinearElement({
-    backgroundColor,
-    groupIds: [groupId],
-    ...commonProps,
-    type: "line",
-    x,
-    y,
-    startArrowhead: null,
-    endArrowhead: null,
-    height: chartHeight,
-    points: [
-      [0, 0],
-      [0, -chartHeight],
-    ],
-  });
-
-  const maxLine = newLinearElement({
-    backgroundColor,
-    groupIds: [groupId],
-    ...commonProps,
-    type: "line",
-    x,
-    y: y - BAR_HEIGHT - BAR_GAP,
-    startArrowhead: null,
-    endArrowhead: null,
-    strokeStyle: "dotted",
-    width: chartWidth,
-    opacity: GRID_OPACITY,
-    points: [
-      [0, 0],
-      [chartWidth, 0],
-    ],
-  });
-
-  return [xLine, yLine, maxLine];
-};
-
-// For the maths behind it https://excalidraw.com/#json=6320864370884608,O_5xfD-Agh32tytHpRJx1g
-const chartBaseElements = (
-  spreadsheet: Spreadsheet,
-  x: number,
-  y: number,
-  groupId: string,
-  backgroundColor: string,
-  debug?: boolean,
-): ChartElements => {
-  const { chartWidth, chartHeight } = getChartDimentions(spreadsheet);
-
-  const title = spreadsheet.title
-    ? newTextElement({
-        backgroundColor,
-        groupIds: [groupId],
-        ...commonProps,
-        text: spreadsheet.title,
-        x: x + chartWidth / 2,
-        y: y - BAR_HEIGHT - BAR_GAP * 2 - DEFAULT_FONT_SIZE,
-        strokeSharpness: "sharp",
-        strokeStyle: "solid",
-        textAlign: "center",
-      })
-    : null;
-
-  const debugRect = debug
-    ? newElement({
-        backgroundColor,
-        groupIds: [groupId],
-        ...commonProps,
-        type: "rectangle",
-        x,
-        y: y - chartHeight,
-        width: chartWidth,
-        height: chartHeight,
-        strokeColor: colors.elementStroke[0],
-        fillStyle: "solid",
-        opacity: 6,
-      })
-    : null;
-
-  return [
-    ...(debugRect ? [debugRect] : []),
-    ...(title ? [title] : []),
-    ...chartXLabels(spreadsheet, x, y, groupId, backgroundColor),
-    ...chartYLabels(spreadsheet, x, y, groupId, backgroundColor),
-    ...chartLines(spreadsheet, x, y, groupId, backgroundColor),
-  ];
-};
-
-const chartTypeBar = (
-  spreadsheet: Spreadsheet,
-  x: number,
-  y: number,
-): ChartElements => {
-  const max = Math.max(...spreadsheet.values);
-  const groupId = randomId();
-  const backgroundColor = bgColors[Math.floor(Math.random() * bgColors.length)];
-
-  const bars = spreadsheet.values.map((value, index) => {
-    const barHeight = (value / max) * BAR_HEIGHT;
-    return newElement({
-      backgroundColor,
-      groupIds: [groupId],
-      ...commonProps,
-      type: "rectangle",
-      x: x + index * (BAR_WIDTH + BAR_GAP) + BAR_GAP,
-      y: y - barHeight - BAR_GAP,
-      width: BAR_WIDTH,
-      height: barHeight,
-    });
-  });
-
-  return [
-    ...bars,
-    ...chartBaseElements(
-      spreadsheet,
-      x,
-      y,
-      groupId,
-      backgroundColor,
-      process.env.NODE_ENV === ENV.DEVELOPMENT,
-    ),
-  ];
-};
-
-const chartTypeLine = (
-  spreadsheet: Spreadsheet,
-  x: number,
-  y: number,
-): ChartElements => {
-  const max = Math.max(...spreadsheet.values);
-  const groupId = randomId();
-  const backgroundColor = bgColors[Math.floor(Math.random() * bgColors.length)];
-
-  let index = 0;
-  const points = [];
-  for (const value of spreadsheet.values) {
-    const cx = index * (BAR_WIDTH + BAR_GAP);
-    const cy = -(value / max) * BAR_HEIGHT;
-    points.push([cx, cy]);
-    index++;
-  }
-
-  const maxX = Math.max(...points.map((element) => element[0]));
-  const maxY = Math.max(...points.map((element) => element[1]));
-  const minX = Math.min(...points.map((element) => element[0]));
-  const minY = Math.min(...points.map((element) => element[1]));
-
-  const line = newLinearElement({
-    backgroundColor,
-    groupIds: [groupId],
-    ...commonProps,
-    type: "line",
-    x: x + BAR_GAP + BAR_WIDTH / 2,
-    y: y - BAR_GAP,
-    startArrowhead: null,
-    endArrowhead: null,
-    height: maxY - minY,
-    width: maxX - minX,
-    strokeWidth: 2,
-    points: points as any,
-  });
-
-  const dots = spreadsheet.values.map((value, index) => {
-    const cx = index * (BAR_WIDTH + BAR_GAP) + BAR_GAP / 2;
-    const cy = -(value / max) * BAR_HEIGHT + BAR_GAP / 2;
-    return newElement({
-      backgroundColor,
-      groupIds: [groupId],
-      ...commonProps,
-      fillStyle: "solid",
-      strokeWidth: 2,
-      type: "ellipse",
-      x: x + cx + BAR_WIDTH / 2,
-      y: y + cy - BAR_GAP * 2,
-      width: BAR_GAP,
-      height: BAR_GAP,
-    });
-  });
-
-  const lines = spreadsheet.values.map((value, index) => {
-    const cx = index * (BAR_WIDTH + BAR_GAP) + BAR_GAP / 2;
-    const cy = (value / max) * BAR_HEIGHT + BAR_GAP / 2 + BAR_GAP;
-    return newLinearElement({
-      backgroundColor,
-      groupIds: [groupId],
-      ...commonProps,
-      type: "line",
-      x: x + cx + BAR_WIDTH / 2 + BAR_GAP / 2,
-      y: y - cy,
-      startArrowhead: null,
-      endArrowhead: null,
-      height: cy,
-      strokeStyle: "dotted",
-      opacity: GRID_OPACITY,
-      points: [
-        [0, 0],
-        [0, cy],
-      ],
-    });
-  });
-
-  return [
-    ...chartBaseElements(
-      spreadsheet,
-      x,
-      y,
-      groupId,
-      backgroundColor,
-      process.env.NODE_ENV === ENV.DEVELOPMENT,
-    ),
-    line,
-    ...lines,
-    ...dots,
-  ];
-};
-
-export const renderSpreadsheet = (
-  chartType: string,
-  spreadsheet: Spreadsheet,
-  x: number,
-  y: number,
-): ChartElements => {
-  if (chartType === "line") {
-    return chartTypeLine(spreadsheet, x, y);
-  }
-  return chartTypeBar(spreadsheet, x, y);
-};

src/clients.ts

@@ -1,30 +0,0 @@
-import colors from "./colors";
-
-export const getClientColors = (clientId: string) => {
-  // Naive way of getting an integer out of the clientId
-  const sum = clientId.split("").reduce((a, str) => a + str.charCodeAt(0), 0);
-
-  // Skip transparent background.
-  const backgrounds = colors.elementBackground.slice(1);
-  const strokes = colors.elementStroke.slice(1);
-  return {
-    background: backgrounds[sum % backgrounds.length],
-    stroke: strokes[sum % strokes.length],
-  };
-};
-
-export const getClientInitials = (username?: string | null) => {
-  if (!username) {
-    return "?";
-  }
-  const names = username.trim().split(" ");
-
-  if (names.length < 2) {
-    return names[0].substring(0, 2).toUpperCase();
-  }
-
-  const firstName = names[0];
-  const lastName = names[names.length - 1];
-
-  return (firstName[0] + lastName[0]).toUpperCase();
-};

src/clipboard.ts

@@ -5,16 +5,6 @@ import {
 import { getSelectedElements } from "./scene";
 import { AppState } from "./types";
 import { SVG_EXPORT_TAG } from "./scene/export";
-import { tryParseSpreadsheet, Spreadsheet, VALID_SPREADSHEET } from "./charts";
-import { canvasToBlob } from "./data/blob";
-
-const TYPE_ELEMENTS = "excalidraw/elements";
-
-type ElementsClipboard = {
-  type: typeof TYPE_ELEMENTS;
-  created: number;
-  elements: ExcalidrawElement[];
-};
 
 let CLIPBOARD = "";
 let PREFER_APP_CLIPBOARD = false;
@@ -31,139 +21,104 @@ export const probablySupportsClipboardBlob =
   "ClipboardItem" in window &&
   "toBlob" in HTMLCanvasElement.prototype;
 
-const isElementsClipboard = (contents: any): contents is ElementsClipboard => {
-  if (contents?.type === TYPE_ELEMENTS) {
-    return true;
-  }
-  return false;
-};
-
-export const copyToClipboard = async (
+export async function copyToAppClipboard(
   elements: readonly NonDeletedExcalidrawElement[],
   appState: AppState,
-) => {
-  const contents: ElementsClipboard = {
-    type: TYPE_ELEMENTS,
-    created: Date.now(),
-    elements: getSelectedElements(elements, appState),
-  };
-  const json = JSON.stringify(contents);
-  CLIPBOARD = json;
+) {
+  CLIPBOARD = JSON.stringify(getSelectedElements(elements, appState));
   try {
+    // when copying to in-app clipboard, clear system clipboard so that if
+    //  system clip contains text on paste we know it was copied *after* user
+    //  copied elements, and thus we should prefer the text content.
+    await copyTextToSystemClipboard(null);
     PREFER_APP_CLIPBOARD = false;
-    await copyTextToSystemClipboard(json);
-  } catch (error) {
+  } catch {
+    // if clearing system clipboard didn't work, we should prefer in-app
+    //  clipboard even if there's text in system clipboard on paste, because
+    //  we can't be sure of the order of copy operations
     PREFER_APP_CLIPBOARD = true;
-    console.error(error);
   }
-};
+}
 
-const getAppClipboard = (): Partial<ElementsClipboard> => {
+export function getAppClipboard(): {
+  elements?: readonly ExcalidrawElement[];
+} {
   if (!CLIPBOARD) {
     return {};
   }
 
   try {
-    return JSON.parse(CLIPBOARD);
+    const clipboardElements = JSON.parse(CLIPBOARD);
+
+    if (
+      Array.isArray(clipboardElements) &&
+      clipboardElements.length > 0 &&
+      clipboardElements[0].type // need to implement a better check here...
+    ) {
+      return { elements: clipboardElements };
+    }
   } catch (error) {
     console.error(error);
-    return {};
   }
-};
 
-const parsePotentialSpreadsheet = (
-  text: string,
-): { spreadsheet: Spreadsheet } | { errorMessage: string } | null => {
-  const result = tryParseSpreadsheet(text);
-  if (result.type === VALID_SPREADSHEET) {
-    return { spreadsheet: result.spreadsheet };
-  }
-  return null;
-};
+  return {};
+}
 
-/**
- * Retrieves content from system clipboard (either from ClipboardEvent or
- *  via async clipboard API if supported)
- */
-const getSystemClipboard = async (
+export async function getClipboardContent(
   event: ClipboardEvent | null,
-): Promise<string> => {
+): Promise<{
+  text?: string;
+  elements?: readonly ExcalidrawElement[];
+}> {
   try {
     const text = event
       ? event.clipboardData?.getData("text/plain").trim()
       : probablySupportsClipboardReadText &&
         (await navigator.clipboard.readText());
 
-    return text || "";
-  } catch {
-    return "";
-  }
-};
-
-/**
- * Attemps to parse clipboard. Prefers system clipboard.
- */
-export const parseClipboard = async (
-  event: ClipboardEvent | null,
-): Promise<{
-  spreadsheet?: Spreadsheet;
-  elements?: readonly ExcalidrawElement[];
-  text?: string;
-  errorMessage?: string;
-}> => {
-  const systemClipboard = await getSystemClipboard(event);
-
-  // if system clipboard empty, couldn't be resolved, or contains previously
-  // copied excalidraw scene as SVG, fall back to previously copied excalidraw
-  // elements
-  if (!systemClipboard || systemClipboard.includes(SVG_EXPORT_TAG)) {
-    return getAppClipboard();
-  }
-
-  // if system clipboard contains spreadsheet, use it even though it's
-  // technically possible it's staler than in-app clipboard
-  const spreadsheetResult = parsePotentialSpreadsheet(systemClipboard);
-  if (spreadsheetResult) {
-    return spreadsheetResult;
-  }
-
-  const appClipboardData = getAppClipboard();
-
-  try {
-    const systemClipboardData = JSON.parse(systemClipboard);
-    // system clipboard elements are newer than in-app clipboard
-    if (
-      isElementsClipboard(systemClipboardData) &&
-      (!appClipboardData?.created ||
-        appClipboardData.created < systemClipboardData.created)
-    ) {
-      return { elements: systemClipboardData.elements };
+    if (text && !PREFER_APP_CLIPBOARD && !text.includes(SVG_EXPORT_TAG)) {
+      return { text };
     }
-    // in-app clipboard is newer than system clipboard
-    return appClipboardData;
-  } catch {
-    // system clipboard doesn't contain excalidraw elements → return plaintext
-    // unless we set a flag to prefer in-app clipboard because browser didn't
-    // support storing to system clipboard on copy
-    return PREFER_APP_CLIPBOARD && appClipboardData.elements
-      ? appClipboardData
-      : { text: systemClipboard };
+  } catch (error) {
+    console.error(error);
   }
-};
 
-export const copyCanvasToClipboardAsPng = async (canvas: HTMLCanvasElement) => {
-  const blob = await canvasToBlob(canvas);
-  await navigator.clipboard.write([
-    new window.ClipboardItem({ "image/png": blob }),
-  ]);
-};
+  return getAppClipboard();
+}
 
-export const copyTextToSystemClipboard = async (text: string | null) => {
+export async function copyCanvasToClipboardAsPng(canvas: HTMLCanvasElement) {
+  return new Promise((resolve, reject) => {
+    try {
+      canvas.toBlob(async function (blob: any) {
+        try {
+          await navigator.clipboard.write([
+            new window.ClipboardItem({ "image/png": blob }),
+          ]);
+          resolve();
+        } catch (error) {
+          reject(error);
+        }
+      });
+    } catch (error) {
+      reject(error);
+    }
+  });
+}
+
+export async function copyCanvasToClipboardAsSvg(svgroot: SVGSVGElement) {
+  try {
+    await navigator.clipboard.writeText(svgroot.outerHTML);
+  } catch (error) {
+    console.error(error);
+  }
+}
+
+export async function copyTextToSystemClipboard(text: string | null) {
   let copied = false;
   if (probablySupportsClipboardWriteText) {
     try {
       // NOTE: doesn't work on FF on non-HTTPS domains, or when document
-      // not focused
+      //  not focused
       await navigator.clipboard.writeText(text || "");
       copied = true;
     } catch (error) {
@@ -172,14 +127,14 @@ export const copyTextToSystemClipboard = async (text: string | null) => {
   }
 
   // Note that execCommand doesn't allow copying empty strings, so if we're
-  // clearing clipboard using this API, we must copy at least an empty char
+  //  clearing clipboard using this API, we must copy at least an empty char
   if (!copied && !copyTextViaExecCommand(text || " ")) {
     throw new Error("couldn't copy");
   }
-};
+}
 
 // adapted from https://github.com/zenorocha/clipboard.js/blob/ce79f170aa655c408b6aab33c9472e8e4fa52e19/src/clipboard-action.js#L48
-const copyTextViaExecCommand = (text: string) => {
+function copyTextViaExecCommand(text: string) {
   const isRTL = document.documentElement.getAttribute("dir") === "rtl";
 
   const textarea = document.createElement("textarea");
@@ -213,4 +168,4 @@ const copyTextViaExecCommand = (text: string) => {
   textarea.remove();
 
   return success;
-};
+}

src/colors.ts

@@ -1,18 +1,18 @@
 import oc from "open-color";
 
-const shades = (index: number) => [
-  oc.red[index],
-  oc.pink[index],
-  oc.grape[index],
-  oc.violet[index],
-  oc.indigo[index],
-  oc.blue[index],
-  oc.cyan[index],
-  oc.teal[index],
-  oc.green[index],
-  oc.lime[index],
-  oc.yellow[index],
-  oc.orange[index],
+const shades = (i: number) => [
+  oc.red[i],
+  oc.pink[i],
+  oc.grape[i],
+  oc.violet[i],
+  oc.indigo[i],
+  oc.blue[i],
+  oc.cyan[i],
+  oc.teal[i],
+  oc.green[i],
+  oc.lime[i],
+  oc.yellow[i],
+  oc.orange[i],
 ];
 
 export default {

src/components/Actions.tsx

@@ -1,24 +1,17 @@
 import React from "react";
-import { ActionManager } from "../actions/manager";
-import { getNonDeletedElements } from "../element";
+import { AppState } from "../types";
 import { ExcalidrawElement } from "../element/types";
+import { ActionManager } from "../actions/manager";
+import { hasBackground, hasStroke, hasText, getTargetElement } from "../scene";
 import { t } from "../i18n";
-import useIsMobile from "../is-mobile";
-import {
-  canChangeSharpness,
-  canHaveArrowheads,
-  getTargetElements,
-  hasBackground,
-  hasStroke,
-  hasText,
-} from "../scene";
 import { SHAPES } from "../shapes";
-import { AppState, Zoom } from "../types";
-import { capitalizeString, isTransparent, setCursorForShape } from "../utils";
-import Stack from "./Stack";
 import { ToolButton } from "./ToolButton";
+import { capitalizeString, setCursorForShape } from "../utils";
+import Stack from "./Stack";
+import useIsMobile from "../is-mobile";
+import { getNonDeletedElements } from "../element";
 
-export const SelectedShapeActions = ({
+export function SelectedShapeActions({
   appState,
   elements,
   renderAction,
@@ -28,45 +21,35 @@ export const SelectedShapeActions = ({
   elements: readonly ExcalidrawElement[];
   renderAction: ActionManager["renderAction"];
   elementType: ExcalidrawElement["type"];
-}) => {
-  const targetElements = getTargetElements(
+}) {
+  const targetElements = getTargetElement(
     getNonDeletedElements(elements),
     appState,
   );
   const isEditing = Boolean(appState.editingElement);
   const isMobile = useIsMobile();
-  const isRTL = document.documentElement.getAttribute("dir") === "rtl";
-
-  const showFillIcons =
-    hasBackground(elementType) ||
-    targetElements.some(
-      (element) =>
-        hasBackground(element.type) && !isTransparent(element.backgroundColor),
-    );
-  const showChangeBackgroundIcons =
-    hasBackground(elementType) ||
-    targetElements.some((element) => hasBackground(element.type));
 
   return (
     <div className="panelColumn">
       {renderAction("changeStrokeColor")}
-      {showChangeBackgroundIcons && renderAction("changeBackgroundColor")}
-      {showFillIcons && renderAction("changeFillStyle")}
+      {(hasBackground(elementType) ||
+        targetElements.some((element) => hasBackground(element.type))) && (
+        <>
+          {renderAction("changeBackgroundColor")}
+
+          {renderAction("changeFillStyle")}
+        </>
+      )}
 
       {(hasStroke(elementType) ||
         targetElements.some((element) => hasStroke(element.type))) && (
         <>
           {renderAction("changeStrokeWidth")}
-          {renderAction("changeStrokeStyle")}
+
           {renderAction("changeSloppiness")}
         </>
       )}
 
-      {(canChangeSharpness(elementType) ||
-        targetElements.some((element) => canChangeSharpness(element.type))) && (
-        <>{renderAction("changeSharpness")}</>
-      )}
-
       {(hasText(elementType) ||
         targetElements.some((element) => hasText(element.type))) && (
         <>
@@ -78,11 +61,6 @@ export const SelectedShapeActions = ({
         </>
       )}
 
-      {(canHaveArrowheads(elementType) ||
-        targetElements.some((element) => canHaveArrowheads(element.type))) && (
-        <>{renderAction("changeArrowhead")}</>
-      )}
-
       {renderAction("changeOpacity")}
 
       <fieldset>
@@ -94,134 +72,76 @@ export const SelectedShapeActions = ({
           {renderAction("bringForward")}
         </div>
       </fieldset>
-
-      {targetElements.length > 1 && (
-        <fieldset>
-          <legend>{t("labels.align")}</legend>
-          <div className="buttonList">
-            {
-              // swap this order for RTL so the button positions always match their action
-              // (i.e. the leftmost button aligns left)
-            }
-            {isRTL ? (
-              <>
-                {renderAction("alignRight")}
-                {renderAction("alignHorizontallyCentered")}
-                {renderAction("alignLeft")}
-              </>
-            ) : (
-              <>
-                {renderAction("alignLeft")}
-                {renderAction("alignHorizontallyCentered")}
-                {renderAction("alignRight")}
-              </>
-            )}
-            {targetElements.length > 2 &&
-              renderAction("distributeHorizontally")}
-            <div className="iconRow">
-              {renderAction("alignTop")}
-              {renderAction("alignVerticallyCentered")}
-              {renderAction("alignBottom")}
-              {targetElements.length > 2 &&
-                renderAction("distributeVertically")}
-            </div>
-          </div>
-        </fieldset>
-      )}
       {!isMobile && !isEditing && targetElements.length > 0 && (
         <fieldset>
           <legend>{t("labels.actions")}</legend>
           <div className="buttonList">
             {renderAction("duplicateSelection")}
             {renderAction("deleteSelectedElements")}
-            {renderAction("group")}
-            {renderAction("ungroup")}
           </div>
         </fieldset>
       )}
     </div>
   );
-};
+}
 
-const LIBRARY_ICON = (
-  // fa-th-large
-  <svg viewBox="0 0 512 512">
-    <path d="M296 32h192c13.255 0 24 10.745 24 24v160c0 13.255-10.745 24-24 24H296c-13.255 0-24-10.745-24-24V56c0-13.255 10.745-24 24-24zm-80 0H24C10.745 32 0 42.745 0 56v160c0 13.255 10.745 24 24 24h192c13.255 0 24-10.745 24-24V56c0-13.255-10.745-24-24-24zM0 296v160c0 13.255 10.745 24 24 24h192c13.255 0 24-10.745 24-24V296c0-13.255-10.745-24-24-24H24c-13.255 0-24 10.745-24 24zm296 184h192c13.255 0 24-10.745 24-24V296c0-13.255-10.745-24-24-24H296c-13.255 0-24 10.745-24 24v160c0 13.255 10.745 24 24 24z" />
-  </svg>
-);
-
-export const ShapesSwitcher = ({
+export function ShapesSwitcher({
   elementType,
   setAppState,
-  isLibraryOpen,
 }: {
   elementType: ExcalidrawElement["type"];
-  setAppState: React.Component<any, AppState>["setState"];
-  isLibraryOpen: boolean;
-}) => (
-  <>
-    {SHAPES.map(({ value, icon, key }, index) => {
-      const label = t(`toolBar.${value}`);
-      const letter = typeof key === "string" ? key : key[0];
-      const shortcut = `${capitalizeString(letter)} ${t("helpDialog.or")} ${
-        index + 1
-      }`;
-      return (
-        <ToolButton
-          className="Shape"
-          key={value}
-          type="radio"
-          icon={icon}
-          checked={elementType === value}
-          name="editor-current-shape"
-          title={`${capitalizeString(label)} — ${shortcut}`}
-          keyBindingLabel={`${index + 1}`}
-          aria-label={capitalizeString(label)}
-          aria-keyshortcuts={shortcut}
-          data-testid={value}
-          onChange={() => {
-            setAppState({
-              elementType: value,
-              multiElement: null,
-              selectedElementIds: {},
-            });
-            setCursorForShape(value);
-            setAppState({});
-          }}
-        />
-      );
-    })}
-    <ToolButton
-      className="Shape ToolIcon_type_button__library"
-      type="button"
-      icon={LIBRARY_ICON}
-      name="editor-library"
-      keyBindingLabel="9"
-      aria-keyshortcuts="9"
-      title={`${capitalizeString(t("toolBar.library"))} — 9`}
-      aria-label={capitalizeString(t("toolBar.library"))}
-      onClick={() => {
-        setAppState({ isLibraryOpen: !isLibraryOpen });
-      }}
-    />
-  </>
-);
+  setAppState: any;
+}) {
+  return (
+    <>
+      {SHAPES.map(({ value, icon, key }, index) => {
+        const label = t(`toolBar.${value}`);
+        const shortcut = `${capitalizeString(key)} ${t("shortcutsDialog.or")} ${
+          index + 1
+        }`;
+        return (
+          <ToolButton
+            key={value}
+            type="radio"
+            icon={icon}
+            checked={elementType === value}
+            name="editor-current-shape"
+            title={`${capitalizeString(label)} — ${shortcut}`}
+            keyBindingLabel={`${index + 1}`}
+            aria-label={capitalizeString(label)}
+            aria-keyshortcuts={`${key} ${index + 1}`}
+            data-testid={value}
+            onChange={() => {
+              setAppState({
+                elementType: value,
+                multiElement: null,
+                selectedElementIds: {},
+              });
+              setCursorForShape(value);
+              setAppState({});
+            }}
+          ></ToolButton>
+        );
+      })}
+    </>
+  );
+}
 
-export const ZoomActions = ({
+export function ZoomActions({
   renderAction,
   zoom,
 }: {
   renderAction: ActionManager["renderAction"];
-  zoom: Zoom;
-}) => (
-  <Stack.Col gap={1}>
-    <Stack.Row gap={1} align="center">
-      {renderAction("zoomIn")}
-      {renderAction("zoomOut")}
-      {renderAction("resetZoom")}
-      <div style={{ marginInlineStart: 4 }}>
-        {(zoom.value * 100).toFixed(0)}%
-      </div>
-    </Stack.Row>
-  </Stack.Col>
-);
+  zoom: number;
+}) {
+  return (
+    <Stack.Col gap={1}>
+      <Stack.Row gap={1} align="center">
+        {renderAction("zoomIn")}
+        {renderAction("zoomOut")}
+        {renderAction("resetZoom")}
+        <div style={{ marginInlineStart: 4 }}>{(zoom * 100).toFixed(0)}%</div>
+      </Stack.Row>
+    </Stack.Col>
+  );
+}

src/components/Avatar.scss

@@ -1,16 +0,0 @@
-@import "../css/variables.module";
-
-.excalidraw {
-  .Avatar {
-    width: 2.5rem;
-    height: 2.5rem;
-    border-radius: 1.25rem;
-    display: flex;
-    justify-content: center;
-    align-items: center;
-    color: $oc-white;
-    cursor: pointer;
-    font-size: 0.8rem;
-    font-weight: 500;
-  }
-}

src/components/Avatar.tsx

@@ -1,20 +0,0 @@
-import "./Avatar.scss";
-
-import React from "react";
-
-type AvatarProps = {
-  children: string;
-  onClick: (e: React.MouseEvent<HTMLDivElement, MouseEvent>) => void;
-  color: string;
-  border: string;
-};
-
-export const Avatar = ({ children, color, border, onClick }: AvatarProps) => (
-  <div
-    className="Avatar"
-    style={{ background: color, border: `1px solid ${border}` }}
-    onClick={onClick}
-  >
-    {children}
-  </div>
-);

src/components/BackgroundPickerAndDarkModeToggle.tsx

@@ -1,26 +0,0 @@
-import React from "react";
-import { ActionManager } from "../actions/manager";
-import { AppState } from "../types";
-import { DarkModeToggle } from "./DarkModeToggle";
-
-export const BackgroundPickerAndDarkModeToggle = ({
-  appState,
-  setAppState,
-  actionManager,
-}: {
-  actionManager: ActionManager;
-  appState: AppState;
-  setAppState: React.Component<any, AppState>["setState"];
-}) => (
-  <div style={{ display: "flex" }}>
-    {actionManager.renderAction("changeViewBackgroundColor")}
-    <div style={{ marginInlineStart: "0.25rem" }}>
-      <DarkModeToggle
-        value={appState.appearance}
-        onChange={(appearance) => {
-          setAppState({ appearance });
-        }}
-      />
-    </div>
-  </div>
-);

src/components/ButtonIconCycle.tsx

@@ -1,29 +0,0 @@
-import React from "react";
-import clsx from "clsx";
-
-export const ButtonIconCycle = <T extends any>({
-  options,
-  value,
-  onChange,
-  group,
-}: {
-  options: { value: T; text: string; icon: JSX.Element }[];
-  value: T | null;
-  onChange: (value: T) => void;
-  group: string;
-}) => {
-  const current = options.find((op) => op.value === value);
-
-  function cycle() {
-    const index = options.indexOf(current!);
-    const next = (index + 1) % options.length;
-    onChange(options[next].value);
-  }
-
-  return (
-    <label key={group} className={clsx({ active: current!.value !== null })}>
-      <input type="button" name={group} onClick={cycle} />
-      {current!.icon}
-    </label>
-  );
-};

src/components/ButtonIconSelect.tsx

@@ -1,33 +0,0 @@
-import React from "react";
-import clsx from "clsx";
-
-// TODO: It might be "clever" to add option.icon to the existing component <ButtonSelect />
-export const ButtonIconSelect = <T extends Object>({
-  options,
-  value,
-  onChange,
-  group,
-}: {
-  options: { value: T; text: string; icon: JSX.Element }[];
-  value: T | null;
-  onChange: (value: T) => void;
-  group: string;
-}) => (
-  <div className="buttonList buttonListIcon">
-    {options.map((option) => (
-      <label
-        key={option.text}
-        className={clsx({ active: value === option.value })}
-        title={option.text}
-      >
-        <input
-          type="radio"
-          name={group}
-          onChange={() => onChange(option.value)}
-          checked={value === option.value}
-        />
-        {option.icon}
-      </label>
-    ))}
-  </div>
-);

src/components/ButtonSelect.tsx

@@ -1,7 +1,6 @@
 import React from "react";
-import clsx from "clsx";
 
-export const ButtonSelect = <T extends Object>({
+export function ButtonSelect<T>({
   options,
   value,
   onChange,
@@ -11,21 +10,23 @@ export const ButtonSelect = <T extends Object>({
   value: T | null;
   onChange: (value: T) => void;
   group: string;
-}) => (
-  <div className="buttonList">
-    {options.map((option) => (
-      <label
-        key={option.text}
-        className={clsx({ active: value === option.value })}
-      >
-        <input
-          type="radio"
-          name={group}
-          onChange={() => onChange(option.value)}
-          checked={value === option.value}
-        />
-        {option.text}
-      </label>
-    ))}
-  </div>
-);
+}) {
+  return (
+    <div className="buttonList">
+      {options.map((option) => (
+        <label
+          key={option.text}
+          className={value === option.value ? "active" : ""}
+        >
+          <input
+            type="radio"
+            name={group}
+            onChange={() => onChange(option.value)}
+            checked={value === option.value ? true : false}
+          />
+          {option.text}
+        </label>
+      ))}
+    </div>
+  );
+}

src/components/CollabButton.scss

@@ -1,29 +0,0 @@
-@import "../css/variables.module";
-
-.excalidraw {
-  .CollabButton.is-collaborating {
-    background-color: var(--button-special-active-background-color);
-
-    .ToolIcon__icon svg {
-      color: var(--icon-green-fill-color);
-    }
-  }
-
-  .CollabButton-collaborators {
-    :root[dir="ltr"] & {
-      right: -5px;
-    }
-    :root[dir="rtl"] & {
-      left: -5px;
-    }
-    min-width: 1em;
-    position: absolute;
-    bottom: -5px;
-    padding: 3px;
-    border-radius: 50%;
-    background-color: $oc-green-6;
-    color: $oc-white;
-    font-size: 0.7em;
-    font-family: var(--ui-font);
-  }
-}

src/components/CollabButton.tsx

@@ -1,40 +0,0 @@
-import React from "react";
-import clsx from "clsx";
-import { ToolButton } from "./ToolButton";
-import { t } from "../i18n";
-import useIsMobile from "../is-mobile";
-import { users } from "./icons";
-
-import "./CollabButton.scss";
-
-const CollabButton = ({
-  isCollaborating,
-  collaboratorCount,
-  onClick,
-}: {
-  isCollaborating: boolean;
-  collaboratorCount: number;
-  onClick: () => void;
-}) => {
-  return (
-    <>
-      <ToolButton
-        className={clsx("CollabButton", {
-          "is-collaborating": isCollaborating,
-        })}
-        onClick={onClick}
-        icon={users}
-        type="button"
-        title={t("buttons.roomDialog")}
-        aria-label={t("buttons.roomDialog")}
-        showAriaLabel={useIsMobile()}
-      >
-        {collaboratorCount > 0 && (
-          <div className="CollabButton-collaborators">{collaboratorCount}</div>
-        )}
-      </ToolButton>
-    </>
-  );
-};
-
-export default CollabButton;

src/components/ColorPicker.scss

@@ -1,250 +1,191 @@
-@import "../css/variables.module";
+@import "open-color/open-color.scss";
 
-.excalidraw {
-  .color-picker {
-    background: var(--popup-background-color);
-    border: 0px solid transparentize($oc-white, 0.75);
-    box-shadow: transparentize($oc-black, 0.75) 0px 1px 4px;
-    border-radius: 4px;
-    position: absolute;
-
-    :root[dir="ltr"] & {
-      left: -5.5px;
-    }
-
-    :root[dir="rtl"] & {
-      right: -5.5px;
-    }
+.color-picker {
+  background: $oc-white;
+  border: 0px solid transparentize($oc-white, 0.75);
+  box-shadow: transparentize($oc-black, 0.75) 0px 1px 4px;
+  border-radius: 4px;
+  position: absolute;
+  :root[dir="ltr"] & {
+    left: -5.5px;
   }
-
-  .color-picker-control-container {
-    display: grid;
-    grid-template-columns: auto 1fr;
-    align-items: center;
-  }
-
-  .color-picker-triangle {
-    width: 0px;
-    height: 0px;
-    border-style: solid;
-    border-width: 0px 9px 10px;
-    border-color: transparent transparent var(--popup-background-color);
-    position: absolute;
-    top: -10px;
-
-    :root[dir="ltr"] & {
-      left: 12px;
-    }
-
-    :root[dir="rtl"] & {
-      right: 12px;
-    }
-  }
-
-  .color-picker-triangle-shadow {
-    border-color: transparent transparent transparentize($oc-black, 0.9);
-    top: -11px;
-  }
-
-  .color-picker-content {
-    padding: 0.5rem;
-    display: grid;
-    grid-template-columns: repeat(5, auto);
-    grid-gap: 0.5rem;
-    border-radius: 4px;
-
-    &:focus {
-      outline: none;
-      box-shadow: 0 0 0 2px var(--focus-highlight-color);
-    }
-  }
-
-  .color-picker-content .color-input-container {
-    grid-column: 1 / span 5;
-  }
-
-  .color-picker-swatch {
-    position: relative;
-    height: 1.875rem;
-    width: 1.875rem;
-    cursor: pointer;
-    border-radius: 4px;
-    margin: 0;
-    box-sizing: border-box;
-    border: 1px solid #ddd;
-    background-color: currentColor !important;
-    filter: var(--appearance-filter);
-
-    &:focus {
-      /* TODO: only show the border when the color is too light to see as a shadow */
-      box-shadow: 0 0 4px 1px currentColor;
-      border-color: var(--select-highlight-color);
-    }
-  }
-
-  .color-picker-transparent {
-    border-radius: 4px;
-    box-shadow: transparentize($oc-black, 0.9) 0px 0px 0px 1px inset;
-    position: absolute;
-    top: 0px;
-    right: 0px;
-    bottom: 0px;
-    left: 0px;
-  }
-
-  .color-picker-transparent,
-  .color-picker-label-swatch {
-    background: url("data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAAMUlEQVQ4T2NkYGAQYcAP3uCTZhw1gGGYhAGBZIA/nYDCgBDAm9BGDWAAJyRCgLaBCAAgXwixzAS0pgAAAABJRU5ErkJggg==")
-      left center;
-  }
-
-  .color-picker-hash {
-    background: var(--input-border-color);
-    height: 1.875rem;
-    width: 1.875rem;
-
-    :root[dir="ltr"] & {
-      border-radius: 4px 0px 0px 4px;
-    }
-
-    :root[dir="rtl"] & {
-      border-radius: 0px 4px 4px 0px;
-    }
-
-    color: var(--input-label-color);
-    display: flex;
-    align-items: center;
-    justify-content: center;
-    position: relative;
-  }
-
-  .color-input-container:focus-within .color-picker-hash {
-    box-shadow: 0 0 0 2px var(--focus-highlight-color);
-  }
-
-  .color-input-container:focus-within .color-picker-hash::before,
-  .color-input-container:focus-within .color-picker-hash::after {
-    content: "";
-    width: 1px;
-    height: 100%;
-    position: absolute;
-    top: 0;
-  }
-
-  .color-input-container:focus-within .color-picker-hash::before {
-    background: var(--input-border-color);
-
-    :root[dir="ltr"] & {
-      right: -1px;
-    }
-
-    :root[dir="rtl"] & {
-      left: -1px;
-    }
-  }
-
-  .color-input-container:focus-within .color-picker-hash::after {
-    background: var(--input-background-color);
-
-    :root[dir="ltr"] & {
-      right: -2px;
-    }
-
-    :root[dir="rtl"] & {
-      left: -2px;
-    }
-  }
-
-  .color-input-container {
-    display: flex;
-  }
-
-  .color-picker-input {
-    width: 12ch; /* length of `transparent` + 1 */
-    margin: 0;
-    font-size: 1rem;
-    background-color: var(--input-background-color);
-    color: var(--text-color-primary);
-    border: 0px;
-    outline: none;
-    height: 1.75em;
-    box-shadow: var(--input-border-color) 0px 0px 0px 1px inset;
-
-    :root[dir="ltr"] & {
-      border-radius: 0px 4px 4px 0px;
-    }
-
-    :root[dir="rtl"] & {
-      border-radius: 4px 0px 0px 4px;
-    }
-
-    float: left;
-    padding: 1px;
-    padding-inline-start: 0.5em;
-    appearance: none;
-  }
-
-  .color-picker-label-swatch {
-    height: 1.875rem;
-    width: 1.875rem;
-    margin-inline-end: 0.25rem;
-    border: 1px solid $oc-gray-3;
-    position: relative;
-    overflow: hidden;
-    background-color: transparent !important;
-    filter: var(--appearance-filter);
-
-    &:after {
-      content: "";
-      position: absolute;
-      top: 0;
-      left: 0;
-      width: 100%;
-      height: 100%;
-      background: var(--swatch-color);
-    }
-  }
-
-  .color-picker-keybinding {
-    position: absolute;
-    bottom: 2px;
-    font-size: 0.7em;
-
-    :root[dir="ltr"] & {
-      right: 2px;
-    }
-
-    :root[dir="rtl"] & {
-      left: 2px;
-    }
-
-    @media #{$is-mobile-query} {
-      display: none;
-    }
-  }
-
-  .color-picker-type-canvasBackground .color-picker-keybinding {
-    color: #aaa;
-  }
-
-  .color-picker-type-elementBackground .color-picker-keybinding {
-    color: #fff;
-  }
-
-  .color-picker-swatch[aria-label="transparent"] .color-picker-keybinding {
-    color: #aaa;
-  }
-
-  .color-picker-type-elementStroke .color-picker-keybinding {
-    color: #d4d4d4;
-  }
-
-  &.Appearance_dark {
-    .color-picker-type-elementBackground .color-picker-keybinding {
-      color: #000;
-    }
-    .color-picker-swatch[aria-label="transparent"] .color-picker-keybinding {
-      color: #000;
-    }
+  :root[dir="rtl"] & {
+    right: -5.5px;
   }
 }
+
+.color-picker-control-container {
+  display: grid;
+  grid-template-columns: auto 1fr;
+  align-items: center;
+}
+
+.color-picker-triangle {
+  width: 0px;
+  height: 0px;
+  border-style: solid;
+  border-width: 0px 9px 10px;
+  border-color: transparent transparent $oc-white;
+  position: absolute;
+  top: -10px;
+  :root[dir="ltr"] & {
+    left: 12px;
+  }
+  :root[dir="rtl"] & {
+    right: 12px;
+  }
+}
+
+.color-picker-triangle-shadow {
+  border-color: transparent transparent transparentize($oc-black, 0.9);
+  top: -11px;
+}
+
+.color-picker-content {
+  padding: 0.5rem;
+  display: grid;
+  grid-template-columns: repeat(5, auto);
+  grid-gap: 0.5rem;
+}
+
+.color-picker-content .color-input-container {
+  grid-column: 1 / span 5;
+}
+
+.color-picker-swatch {
+  position: relative;
+  height: 1.875rem;
+  width: 1.875rem;
+  cursor: pointer;
+  border-radius: 4px;
+  margin: 0;
+  box-sizing: border-box;
+  border: 1px solid #ddd;
+  background-color: currentColor !important;
+}
+
+.color-picker-swatch:focus {
+  /* TODO: only show the border when the color is too light to see as a shadow */
+  box-shadow: 0 0 4px 1px currentColor;
+  border-color: $oc-blue-5;
+}
+
+.color-picker-transparent {
+  border-radius: 4px;
+  box-shadow: transparentize($oc-black, 0.9) 0px 0px 0px 1px inset;
+  position: absolute;
+  top: 0px;
+  right: 0px;
+  bottom: 0px;
+  left: 0px;
+}
+.color-picker-transparent,
+.color-picker-label-swatch {
+  background: url("data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAAMUlEQVQ4T2NkYGAQYcAP3uCTZhw1gGGYhAGBZIA/nYDCgBDAm9BGDWAAJyRCgLaBCAAgXwixzAS0pgAAAABJRU5ErkJggg==")
+    left center;
+}
+
+.color-picker-hash {
+  background: $oc-gray-3;
+  height: 1.875rem;
+  width: 1.875rem;
+  :root[dir="ltr"] & {
+    border-radius: 4px 0px 0px 4px;
+  }
+  :root[dir="rtl"] & {
+    border-radius: 0px 4px 4px 0px;
+  }
+  color: $oc-gray-7;
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  z-index: 1;
+  position: relative;
+}
+.color-input-container:focus-within .color-picker-hash {
+  box-shadow: 0 0 0 2px $oc-blue-2;
+}
+.color-input-container:focus-within .color-picker-hash::before,
+.color-input-container:focus-within .color-picker-hash::after {
+  content: "";
+  width: 1px;
+  height: 100%;
+  position: absolute;
+  top: 0;
+}
+.color-input-container:focus-within .color-picker-hash::before {
+  background: $oc-gray-3;
+  :root[dir="ltr"] & {
+    right: -1px;
+  }
+  :root[dir="rtl"] & {
+    left: -1px;
+  }
+}
+.color-input-container:focus-within .color-picker-hash::after {
+  background: #fff;
+  :root[dir="ltr"] & {
+    right: -2px;
+  }
+  :root[dir="rtl"] & {
+    left: -2px;
+  }
+}
+
+.color-input-container {
+  display: flex;
+}
+
+.color-picker-input {
+  width: 12ch; /* length of `transparent` + 1 */
+  margin: 0;
+  font-size: 1rem;
+  color: $oc-gray-8;
+  border: 0px;
+  outline: none;
+  height: 1.75em;
+  box-shadow: $oc-gray-3 0px 0px 0px 1px inset;
+  :root[dir="ltr"] & {
+    border-radius: 0px 4px 4px 0px;
+  }
+  :root[dir="rtl"] & {
+    border-radius: 4px 0px 0px 4px;
+  }
+  float: left;
+  padding: 1px;
+  padding-inline-start: 0.5em;
+  appearance: none;
+}
+
+.color-picker-label-swatch {
+  height: 1.875rem;
+  width: 1.875rem;
+  margin-inline-end: 0.25rem;
+  border: 1px solid $oc-gray-3;
+  position: relative;
+  overflow: hidden;
+  background-color: transparent !important;
+}
+
+.color-picker-label-swatch::after {
+  content: "";
+  position: absolute;
+  top: 0;
+  left: 0;
+  width: 100%;
+  height: 100%;
+  background: var(--swatch-color);
+}
+
+.color-picker-keybinding {
+  position: absolute;
+  bottom: 2px;
+  :root[dir="ltr"] & {
+    right: 2px;
+  }
+  :root[dir="rtl"] & {
+    left: 2px;
+  }
+  font-size: 0.7em;
+  color: #ccc;
+}

src/components/ColorPicker.tsx

@@ -2,16 +2,16 @@ import React from "react";
 import { Popover } from "./Popover";
 
 import "./ColorPicker.scss";
-import { isArrowKey, KEYS } from "../keys";
+import { KEYS } from "../keys";
 import { t, getLanguage } from "../i18n";
 import { isWritableElement } from "../utils";
 import colors from "../colors";
 
-const isValidColor = (color: string) => {
+function isValidColor(color: string) {
   const style = new Option().style;
   style.color = color;
   return !!style.color;
-};
+}
 
 const getColor = (color: string): string | null => {
   if (color === "transparent") {
@@ -36,14 +36,13 @@ const keyBindings = [
   ["a", "s", "d", "f", "g"],
 ].flat();
 
-const Picker = ({
+const Picker = function ({
   colors,
   color,
   onChange,
   onClose,
   label,
   showInput = true,
-  type,
 }: {
   colors: string[];
   color: string | null;
@@ -51,21 +50,19 @@ const Picker = ({
   onClose: () => void;
   label: string;
   showInput: boolean;
-  type: "canvasBackground" | "elementBackground" | "elementStroke";
-}) => {
+}) {
   const firstItem = React.useRef<HTMLButtonElement>();
   const activeItem = React.useRef<HTMLButtonElement>();
   const gallery = React.useRef<HTMLDivElement>();
   const colorInput = React.useRef<HTMLInputElement>();
 
   React.useEffect(() => {
-    // After the component is first mounted focus on first input
+    // After the component is first mounted
+    // focus on first input
     if (activeItem.current) {
       activeItem.current.focus();
     } else if (colorInput.current) {
       colorInput.current.focus();
-    } else if (gallery.current) {
-      gallery.current.focus();
     }
   }, []);
 
@@ -77,11 +74,18 @@ const Picker = ({
           colorInput.current?.focus();
           event.preventDefault();
         }
-      } else if (activeElement === colorInput.current) {
-        firstItem.current?.focus();
-        event.preventDefault();
+      } else {
+        if (activeElement === colorInput.current) {
+          firstItem.current?.focus();
+          event.preventDefault();
+        }
       }
-    } else if (isArrowKey(event.key)) {
+    } else if (
+      event.key === KEYS.ARROW_RIGHT ||
+      event.key === KEYS.ARROW_LEFT ||
+      event.key === KEYS.ARROW_UP ||
+      event.key === KEYS.ARROW_DOWN
+    ) {
       const { activeElement } = document;
       const isRTL = getLanguage().rtl;
       const index = Array.prototype.indexOf.call(
@@ -119,7 +123,7 @@ const Picker = ({
 
   return (
     <div
-      className={`color-picker color-picker-type-${type}`}
+      className="color-picker"
       role="dialog"
       aria-modal="true"
       aria-label={t("labels.colorPicker")}
@@ -134,7 +138,6 @@ const Picker = ({
             gallery.current = el;
           }
         }}
-        tabIndex={0}
       >
         {colors.map((_color, i) => (
           <button
@@ -232,7 +235,7 @@ const ColorInput = React.forwardRef(
   },
 );
 
-export const ColorPicker = ({
+export function ColorPicker({
   type,
   color,
   onChange,
@@ -242,7 +245,7 @@ export const ColorPicker = ({
   color: string | null;
   onChange: (color: string) => void;
   label: string;
-}) => {
+}) {
   const [isActive, setActive] = React.useState(false);
   const pickerButton = React.useRef<HTMLButtonElement>(null);
 
@@ -252,7 +255,11 @@ export const ColorPicker = ({
         <button
           className="color-picker-label-swatch"
           aria-label={label}
-          style={color ? { "--swatch-color": color } : undefined}
+          style={
+            color
+              ? ({ "--swatch-color": color } as React.CSSProperties)
+              : undefined
+          }
           onClick={() => setActive(!isActive)}
           ref={pickerButton}
         />
@@ -283,11 +290,10 @@ export const ColorPicker = ({
               }}
               label={label}
               showInput={false}
-              type={type}
             />
           </Popover>
         ) : null}
       </React.Suspense>
     </div>
   );
-};
+}

src/components/ContextMenu.scss

@@ -1,97 +1,36 @@
-@import "../css/variables.module";
+@import "open-color/open-color.scss";
 
-.excalidraw {
-  .context-menu {
-    position: relative;
-    border-radius: 4px;
-    box-shadow: 0px 3px 10px transparentize($oc-black, 0.8);
-    padding: 0;
-    list-style: none;
-    user-select: none;
-    margin: -0.25rem 0 0 0.125rem;
-    padding: 0.5rem 0;
-    background-color: var(--popup-secondary-background-color);
-    border: 1px solid var(--button-gray-3);
-    cursor: default;
-  }
-
-  .context-menu button {
-    color: var(--popup-text-color);
-  }
-
-  .context-menu-option {
-    position: relative;
-    width: 100%;
-    min-width: 9.5rem;
-    margin: 0;
-    padding: 0.25rem 1rem 0.25rem 1.25rem;
-    text-align: start;
-    border-radius: 0;
-    background-color: transparent;
-    border: none;
-    white-space: nowrap;
-
-    display: grid;
-    grid-template-columns: 1fr 0.2fr;
-    align-items: center;
-
-    &.checkmark::before {
-      position: absolute;
-      left: 6px;
-      margin-bottom: 1px;
-      content: "\2713";
-    }
-
-    &.dangerous {
-      .context-menu-option__label {
-        color: $oc-red-7;
-      }
-    }
-
-    .context-menu-option__label {
-      justify-self: start;
-      margin-inline-end: 20px;
-    }
-    .context-menu-option__shortcut {
-      justify-self: end;
-      opacity: 0.6;
-      font-family: inherit;
-      font-size: 0.7rem;
-    }
-  }
-
-  .context-menu-option:hover {
-    color: var(--popup-background-color);
-    background-color: var(--select-highlight-color);
-
-    &.dangerous {
-      .context-menu-option__label {
-        color: var(--popup-background-color);
-      }
-      background-color: $oc-red-6;
-    }
-  }
-
-  .context-menu-option:focus {
-    z-index: 1;
-  }
-
-  @media #{$is-mobile-query} {
-    .context-menu-option {
-      display: block;
-
-      .context-menu-option__label {
-        margin-inline-end: 0;
-      }
-
-      .context-menu-option__shortcut {
-        display: none;
-      }
-    }
-  }
-
-  .context-menu-option-separator {
-    border: none;
-    border-top: 1px solid $oc-gray-5;
-  }
+.context-menu {
+  position: relative;
+  border-radius: 4px;
+  box-shadow: 0px 3px 10px transparentize($oc-black, 0.8);
+  padding: 0;
+  list-style: none;
+  user-select: none;
+  margin: -0.25rem 0 0 0.125rem;
+  padding: 0.25rem 0;
+  background-color: $oc-gray-1;
+  border: 1px solid $oc-gray-5;
+}
+
+.context-menu-option {
+  position: relative;
+  width: 100%;
+  min-width: 9.5rem;
+  margin: 0;
+  padding: 0.25rem 1rem 0.25rem 1.25rem;
+  text-align: start;
+  border-radius: 0;
+  background-color: transparent;
+  border: none;
+  white-space: nowrap;
+}
+
+.context-menu-option:hover {
+  color: $oc-white;
+  background-color: $oc-blue-5;
+}
+
+.context-menu-option:focus {
+  z-index: 1;
 }

src/components/ContextMenu.tsx

@@ -1,111 +1,70 @@
 import React from "react";
-import { render, unmountComponentAtNode } from "react-dom";
-import clsx from "clsx";
 import { Popover } from "./Popover";
-import { t } from "../i18n";
+import { render, unmountComponentAtNode } from "react-dom";
 
 import "./ContextMenu.scss";
-import {
-  getShortcutFromShortcutName,
-  ShortcutName,
-} from "../actions/shortcuts";
-import { Action } from "../actions/types";
-import { ActionManager } from "../actions/manager";
-import { AppState } from "../types";
 
-export type ContextMenuOption = "separator" | Action;
+type ContextMenuOption = {
+  label: string;
+  action(): void;
+};
 
-type ContextMenuProps = {
+type Props = {
   options: ContextMenuOption[];
   onCloseRequest?(): void;
   top: number;
   left: number;
-  actionManager: ActionManager;
-  appState: Readonly<AppState>;
 };
 
-const ContextMenu = ({
-  options,
-  onCloseRequest,
-  top,
-  left,
-  actionManager,
-  appState,
-}: ContextMenuProps) => {
-  const isDarkTheme = !!document
-    .querySelector(".excalidraw")
-    ?.classList.contains("Appearance_dark");
+function ContextMenu({ options, onCloseRequest, top, left }: Props) {
   return (
-    <div
-      className={clsx("excalidraw", {
-        "Appearance_dark Appearance_dark-background-none": isDarkTheme,
-      })}
+    <Popover
+      onCloseRequest={onCloseRequest}
+      top={top}
+      left={left}
+      fitInViewport={true}
     >
-      <Popover
-        onCloseRequest={onCloseRequest}
-        top={top}
-        left={left}
-        fitInViewport={true}
+      <ul
+        className="context-menu"
+        onContextMenu={(event) => event.preventDefault()}
       >
-        <ul
-          className="context-menu"
-          onContextMenu={(event) => event.preventDefault()}
-        >
-          {options.map((option, idx) => {
-            if (option === "separator") {
-              return <hr key={idx} className="context-menu-option-separator" />;
-            }
-
-            const actionName = option.name;
-            const label = option.contextItemLabel
-              ? t(option.contextItemLabel)
-              : "";
-            return (
-              <li key={idx} data-testid={actionName} onClick={onCloseRequest}>
-                <button
-                  className={clsx("context-menu-option", {
-                    dangerous: actionName === "deleteSelectedElements",
-                    checkmark: option.checked?.(appState),
-                  })}
-                  onClick={() => actionManager.executeAction(option)}
-                >
-                  <div className="context-menu-option__label">{label}</div>
-                  <kbd className="context-menu-option__shortcut">
-                    {actionName
-                      ? getShortcutFromShortcutName(actionName as ShortcutName)
-                      : ""}
-                  </kbd>
-                </button>
-              </li>
-            );
-          })}
-        </ul>
-      </Popover>
-    </div>
+        {options.map((option, idx) => (
+          <li key={idx} onClick={onCloseRequest}>
+            <ContextMenuOption {...option} />
+          </li>
+        ))}
+      </ul>
+    </Popover>
   );
-};
+}
+
+function ContextMenuOption({ label, action }: ContextMenuOption) {
+  return (
+    <button className="context-menu-option" onClick={action}>
+      {label}
+    </button>
+  );
+}
 
 let contextMenuNode: HTMLDivElement;
-const getContextMenuNode = (): HTMLDivElement => {
+function getContextMenuNode(): HTMLDivElement {
   if (contextMenuNode) {
     return contextMenuNode;
   }
   const div = document.createElement("div");
   document.body.appendChild(div);
   return (contextMenuNode = div);
-};
+}
 
 type ContextMenuParams = {
   options: (ContextMenuOption | false | null | undefined)[];
-  top: ContextMenuProps["top"];
-  left: ContextMenuProps["left"];
-  actionManager: ContextMenuProps["actionManager"];
-  appState: Readonly<AppState>;
+  top: number;
+  left: number;
 };
 
-const handleClose = () => {
+function handleClose() {
   unmountComponentAtNode(getContextMenuNode());
-};
+}
 
 export default {
   push(params: ContextMenuParams) {
@@ -122,8 +81,6 @@ export default {
           left={params.left}
           options={options}
           onCloseRequest={handleClose}
-          actionManager={params.actionManager}
-          appState={params.appState}
         />,
         getContextMenuNode(),
       );

src/components/DarkModeToggle.tsx

@@ -1,58 +0,0 @@
-import "./ToolIcon.scss";
-
-import React from "react";
-import { t } from "../i18n";
-
-export type Appearence = "light" | "dark";
-
-// We chose to use only explicit toggle and not a third option for system value,
-// but this could be added in the future.
-export const DarkModeToggle = (props: {
-  value: Appearence;
-  onChange: (value: Appearence) => void;
-}) => {
-  return (
-    <label
-      className={`ToolIcon ToolIcon_type_floating ToolIcon_size_M`}
-      title={
-        props.value === "dark" ? t("buttons.lightMode") : t("buttons.darkMode")
-      }
-    >
-      <input
-        className="ToolIcon_type_checkbox ToolIcon_toggle_opaque"
-        type="checkbox"
-        onChange={(event) =>
-          props.onChange(event.target.checked ? "dark" : "light")
-        }
-        checked={props.value === "dark"}
-        aria-label={
-          props.value === "dark"
-            ? t("buttons.lightMode")
-            : t("buttons.darkMode")
-        }
-      />
-      <div className="ToolIcon__icon">
-        {props.value === "light" ? ICONS.MOON : ICONS.SUN}
-      </div>
-    </label>
-  );
-};
-
-const ICONS = {
-  SUN: (
-    <svg width="512" height="512" className="rtl-mirror" viewBox="0 0 512 512">
-      <path
-        fill="currentColor"
-        d="M256 160c-52.9 0-96 43.1-96 96s43.1 96 96 96 96-43.1 96-96-43.1-96-96-96zm246.4 80.5l-94.7-47.3 33.5-100.4c4.5-13.6-8.4-26.5-21.9-21.9l-100.4 33.5-47.4-94.8c-6.4-12.8-24.6-12.8-31 0l-47.3 94.7L92.7 70.8c-13.6-4.5-26.5 8.4-21.9 21.9l33.5 100.4-94.7 47.4c-12.8 6.4-12.8 24.6 0 31l94.7 47.3-33.5 100.5c-4.5 13.6 8.4 26.5 21.9 21.9l100.4-33.5 47.3 94.7c6.4 12.8 24.6 12.8 31 0l47.3-94.7 100.4 33.5c13.6 4.5 26.5-8.4 21.9-21.9l-33.5-100.4 94.7-47.3c13-6.5 13-24.7.2-31.1zm-155.9 106c-49.9 49.9-131.1 49.9-181 0-49.9-49.9-49.9-131.1 0-181 49.9-49.9 131.1-49.9 181 0 49.9 49.9 49.9 131.1 0 181z"
-      ></path>
-    </svg>
-  ),
-  MOON: (
-    <svg width="512" height="512" className="rtl-mirror" viewBox="0 0 512 512">
-      <path
-        fill="currentColor"
-        d="M283.211 512c78.962 0 151.079-35.925 198.857-94.792 7.068-8.708-.639-21.43-11.562-19.35-124.203 23.654-238.262-71.576-238.262-196.954 0-72.222 38.662-138.635 101.498-174.394 9.686-5.512 7.25-20.197-3.756-22.23A258.156 258.156 0 0 0 283.211 0c-141.309 0-256 114.511-256 256 0 141.309 114.511 256 256 256z"
-      ></path>
-    </svg>
-  ),
-};

src/components/Dialog.scss

@@ -1,74 +1,60 @@
-@import "../css/variables.module";
+@import "../css/_variables";
 
-.excalidraw {
+.Dialog__title {
+  display: grid;
+  align-items: center;
+  margin-top: 0;
+  grid-template-columns: 1fr calc(var(--space-factor) * 7);
+  grid-gap: var(--metric);
+}
+
+.Dialog__titleContent {
+  flex: 1;
+}
+
+.Dialog .Modal__close {
+  margin: 0;
+}
+
+@media #{$media-query} {
   .Dialog {
-    user-select: text;
-    cursor: auto;
+    --metric: calc(var(--space-factor) * 4);
+    --inset-left: #{"max(var(--metric), var(--sal))"};
+    --inset-right: #{"max(var(--metric), var(--sar))"};
   }
-
   .Dialog__title {
-    display: grid;
-    align-items: center;
-    margin-top: 0;
-    grid-template-columns: 1fr calc(var(--space-factor) * 7);
-    grid-gap: var(--metric);
+    grid-template-columns: calc(var(--space-factor) * 7) 1fr calc(
+        var(--space-factor) * 7
+      );
+    position: sticky;
+    top: calc(-1 * var(--metric));
+    margin: calc(-1 * var(--inset-right));
+    margin-top: calc(-1 * var(--metric));
+    margin-bottom: var(--metric);
     padding: calc(var(--space-factor) * 2);
-    text-align: center;
-    font-variant: small-caps;
-    font-size: 1.2em;
-  }
+    padding-left: var(--inset-left);
+    padding-right: var(--inset-right);
+    background: white;
+    font-size: 1.25em;
 
+    box-sizing: border-box;
+    border-bottom: 1px solid $oc-gray-4;
+    z-index: 1;
+  }
   .Dialog__titleContent {
-    flex: 1;
+    text-align: center;
+  }
+  .Dialog .Island {
+    width: 100vw;
+    height: 100%;
+    box-sizing: border-box;
+    overflow-y: auto;
+    padding-left: #{"max(calc(var(--padding) * var(--space-factor)), var(--sal))"};
+    padding-right: #{"max(calc(var(--padding) * var(--space-factor)), var(--sar))"};
+    padding-bottom: #{"max(calc(var(--padding) * var(--space-factor)), var(--sab))"};
   }
 
   .Dialog .Modal__close {
-    color: var(--icon-fill-color);
-    margin: 0;
-  }
-
-  .Dialog__content {
-    padding: 0 16px 16px;
-  }
-
-  @media #{$is-mobile-query} {
-    .Dialog {
-      --metric: calc(var(--space-factor) * 4);
-      --inset-left: #{"max(var(--metric), var(--sal))"};
-      --inset-right: #{"max(var(--metric), var(--sar))"};
-    }
-
-    .Dialog__title {
-      grid-template-columns: calc(var(--space-factor) * 7) 1fr calc(
-          var(--space-factor) * 7
-        );
-      position: sticky;
-      top: 0;
-      padding: calc(var(--space-factor) * 2);
-      background: var(--bg-color-island);
-      font-size: 1.25em;
-
-      box-sizing: border-box;
-      border-bottom: 1px solid var(--button-gray-2);
-      z-index: 1;
-    }
-
-    .Dialog__titleContent {
-      text-align: center;
-    }
-
-    .Dialog .Island {
-      width: 100vw;
-      height: 100%;
-      box-sizing: border-box;
-      overflow-y: auto;
-      padding-left: #{"max(calc(var(--padding) * var(--space-factor)), var(--sal))"};
-      padding-right: #{"max(calc(var(--padding) * var(--space-factor)), var(--sar))"};
-      padding-bottom: #{"max(calc(var(--padding) * var(--space-factor)), var(--sab))"};
-    }
-
-    .Dialog .Modal__close {
-      order: -1;
-    }
+    order: -1;
   }
 }

src/components/Dialog.tsx

@@ -1,39 +1,39 @@
-import clsx from "clsx";
-import React, { useEffect } from "react";
-import { useCallbackRefState } from "../hooks/useCallbackRefState";
+import React, { useEffect, useRef } from "react";
+import { Modal } from "./Modal";
+import { Island } from "./Island";
 import { t } from "../i18n";
 import useIsMobile from "../is-mobile";
-import { KEYS } from "../keys";
-import "./Dialog.scss";
 import { back, close } from "./icons";
-import { Island } from "./Island";
-import { Modal } from "./Modal";
+import { KEYS } from "../keys";
 
-export const Dialog = (props: {
+import "./Dialog.scss";
+
+export function Dialog(props: {
   children: React.ReactNode;
   className?: string;
-  small?: boolean;
+  maxWidth?: number;
   onCloseRequest(): void;
   title: React.ReactNode;
-  autofocus?: boolean;
-}) => {
-  const [islandNode, setIslandNode] = useCallbackRefState<HTMLDivElement>();
+}) {
+  const islandRef = useRef<HTMLDivElement>(null);
 
   useEffect(() => {
-    if (!islandNode) {
-      return;
-    }
+    const focusableElements = queryFocusableElements();
 
-    const focusableElements = queryFocusableElements(islandNode);
-
-    if (focusableElements.length > 0 && props.autofocus !== false) {
+    if (focusableElements.length > 0) {
       // If there's an element other than close, focus it.
       (focusableElements[1] || focusableElements[0]).focus();
     }
+  }, []);
 
-    const handleKeyDown = (event: KeyboardEvent) => {
+  useEffect(() => {
+    if (!islandRef.current) {
+      return;
+    }
+
+    function handleKeyDown(event: KeyboardEvent) {
       if (event.key === KEYS.TAB) {
-        const focusableElements = queryFocusableElements(islandNode);
+        const focusableElements = queryFocusableElements();
         const { activeElement } = document;
         const currentIndex = focusableElements.findIndex(
           (element) => element === activeElement,
@@ -50,29 +50,30 @@ export const Dialog = (props: {
           event.preventDefault();
         }
       }
-    };
+    }
 
-    islandNode.addEventListener("keydown", handleKeyDown);
+    const node = islandRef.current;
+    node.addEventListener("keydown", handleKeyDown);
 
-    return () => islandNode.removeEventListener("keydown", handleKeyDown);
-  }, [islandNode, props.autofocus]);
+    return () => node.removeEventListener("keydown", handleKeyDown);
+  }, []);
 
-  const queryFocusableElements = (node: HTMLElement) => {
-    const focusableElements = node.querySelectorAll<HTMLElement>(
+  function queryFocusableElements() {
+    const focusableElements = islandRef.current?.querySelectorAll<HTMLElement>(
       "button, a, input, select, textarea, div[tabindex]",
     );
 
     return focusableElements ? Array.from(focusableElements) : [];
-  };
+  }
 
   return (
     <Modal
-      className={clsx("Dialog", props.className)}
+      className={`${props.className ?? ""} Dialog`}
       labelledBy="dialog-title"
-      maxWidth={props.small ? 550 : 800}
+      maxWidth={props.maxWidth}
       onCloseRequest={props.onCloseRequest}
     >
-      <Island ref={setIslandNode}>
+      <Island padding={4} ref={islandRef}>
         <h2 id="dialog-title" className="Dialog__title">
           <span className="Dialog__titleContent">{props.title}</span>
           <button
@@ -83,8 +84,8 @@ export const Dialog = (props: {
             {useIsMobile() ? back : close}
           </button>
         </h2>
-        <div className="Dialog__content">{props.children}</div>
+        {props.children}
       </Island>
     </Modal>
   );
-};
+}

src/components/ErrorDialog.tsx

@@ -3,13 +3,13 @@ import { t } from "../i18n";
 
 import { Dialog } from "./Dialog";
 
-export const ErrorDialog = ({
+export function ErrorDialog({
   message,
   onClose,
 }: {
   message: string;
   onClose?: () => void;
-}) => {
+}) {
   const [modalIsShown, setModalIsShown] = useState(!!message);
 
   const handleClose = React.useCallback(() => {
@@ -24,20 +24,13 @@ export const ErrorDialog = ({
     <>
       {modalIsShown && (
         <Dialog
-          small
+          maxWidth={500}
           onCloseRequest={handleClose}
           title={t("errorDialog.title")}
         >
-          <div>
-            {message.split("\n").map((line) => (
-              <>
-                {line}
-                <br />
-              </>
-            ))}
-          </div>
+          <div>{message}</div>
         </Dialog>
       )}
     </>
   );
-};
+}

src/components/ExportDialog.scss

@@ -1,69 +1,57 @@
-@import "../css/variables.module";
+@import "../css/_variables";
 
-.excalidraw {
-  .ExportDialog__preview {
-    --preview-padding: calc(var(--space-factor) * 4);
+.ExportDialog__preview {
+  --preview-padding: calc(var(--space-factor) * 4);
 
-    background: url("data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAAMUlEQVQ4T2NkYGAQYcAP3uCTZhw1gGGYhAGBZIA/nYDCgBDAm9BGDWAAJyRCgLaBCAAgXwixzAS0pgAAAABJRU5ErkJggg==")
-      left center;
-    text-align: center;
-    padding: var(--preview-padding);
+  background: url("data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAAMUlEQVQ4T2NkYGAQYcAP3uCTZhw1gGGYhAGBZIA/nYDCgBDAm9BGDWAAJyRCgLaBCAAgXwixzAS0pgAAAABJRU5ErkJggg==")
+    left center;
+  text-align: center;
+  padding: var(--preview-padding);
+  margin-bottom: calc(var(--space-factor) * 3);
+}
+
+.ExportDialog__preview canvas {
+  max-width: calc(100% - var(--preview-padding) * 2);
+  max-height: 25rem;
+}
+
+.ExportDialog__actions {
+  width: 100%;
+  display: flex;
+  grid-gap: calc(var(--space-factor) * 2);
+  align-items: top;
+  justify-content: space-between;
+}
+
+.ExportDialog__name {
+  grid-column: project-name;
+  margin: auto;
+}
+
+@media (max-width: 550px) {
+  .ExportDialog {
+    display: flex;
+    flex-direction: column;
+  }
+  .ExportDialog__actions {
+    flex-direction: column;
+    align-items: center;
+  }
+  .ExportDialog__actions > * {
     margin-bottom: calc(var(--space-factor) * 3);
   }
+}
 
+@media #{$media-query} {
   .ExportDialog__preview canvas {
-    max-width: calc(100% - var(--preview-padding) * 2);
-    max-height: 25rem;
+    max-height: 30vh;
   }
-
-  &.Appearance_dark .ExportDialog__preview canvas {
-    filter: none;
+  .ExportDialog__dialog,
+  .ExportDialog__dialog .Island {
+    height: 100%;
+    box-sizing: border-box;
   }
-
-  .ExportDialog__actions {
-    width: 100%;
-    display: flex;
-    grid-gap: calc(var(--space-factor) * 2);
-    align-items: top;
-    justify-content: space-between;
-  }
-
-  .ExportDialog__name {
-    grid-column: project-name;
-    margin: auto;
-
-    .TextInput {
-      height: calc(1rem - 3px);
-    }
-  }
-
-  @media #{$is-mobile-query} {
-    .ExportDialog {
-      display: flex;
-      flex-direction: column;
-    }
-
-    .ExportDialog__actions {
-      flex-direction: column;
-      align-items: center;
-    }
-
-    .ExportDialog__actions > * {
-      margin-bottom: calc(var(--space-factor) * 3);
-    }
-
-    .ExportDialog__preview canvas {
-      max-height: 30vh;
-    }
-
-    .ExportDialog__dialog,
-    .ExportDialog__dialog .Island {
-      height: 100%;
-      box-sizing: border-box;
-    }
-
-    .ExportDialog__dialog .Island {
-      overflow-y: auto;
-    }
+  .ExportDialog__dialog .Island {
+    overflow-y: auto;
   }
 }

src/components/ExportDialog.tsx

@@ -1,55 +1,30 @@
-import React, { useEffect, useRef, useState } from "react";
-import { render, unmountComponentAtNode } from "react-dom";
-import { ActionsManagerInterface } from "../actions/types";
-import { probablySupportsClipboardBlob } from "../clipboard";
-import { canvasToBlob } from "../data/blob";
-import { NonDeletedExcalidrawElement } from "../element/types";
-import { CanvasError } from "../errors";
-import { t } from "../i18n";
-import useIsMobile from "../is-mobile";
-import { getSelectedElements, isSomeElementSelected } from "../scene";
-import { exportToCanvas, getExportSize } from "../scene/export";
-import { AppState } from "../types";
-import { Dialog } from "./Dialog";
 import "./ExportDialog.scss";
-import { clipboard, exportFile, link } from "./icons";
-import Stack from "./Stack";
+
+import React, { useState, useEffect, useRef } from "react";
+
 import { ToolButton } from "./ToolButton";
+import { clipboard, exportFile, link } from "./icons";
+import { NonDeletedExcalidrawElement } from "../element/types";
+import { AppState } from "../types";
+import { exportToCanvas } from "../scene/export";
+import { ActionsManagerInterface } from "../actions/types";
+import Stack from "./Stack";
+import { t } from "../i18n";
+
+import { probablySupportsClipboardBlob } from "../clipboard";
+import { getSelectedElements, isSomeElementSelected } from "../scene";
+import useIsMobile from "../is-mobile";
+import { Dialog } from "./Dialog";
 
 const scales = [1, 2, 3];
 const defaultScale = scales.includes(devicePixelRatio) ? devicePixelRatio : 1;
 
-export const ErrorCanvasPreview = () => {
-  return (
-    <div>
-      <h3>{t("canvasError.cannotShowPreview")}</h3>
-      <p>
-        <span>{t("canvasError.canvasTooBig")}</span>
-      </p>
-      <em>({t("canvasError.canvasTooBigTip")})</em>
-    </div>
-  );
-};
-
-const renderPreview = (
-  content: HTMLCanvasElement | Error,
-  previewNode: HTMLDivElement,
-) => {
-  unmountComponentAtNode(previewNode);
-  previewNode.innerHTML = "";
-  if (content instanceof HTMLCanvasElement) {
-    previewNode.appendChild(content);
-  } else {
-    render(<ErrorCanvasPreview />, previewNode);
-  }
-};
-
 export type ExportCB = (
   elements: readonly NonDeletedExcalidrawElement[],
   scale?: number,
 ) => void;
 
-const ExportModal = ({
+function ExportModal({
   elements,
   appState,
   exportPadding = 10,
@@ -66,9 +41,9 @@ const ExportModal = ({
   onExportToPng: ExportCB;
   onExportToSvg: ExportCB;
   onExportToClipboard: ExportCB;
-  onExportToBackend?: ExportCB;
+  onExportToBackend: ExportCB;
   onCloseRequest: () => void;
-}) => {
+}) {
   const someElementIsSelected = isSomeElementSelected(elements, appState);
   const [scale, setScale] = useState(defaultScale);
   const [exportSelected, setExportSelected] = useState(someElementIsSelected);
@@ -89,32 +64,17 @@ const ExportModal = ({
 
   useEffect(() => {
     const previewNode = previewRef.current;
-    if (!previewNode) {
-      return;
-    }
-    try {
-      const canvas = exportToCanvas(exportedElements, appState, {
-        exportBackground,
-        viewBackgroundColor,
-        exportPadding,
-        scale,
-        shouldAddWatermark,
-      });
-
-      // if converting to blob fails, there's some problem that will
-      // likely prevent preview and export (e.g. canvas too big)
-      canvasToBlob(canvas)
-        .then(() => {
-          renderPreview(canvas, previewNode);
-        })
-        .catch((error) => {
-          console.error(error);
-          renderPreview(new CanvasError(), previewNode);
-        });
-    } catch (error) {
-      console.error(error);
-      renderPreview(new CanvasError(), previewNode);
-    }
+    const canvas = exportToCanvas(exportedElements, appState, {
+      exportBackground,
+      viewBackgroundColor,
+      exportPadding,
+      scale,
+      shouldAddWatermark,
+    });
+    previewNode?.appendChild(canvas);
+    return () => {
+      previewNode?.removeChild(canvas);
+    };
   }, [
     appState,
     exportedElements,
@@ -127,7 +87,7 @@ const ExportModal = ({
 
   return (
     <div className="ExportDialog">
-      <div className="ExportDialog__preview" ref={previewRef} />
+      <div className="ExportDialog__preview" ref={previewRef}></div>
       <Stack.Col gap={2} align="center">
         <div className="ExportDialog__actions">
           <Stack.Row gap={2}>
@@ -154,47 +114,31 @@ const ExportModal = ({
                 onClick={() => onExportToClipboard(exportedElements, scale)}
               />
             )}
-            {onExportToBackend && (
-              <ToolButton
-                type="button"
-                icon={link}
-                title={t("buttons.getShareableLink")}
-                aria-label={t("buttons.getShareableLink")}
-                onClick={() => onExportToBackend(exportedElements)}
-              />
-            )}
+            <ToolButton
+              type="button"
+              icon={link}
+              title={t("buttons.getShareableLink")}
+              aria-label={t("buttons.getShareableLink")}
+              onClick={() => onExportToBackend(exportedElements)}
+            />
           </Stack.Row>
           <div className="ExportDialog__name">
             {actionManager.renderAction("changeProjectName")}
           </div>
           <Stack.Row gap={2}>
-            {scales.map((s) => {
-              const [width, height] = getExportSize(
-                exportedElements,
-                exportPadding,
-                shouldAddWatermark,
-                s,
-              );
-
-              const scaleButtonTitle = `${t(
-                "buttons.scale",
-              )} ${s}x (${width}x${height})`;
-
-              return (
-                <ToolButton
-                  key={s}
-                  size="s"
-                  type="radio"
-                  icon={`${s}x`}
-                  name="export-canvas-scale"
-                  title={scaleButtonTitle}
-                  aria-label={scaleButtonTitle}
-                  id="export-canvas-scale"
-                  checked={s === scale}
-                  onChange={() => setScale(s)}
-                />
-              );
-            })}
+            {scales.map((s) => (
+              <ToolButton
+                key={s}
+                size="s"
+                type="radio"
+                icon={`x${s}`}
+                name="export-canvas-scale"
+                aria-label={`Scale ${s} x`}
+                id="export-canvas-scale"
+                checked={s === scale}
+                onChange={() => setScale(s)}
+              />
+            ))}
           </Stack.Row>
         </div>
         {actionManager.renderAction("changeExportBackground")}
@@ -212,14 +156,13 @@ const ExportModal = ({
             </label>
           </div>
         )}
-        {actionManager.renderAction("changeExportEmbedScene")}
         {actionManager.renderAction("changeShouldAddWatermark")}
       </Stack.Col>
     </div>
   );
-};
+}
 
-export const ExportDialog = ({
+export function ExportDialog({
   elements,
   appState,
   exportPadding = 10,
@@ -236,8 +179,8 @@ export const ExportDialog = ({
   onExportToPng: ExportCB;
   onExportToSvg: ExportCB;
   onExportToClipboard: ExportCB;
-  onExportToBackend?: ExportCB;
-}) => {
+  onExportToBackend: ExportCB;
+}) {
   const [modalIsShown, setModalIsShown] = useState(false);
   const triggerButton = useRef<HTMLButtonElement>(null);
 
@@ -249,9 +192,7 @@ export const ExportDialog = ({
   return (
     <>
       <ToolButton
-        onClick={() => {
-          setModalIsShown(true);
-        }}
+        onClick={() => setModalIsShown(true)}
         icon={exportFile}
         type="button"
         aria-label={t("buttons.export")}
@@ -260,7 +201,11 @@ export const ExportDialog = ({
         ref={triggerButton}
       />
       {modalIsShown && (
-        <Dialog onCloseRequest={handleClose} title={t("buttons.export")}>
+        <Dialog
+          maxWidth={800}
+          onCloseRequest={handleClose}
+          title={t("buttons.export")}
+        >
           <ExportModal
             elements={elements}
             appState={appState}
@@ -276,4 +221,4 @@ export const ExportDialog = ({
       )}
     </>
   );
-};
+}

src/components/FixedSideContainer.css

@@ -0,0 +1,36 @@
+.FixedSideContainer {
+  --margin: 0.25rem;
+  position: fixed;
+  pointer-events: none;
+}
+
+.FixedSideContainer > * {
+  pointer-events: all;
+}
+
+.FixedSideContainer_side_top {
+  left: var(--margin);
+  top: var(--margin);
+  right: var(--margin);
+  z-index: 2;
+}
+
+.FixedSideContainer_side_top.zen-mode {
+  right: 42px;
+}
+
+/* TODO: if these are used, make sure to implement RTL support
+.FixedSideContainer_side_left {
+  left: var(--margin);
+  top: var(--margin);
+  bottom: var(--margin);
+  z-index: 1;
+}
+
+.FixedSideContainer_side_right {
+  right: var(--margin);
+  top: var(--margin);
+  bottom: var(--margin);
+  z-index: 3;
+}
+*/

src/components/FixedSideContainer.scss

@@ -1,38 +0,0 @@
-.excalidraw {
-  .FixedSideContainer {
-    --margin: 0.25rem;
-    position: absolute;
-    pointer-events: none;
-  }
-
-  .FixedSideContainer > * {
-    pointer-events: all;
-  }
-
-  .FixedSideContainer_side_top {
-    left: var(--margin);
-    top: var(--margin);
-    right: var(--margin);
-    z-index: 2;
-  }
-
-  .FixedSideContainer_side_top.zen-mode {
-    right: 42px;
-  }
-}
-
-/* TODO: if these are used, make sure to implement RTL support
-.FixedSideContainer_side_left {
-  left: var(--margin);
-  top: var(--margin);
-  bottom: var(--margin);
-  z-index: 1;
-}
-
-.FixedSideContainer_side_right {
-  right: var(--margin);
-  top: var(--margin);
-  bottom: var(--margin);
-  z-index: 3;
-}
-*/