minor cleanup

src/client/components/boardComponent.tsx

@@ -246,7 +246,7 @@ class BoardComponent extends React.Component<Props, State> {
 		if (boardTree.groupByProperty) {
 			card.properties[boardTree.groupByProperty.id] = groupByValue
 		}
-		await mutator.insertBlock(card, "add card", async () => { await this.setState({shownCard: card}) }, async () => { await this.setState({shownCard: undefined}) })
+		await mutator.insertBlock(card, "add card", async () => { this.setState({shownCard: card}) }, async () => { this.setState({shownCard: undefined}) })
 	}
 
 	async propertyNameChanged(option: IPropertyOption, text: string) {

src/client/components/cardDialog.tsx

@@ -48,12 +48,12 @@ class CardDialog extends React.Component<Props, State> {
 		this.cardListener = new OctoListener()
 		this.cardListener.open(this.props.card.id, async () => {
 			await cardTree.sync()
-			this.setState({cardTree: cardTree})
+			this.setState({ cardTree })
 		})
 		const cardTree = new CardTree(this.props.card.id)
 		cardTree.sync().then(() => {
 			this.setState({cardTree})
-		});
+		})
 
 		document.addEventListener("keydown", this.keydownHandler)
 	}

src/client/filterClause.ts

@@ -13,9 +13,11 @@ class FilterClause {
 			case "notIncludes": return "doesn't include"
 			case "isEmpty": return "is empty"
 			case "isNotEmpty": return "is not empty"
+			default: {
+				Utils.assertFailure()
+				return "(unknown)"
+			}
 		}
-		Utils.assertFailure()
-		return "(unknown)"
 	}
 
 	constructor(o: any = {}) {

src/client/mutator.ts

@@ -14,9 +14,6 @@ import { Utils } from "./utils"
 // It also ensures that the Undo-manager is called for each action
 //
 class Mutator {
-	constructor() {
-	}
-
 	async insertBlock(block: IBlock, description: string = "add", afterRedo?: () => Promise<void>, beforeUndo?: () => Promise<void>) {
 		await undoManager.perform(
 			async () => {
@@ -510,21 +507,21 @@ class Mutator {
 		return block
 	}
 
-    async undo() {
-        await undoManager.undo()
-    }
+	async undo() {
+		await undoManager.undo()
+	}
 
-    undoDescription(): string | undefined {
-        return undoManager.undoDescription
-    }
+	undoDescription(): string | undefined {
+		return undoManager.undoDescription
+	}
 
-    async redo() {
-        await undoManager.redo()
-    }
+	async redo() {
+		await undoManager.redo()
+	}
 
-    redoDescription(): string | undefined {
-        return undoManager.redoDescription
-    }
+	redoDescription(): string | undefined {
+		return undoManager.redoDescription
+	}
 }
 
 const mutator = new Mutator()

src/client/undomanager.ts

@@ -1,7 +1,7 @@
 interface UndoCommand {
 	checkpoint: number
-	undo: () => void
-	redo: () => void
+	undo: () => Promise<void>
+	redo: () => Promise<void>
 	description?: string
 }
 
@@ -54,8 +54,8 @@ class UndoManager {
 	}
 
 	async perform(
-		redo: () => void,
-		undo: () => void,
+		redo: () => Promise<void>,
+		undo: () => Promise<void>,
 		description?: string,
 		isDiscardable = false
 	): Promise<UndoManager> {
@@ -64,7 +64,10 @@ class UndoManager {
 	}
 
 	registerUndo(
-		command: { undo: () => void; redo: () => void },
+		command: {
+			undo: () => Promise<void>,
+			redo: () => Promise<void>
+		},
 		description?: string,
 		isDiscardable = false
 	): UndoManager {

tslint.json

@@ -6,7 +6,7 @@
 		"no-bitwise": true,
 		"no-conditional-assignment": true,
 		"no-consecutive-blank-lines": true,
-		"indent": [true, "tabs"],
+		"indent": [false, "tabs"],
 		"member-access": [true, "no-public"],
 		"semicolon": [true, "never"],
 		"variable-name": [