tldraw 3.15.0-next.d30ed5ad740e → 3.15.0-next.f1dfcef63951

package/src/lib/ui/context/actions.tsx

@@ -28,7 +28,6 @@ import { fitFrameToContent, removeFrame } from '../../utils/frames/frames'
 import { generateShapeAnnouncementMessage } from '../components/A11y'
 import { EditLinkDialog } from '../components/EditLinkDialog'
 import { EmbedDialog } from '../components/EmbedDialog'
-import { DefaultKeyboardShortcutsDialog } from '../components/KeyboardShortcutsDialog/DefaultKeyboardShortcutsDialog'
 import { useShowCollaborationUi } from '../hooks/useCollaborationStatus'
 import { flattenShapesToImages } from '../hooks/useFlatten'
 import { TLUiTranslationKey } from '../hooks/useTranslation/TLUiTranslationKey'
@@ -49,7 +48,6 @@ export interface TLUiActionItem<
 	label?: TransationKey | { [key: string]: TransationKey }
 	readonlyOk?: boolean
 	checkbox?: boolean
-	isRequiredA11yAction?: boolean
 	onSelect(source: TLUiEventSource): Promise<void> | void
 }
 
@@ -170,15 +168,6 @@ export function ActionsProvider({ overrides, children }: ActionsProviderProps) {
 					helpers.addDialog({ component: EmbedDialog })
 				},
 			},
-			{
-				id: 'open-kbd-shortcuts',
-				label: 'action.open-kbd-shortcuts',
-				kbd: 'cmd+alt+/,ctrl+alt+/',
-				onSelect(source) {
-					trackEvent('open-kbd-shortcuts', { source })
-					helpers.addDialog({ component: DefaultKeyboardShortcutsDialog })
-				},
-			},
 			{
 				id: 'insert-media',
 				label: 'action.insert-media',
@@ -513,6 +502,7 @@ export function ActionsProvider({ overrides, children }: ActionsProviderProps) {
 								}
 					}
 
+					if (!editor.canCreateShapes(ids)) return
 					editor.markHistoryStoppingPoint('duplicate shapes')
 					editor.duplicateShapes(ids, offset)
 
@@ -1244,21 +1234,6 @@ export function ActionsProvider({ overrides, children }: ActionsProviderProps) {
 				},
 				checkbox: true,
 			},
-			{
-				id: 'toggle-keyboard-shortcuts',
-				label: {
-					default: 'action.toggle-keyboard-shortcuts',
-					menu: 'action.toggle-keyboard-shortcuts.menu',
-				},
-				readonlyOk: true,
-				onSelect(source) {
-					trackEvent('toggle-keyboard-shortcuts', { source })
-					editor.user.updateUserPreferences({
-						areKeyboardShortcutsEnabled: !editor.user.getAreKeyboardShortcutsEnabled(),
-					})
-				},
-				checkbox: true,
-			},
 			{
 				id: 'toggle-edge-scrolling',
 				label: {
@@ -1555,7 +1530,6 @@ export function ActionsProvider({ overrides, children }: ActionsProviderProps) {
 				id: 'adjust-shape-styles',
 				label: 'a11y.adjust-shape-styles',
 				kbd: 'cmd+Enter,ctrl+Enter',
-				isRequiredA11yAction: true,
 				onSelect: async (source) => {
 					if (!canApplySelectionAction()) return
 
@@ -1569,7 +1543,6 @@ export function ActionsProvider({ overrides, children }: ActionsProviderProps) {
 			{
 				id: 'a11y-open-context-menu',
 				kbd: 'cmd+shift+Enter,ctrl+shift+Enter',
-				isRequiredA11yAction: true,
 				readonlyOk: true,
 				onSelect: async (source) => {
 					if (!canApplySelectionAction()) return
@@ -1622,7 +1595,6 @@ export function ActionsProvider({ overrides, children }: ActionsProviderProps) {
 				id: 'a11y-repeat-shape-announce',
 				kbd: 'alt+r',
 				label: 'a11y.repeat-shape',
-				isRequiredA11yAction: true,
 				readonlyOk: true,
 				onSelect: async (source) => {
 					const selectedShapeIds = editor.getSelectedShapeIds()

package/src/lib/ui/context/events.tsx

@@ -107,7 +107,6 @@ export interface TLUiEventMap {
 	'toggle-paste-at-cursor': null
 	'toggle-lock': null
 	'toggle-reduce-motion': null
-	'toggle-keyboard-shortcuts': null
 	'toggle-edge-scrolling': null
 	'color-scheme': { value: string }
 	'exit-pen-mode': null
@@ -128,7 +127,6 @@ export interface TLUiEventMap {
 	'copy-link': null
 	'image-replace': null
 	'video-replace': null
-	'open-kbd-shortcuts': null
 	'rich-text': {
 		operation:
 			| 'bold'

package/src/lib/ui/hooks/useClipboardEvents.ts

@@ -352,7 +352,7 @@ async function handleClipboardThings(editor: Editor, things: ClipboardThing[], p
 
 							if (tldrawHtmlComment) {
 								try {
-									// First try parsing as plain JSON (version 2/3 formats)
+									// First try parsing as plain JSON (version 2 format)
 									let json
 									try {
 										json = JSON.parse(tldrawHtmlComment)
@@ -380,32 +380,19 @@ async function handleClipboardThings(editor: Editor, things: ClipboardThing[], p
 									}
 
 									// Handle versioned clipboard format
-									if (json.version === 3) {
-										// Version 3: Assets are plain, decompress only other data
+									if (json.version === 2) {
+										// Version 2: Assets are plain, decompress only other data
 										try {
-											const otherData = JSON.parse(
-												lz.decompressFromBase64(json.data.otherCompressed) || '{}'
-											)
-											const reconstructedData = {
-												assets: json.data.assets || [],
-												...otherData,
-											}
-
-											r({ type: 'tldraw', data: reconstructedData })
+											r({ type: 'tldraw', data: json.data })
 											return
 										} catch (error) {
 											r({
 												type: 'error',
 												data: json,
-												reason: `failed to decompress version 2 clipboard data: ${error}`,
+												reason: `failed to parse version 2 clipboard data: ${error}`,
 											})
 											return
 										}
-									}
-									if (json.version === 2) {
-										// Version 2: Everything is plain, this had issues with encoding... :-/
-										// TODO: nix this support after some time.
-										r({ type: 'tldraw', data: json.data })
 									} else {
 										// Version 1 or no version: Legacy format
 										if (typeof json.data === 'string') {
@@ -597,21 +584,13 @@ const handleNativeOrMenuCopy = async (editor: Editor) => {
 		return
 	}
 
-	// Use versioned clipboard format for better compression
-	// Version 3: Don't compress assets, only compress other data
-	const { assets, ...otherData } = content
-	const clipboardData = {
+	// Version 2: Don't compress anything.
+	const stringifiedClipboard = JSON.stringify({
 		type: 'application/tldraw',
 		kind: 'content',
-		version: 3,
-		data: {
-			assets: assets || [], // Plain JSON, no compression
-			otherCompressed: lz.compressToBase64(JSON.stringify(otherData)), // Only compress non-asset data
-		},
-	}
-
-	// Don't compress the final structure - just use plain JSON
-	const stringifiedClipboard = JSON.stringify(clipboardData)
+		version: 2,
+		data: content,
+	})
 
 	if (typeof navigator === 'undefined') {
 		return

package/src/lib/ui/hooks/useKeyboardShortcuts.ts

@@ -61,7 +61,7 @@ export function useKeyboardShortcuts() {
 			if (SKIP_KBDS.includes(action.id)) continue
 
 			hot(getHotkeysStringFromKbd(action.kbd), (event) => {
-				if (areShortcutsDisabled(editor) && !action.isRequiredA11yAction) return
+				if (areShortcutsDisabled(editor)) return
 				preventDefault(event)
 				action.onSelect('kbd')
 			})
@@ -149,8 +149,7 @@ export function areShortcutsDisabled(editor: Editor) {
 	return (
 		editor.menus.hasAnyOpenMenus() ||
 		editor.getEditingShapeId() !== null ||
-		editor.getCrashingError() ||
-		!editor.user.getAreKeyboardShortcutsEnabled()
+		editor.getCrashingError()
 	)
 }
 

package/src/lib/ui/hooks/useTranslation/TLUiTranslationKey.ts

@@ -55,7 +55,6 @@ export type TLUiTranslationKey =
 	| 'action.new-project'
 	| 'action.new-shared-project'
 	| 'action.open-cursor-chat'
-	| 'action.open-kbd-shortcuts'
 	| 'action.open-file'
 	| 'action.pack'
 	| 'action.paste'
@@ -91,8 +90,6 @@ export type TLUiTranslationKey =
 	| 'action.toggle-wrap-mode'
 	| 'action.toggle-reduce-motion.menu'
 	| 'action.toggle-reduce-motion'
-	| 'action.toggle-keyboard-shortcuts.menu'
-	| 'action.toggle-keyboard-shortcuts'
 	| 'action.toggle-edge-scrolling.menu'
 	| 'action.toggle-edge-scrolling'
 	| 'action.toggle-debug-mode.menu'
@@ -291,7 +288,6 @@ export type TLUiTranslationKey =
 	| 'a11y.pan-camera'
 	| 'a11y.adjust-shape-styles'
 	| 'a11y.open-context-menu'
-	| 'a11y.open-keyboard-shortcuts'
 	| 'menu.title'
 	| 'menu.theme'
 	| 'menu.copy-as'

package/src/lib/ui/hooks/useTranslation/defaultTranslation.ts

@@ -55,7 +55,6 @@ export const DEFAULT_TRANSLATION = {
 	'action.new-project': 'New project',
 	'action.new-shared-project': 'New shared project',
 	'action.open-cursor-chat': 'Cursor chat',
-	'action.open-kbd-shortcuts': 'Open keyboard shortcuts',
 	'action.open-file': 'Open file',
 	'action.pack': 'Pack',
 	'action.paste': 'Paste',
@@ -92,8 +91,6 @@ export const DEFAULT_TRANSLATION = {
 	'action.toggle-wrap-mode': 'Toggle Select on wrap',
 	'action.toggle-reduce-motion.menu': 'Reduce motion',
 	'action.toggle-reduce-motion': 'Toggle reduce motion',
-	'action.toggle-keyboard-shortcuts.menu': 'Keyboard shortcuts',
-	'action.toggle-keyboard-shortcuts': 'Toggle keyboard shortcuts',
 	'action.toggle-edge-scrolling.menu': 'Edge scrolling',
 	'action.toggle-edge-scrolling': 'Toggle edge scrolling',
 	'action.toggle-debug-mode.menu': 'Debug mode',
@@ -292,7 +289,6 @@ export const DEFAULT_TRANSLATION = {
 	'a11y.pan-camera': 'Pan camera',
 	'a11y.adjust-shape-styles': 'Adjust shape styles',
 	'a11y.open-context-menu': 'Open context menu',
-	'a11y.open-keyboard-shortcuts': 'Open keyboard shortcuts',
 	'menu.title': 'Menu',
 	'menu.theme': 'Theme',
 	'menu.copy-as': 'Copy as',

package/src/lib/ui/version.ts

@@ -1,9 +1,9 @@
 // This file is automatically generated by internal/scripts/refresh-assets.ts.
 // Do not edit manually. Or do, I'm a comment, not a cop.
 
-export const version = '3.15.0-next.d30ed5ad740e'
+export const version = '3.15.0-next.f1dfcef63951'
 export const publishDates = {
 	major: '2024-09-13T14:36:29.063Z',
-	minor: '2025-07-10T10:00:49.217Z',
-	patch: '2025-07-10T10:00:49.217Z',
+	minor: '2025-07-03T10:22:18.515Z',
+	patch: '2025-07-03T10:22:18.515Z',
 }

package/src/test/Editor.test.tsx

@@ -49,7 +49,7 @@ beforeEach(() => {
 })
 
 const moveShapesToPage2 = () => {
-	// directly manipulate parentId like would happen in multiplayer situations
+	// directly maniuplate parentId like would happen in multiplayer situations
 
 	editor.updateShapes([
 		{ id: ids.box1, type: 'geo', parentId: ids.page2 },
@@ -899,70 +899,3 @@ describe('the geometry cache', () => {
 		expect(editor.getShapePageBounds(A)!.width).toBe(200)
 	})
 })
-describe('editor.getShapePageBounds', () => {
-	it('calculates axis aligned bounds correctly', () => {
-		editor.createShape({
-			type: 'geo',
-			x: 99,
-			y: 88,
-			props: {
-				w: 199,
-				h: 188,
-			},
-		})
-		const shape = editor.getLastCreatedShape()
-		expect(editor.getShapePageBounds(shape)!).toMatchInlineSnapshot(`
-	Box {
-	  "h": 188,
-	  "w": 199,
-	  "x": 99,
-	  "y": 88,
-	}
-`)
-	})
-
-	it('calculates rotated bounds correctly', () => {
-		editor.createShape({
-			type: 'geo',
-			x: 99,
-			y: 88,
-			rotation: Math.PI / 4,
-			props: {
-				w: 199,
-				h: 188,
-			},
-		})
-		const shape = editor.getLastCreatedShape()
-		expect(editor.getShapePageBounds(shape)!).toMatchInlineSnapshot(`
-	Box {
-	  "h": 273.65032431919394,
-	  "w": 273.6503243191939,
-	  "x": -33.93607486307093,
-	  "y": 88,
-	}
-`)
-	})
-
-	it('calculates bounds based on vertices, not corners', () => {
-		editor.createShape({
-			type: 'geo',
-			x: 99,
-			y: 88,
-			rotation: Math.PI / 4,
-			props: {
-				geo: 'ellipse',
-				w: 199,
-				h: 188,
-			},
-		})
-		const shape = editor.getLastCreatedShape()
-		expect(editor.getShapePageBounds(shape)!).toMatchInlineSnapshot(`
-	Box {
-	  "h": 193.49999999999997,
-	  "w": 193.50000000000003,
-	  "x": 6.139087296526014,
-	  "y": 128.07516215959694,
-	}
-`)
-	})
-})

package/src/test/commands/clipboard.test.ts

@@ -33,7 +33,7 @@ const doMockClipboard = () => {
 		},
 	})
 
-	globalThis.ClipboardItem = jest.fn((payload: any) => payload) as any
+	globalThis.ClipboardItem = jest.fn((payload: any) => payload)
 
 	return context
 }

package/src/test/editor.test.ts

@@ -0,0 +1,77 @@
+import { TestEditor } from './TestEditor'
+
+describe('editor', () => {
+	let editor: TestEditor
+
+	beforeEach(() => {
+		editor = new TestEditor()
+	})
+
+	describe('editor.getShapePageBounds', () => {
+		it('calculates axis aligned bounds correctly', () => {
+			editor.createShape({
+				type: 'geo',
+				x: 99,
+				y: 88,
+				props: {
+					w: 199,
+					h: 188,
+				},
+			})
+			const shape = editor.getLastCreatedShape()
+			expect(editor.getShapePageBounds(shape)!).toMatchInlineSnapshot(`
+		Box {
+		  "h": 188,
+		  "w": 199,
+		  "x": 99,
+		  "y": 88,
+		}
+	`)
+		})
+
+		it('calculates rotated bounds correctly', () => {
+			editor.createShape({
+				type: 'geo',
+				x: 99,
+				y: 88,
+				rotation: Math.PI / 4,
+				props: {
+					w: 199,
+					h: 188,
+				},
+			})
+			const shape = editor.getLastCreatedShape()
+			expect(editor.getShapePageBounds(shape)!).toMatchInlineSnapshot(`
+		Box {
+		  "h": 273.65032431919394,
+		  "w": 273.6503243191939,
+		  "x": -33.93607486307093,
+		  "y": 88,
+		}
+	`)
+		})
+
+		it('calculates bounds based on vertices, not corners', () => {
+			editor.createShape({
+				type: 'geo',
+				x: 99,
+				y: 88,
+				rotation: Math.PI / 4,
+				props: {
+					geo: 'ellipse',
+					w: 199,
+					h: 188,
+				},
+			})
+			const shape = editor.getLastCreatedShape()
+			expect(editor.getShapePageBounds(shape)!).toMatchInlineSnapshot(`
+		Box {
+		  "h": 193.49999999999997,
+		  "w": 193.50000000000003,
+		  "x": 6.139087296526014,
+		  "y": 128.07516215959694,
+		}
+	`)
+		})
+	})
+})