Headings are flash state.

Author: microbit-matt-hillsdon

src/board/data-logging.ts

@@ -34,22 +34,6 @@ export class DataLogging {
     private onChange: (state: Partial<State>) => void
   ) {}
 
-  delete() {
-    this.resetNonFlashStateExceptTimestamp();
-    this.size = 0;
-    if (this.logFull) {
-      this.logFull = false;
-      this.onChange({
-        dataLogging: {
-          type: "dataLogging",
-          logFull: false,
-        },
-      });
-    }
-
-    this.onLogDelete();
-  }
-
   setMirroring(mirroring: boolean) {
     this.mirroring = mirroring;
   }
@@ -151,9 +135,26 @@ export class DataLogging {
     this.timestamp = MICROBIT_HAL_LOG_TIMESTAMP_NONE;
   }
 
+  delete() {
+    this.resetNonFlashStateExceptTimestamp();
+    this.headings = [];
+    this.size = 0;
+    if (this.logFull) {
+      this.logFull = false;
+      this.onChange({
+        dataLogging: {
+          type: "dataLogging",
+          logFull: false,
+        },
+      });
+    }
+
+    this.onLogDelete();
+  }
+
   private resetNonFlashStateExceptTimestamp() {
+    // headings are considered flash state as MicroBitLog reparses them from flash
     this.mirroring = false;
-    this.headings = [];
     this.headingsChanged = false;
     this.timestampOnLastEndRow = undefined;
     this.row = undefined;