From 01aee58c4f0b5b1e940d38a86fc03e84ef673285 Mon Sep 17 00:00:00 2001
From: andrewlalis <andrewlalisofficial@gmail.com>
Date: Mon, 1 Sep 2025 16:34:39 -0400
Subject: [PATCH] Clean up comment editing.

---
 .../classroom_compliance/entries_table/EntryTableCell.vue    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/app/src/apps/classroom_compliance/entries_table/EntryTableCell.vue b/app/src/apps/classroom_compliance/entries_table/EntryTableCell.vue
index b333359..02ccc1e 100644
--- a/app/src/apps/classroom_compliance/entries_table/EntryTableCell.vue
+++ b/app/src/apps/classroom_compliance/entries_table/EntryTableCell.vue
@@ -37,6 +37,7 @@ const entryChanged = computed(() => JSON.stringify(model.value) !== initialEntry
 const hasComment = computed(() => model.value && (model.value.comment.trim().length > 0 || model.value.checklistItems.length > 0))
 
 const previousCommentValue: Ref<string> = ref('')
+const previousCommentChecklistItems: Ref<string[]> = ref([])
 const commentEditorDialog = useTemplateRef('commentEditorDialog')
 
 onMounted(() => {
@@ -104,12 +105,14 @@ function toggleBehaviorRating() {
 function showCommentEditor() {
   if (!model.value) return
   previousCommentValue.value = model.value?.comment
+  previousCommentChecklistItems.value = [...model.value?.checklistItems]
   commentEditorDialog.value?.showModal()
 }
 
 function cancelCommentEdit() {
   if (model.value) {
     model.value.comment = previousCommentValue.value
+    model.value.checklistItems = previousCommentChecklistItems.value
   }
   commentEditorDialog.value?.close()
 }
@@ -182,7 +185,7 @@ function addEntry() {
         <div v-for="options, category in COMMENT_CHECKLIST_ITEMS" :key="category">
           <h3 v-text="category"></h3>
           <label v-for="opt in options" :key="opt">
-            <input type="checkbox" v-model="model.checklistItems" :value="opt" />
+            <input type="checkbox" v-model="model.checklistItems" :value="opt" :disabled="disabled" />
             <span v-text="opt"></span>
           </label>
         </div>