js-draw 0.11.3 → 0.13.0

package/src/tools/TextTool.ts

@@ -80,11 +80,14 @@ export default class TextTool extends BaseTool {
 		if (this.textInputElem && this.textTargetPosition) {
 			const content = this.textInputElem.value.trimEnd();
 
+			this.textInputElem.value = '';
+
 			if (removeInput) {
-				this.textInputElem.remove();
+				// In some browsers, .remove() triggers a .blur event (synchronously).
+				// Clear this.textInputElem before removal
+				const input = this.textInputElem;
 				this.textInputElem = null;
-			} else {
-				this.textInputElem.value = '';
+				input.remove();
 			}
 
 			if (content === '') {
@@ -100,14 +103,14 @@ export default class TextTool extends BaseTool {
 			).rightMul(
 				Mat33.zRotation(this.textRotation)
 			);
-			
+
 			const textComponent = TextComponent.fromLines(content.split('\n'), textTransform, this.textStyle);
 
 			const action = EditorImage.addElement(textComponent);
 			if (this.removeExistingCommand) {
 				// Unapply so that `removeExistingCommand` can be added to the undo stack.
 				this.removeExistingCommand.unapply(this.editor);
-	
+
 				this.editor.dispatch(uniteCommands([ this.removeExistingCommand, action ]));
 				this.removeExistingCommand = null;
 			} else {
@@ -177,10 +180,13 @@ export default class TextTool extends BaseTool {
 			// Delay removing the input -- flushInput may be called within a blur()
 			// event handler
 			const removeInput = false;
-			this.flushInput(removeInput);
-
 			const input = this.textInputElem;
-			setTimeout(() => input?.remove(), 0);
+
+			this.flushInput(removeInput);
+			this.textInputElem = null;
+			setTimeout(() => {
+				input?.remove();
+			}, 0);
 		};
 		this.textInputElem.onkeyup = (evt) => {
 			if (evt.key === 'Enter' && !evt.shiftKey) {

package/src/tools/ToolSwitcherShortcut.ts

@@ -1,16 +1,21 @@
-// Handles ctrl+1, ctrl+2, ctrl+3, ..., shortcuts for switching tools.
-// @packageDocumentation
-
 import Editor from '../Editor';
 import { KeyPressEvent } from '../types';
 import BaseTool from './BaseTool';
 
-// {@inheritDoc ToolSwitcherShortcut!}
+/**
+ * Handles keyboard events used, by default, to select tools. By default,
+ * 1 maps to the first primary tool, 2 to the second primary tool, ... .
+ *
+ * This is in the default set of {@link ToolController} tools.
+ * 
+ * @deprecated This may be replaced in the future.
+ */
 export default class ToolSwitcherShortcut extends BaseTool {
 	public constructor(private editor: Editor) {
 		super(editor.notifier, editor.localization.changeTool);
 	}
-	
+
+	// @internal
 	public onKeyPress({ key }: KeyPressEvent): boolean {
 		const toolController = this.editor.toolController;
 		const primaryTools = toolController.getPrimaryTools();

package/src/tools/UndoRedoShortcut.ts

@@ -1,17 +1,14 @@
-// Handles ctrl+Z, ctrl+Shift+Z keyboard shortcuts.
-// @packageDocumentation
-
 import Editor from '../Editor';
 import { KeyPressEvent } from '../types';
 import BaseTool from './BaseTool';
 
-// {@inheritDoc UndoRedoShortcut!}
+// Handles ctrl+Z, ctrl+Shift+Z keyboard shortcuts.
 export default class UndoRedoShortcut extends BaseTool {
 	public constructor(private editor: Editor) {
 		super(editor.notifier, editor.localization.undoRedoTool);
 	}
     
-	// Activate undo/redo
+	// @internal
 	public onKeyPress({ key, ctrlKey }: KeyPressEvent): boolean {
 		if (ctrlKey) {
 			if (key === 'z') {

package/src/types.ts

@@ -198,9 +198,9 @@ export type EditorEventDataType = EditorToolEvent | EditorObjectEvent
 // Returns null to continue loading without pause.
 // [totalToProcess] can be an estimate and may change if a better estimate becomes available.
 export type OnProgressListener =
-	(amountProcessed: number, totalToProcess: number)=> Promise<void>|null;
+	(amountProcessed: number, totalToProcess: number)=> Promise<void>|null|void;
 
-export type ComponentAddedListener = (component: AbstractComponent)=> void;
+export type ComponentAddedListener = (component: AbstractComponent)=> Promise<void>|void;
 
 // Called when a new estimate for the import/export rect has been generated. This can be called multiple times.
 // Only the last call to this listener must be accurate.

package/typedoc.json

@@ -1,6 +1,6 @@
 {
 	"entryPoints": [
-		"./src/"
+		"./src/lib.ts", "./src/bundle/bundled.ts"
 	],
 	"exclude": [
 		"**/*.test.ts",
@@ -21,4 +21,4 @@
 		"invalidLink": true,
 		"notDocumented": false
 	}
-}
+}