From 261f7223a9009ae6096aad18722e52c0d1299acb Mon Sep 17 00:00:00 2001
From: Alex Komoroske <jkomoros@gmail.com>
Date: Sat, 30 Mar 2024 16:53:43 -0700
Subject: [PATCH] Flip the naming of TODO enable/disablement to reflect what
 they actually do in the UI.

See #689. Part of #688. Part of #174.
---
 src/components/multi-edit-dialog.ts | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/components/multi-edit-dialog.ts b/src/components/multi-edit-dialog.ts
index 1f0197ec..6fddab11 100644
--- a/src/components/multi-edit-dialog.ts
+++ b/src/components/multi-edit-dialog.ts
@@ -188,6 +188,7 @@ class MultiEditDialog extends connect(store)(DialogElement) {
 		const subtleTags = arrayDiffAsSets(unionTags, intersectionTags)[1];
 
 		//Because TODOs are a weird tri-state, we won't even try to show a intersection/union kind of thing.
+		//And also, yes, the naming of the items feels backwards, see #689.
 
 		return html`
 		<div class='${this._cardModificationPending ? 'modification-pending' : ''}'>
@@ -232,23 +233,23 @@ class MultiEditDialog extends connect(store)(DialogElement) {
 			</tag-list>
 			<label>Enable TODOs</label>
 			<tag-list
-				.tags=${this._todoEnablements}
+				.tags=${this._todoDisablements}
 				.editing=${true}
 				.tapEvents=${true}
 				.tagInfos=${TODO_AUTO_INFOS}
 				.overrideTypeName=${'Enabled'}
-				@tag-added=${this._handleAddTODOEnablement}
-				@tag-removed=${this._handleRemoveTODOEnablement}>
+				@tag-added=${this._handleAddTODODisablement}
+				@tag-removed=${this._handleRemoveTODODisablement}>
 			></tag-list>
 			<label>Disable TODOs</label>
 			<tag-list
-				.tags=${this._todoDisablements}
+				.tags=${this._todoEnablements}
 				.editing=${true}
 				.tapEvents=${true}
 				.tagInfos=${TODO_AUTO_INFOS}
 				.overrideTypeName=${'Disabled'}
-				@tag-added=${this._handleAddTODODisablement}
-				@tag-removed=${this._handleRemoveTODODisablement}>
+				@tag-added=${this._handleAddTODOEnablement}
+				@tag-removed=${this._handleRemoveTODOEnablement}>
 			></tag-list>
 			${Object.values(this._diff).length ? html`<h4>Changes that will be made to selected cards</h4>` : ''}
 			<ul class='readout'>