Merge pull request #704 from lukecotter/feat-optimised-timeline-set-default

feat: Enable new timeline by default

Author: lcottercertinia

lana/package.json

@@ -169,14 +169,11 @@
           "description": "Colors used for each event type on the Timeline",
           "additionalProperties": false
         },
-        "lana.timeline.experimental.timeline": {
-          "title": "Enable the new experimental Timeline",
+        "lana.timeline.legacy": {
+          "title": "Enable/ disable the legacy timeline. Default is false.",
           "type": "boolean",
           "default": false,
-          "description": "Enables the faster experimental Timeline",
-          "tags": [
-            "experimental"
-          ]
+          "description": "Enable/ disable the legacy timeline. Default is false."
         }
       }
     }

log-viewer/src/features/settings/Settings.ts

@@ -15,7 +15,7 @@ export type LanaSettings = {
       SOQL: '#5D4963';
       'System Method': '#5C3444';
     };
-    experimental: { timeline: boolean };
+    legacy: boolean;
   };
 };
 

log-viewer/src/features/timeline/components/TimelineView.ts

@@ -25,7 +25,7 @@ export class TimelineView extends LitElement {
   timelineKeys: TimelineGroup[] = [];
 
   @state()
-  private isNewTimelineEnabled: boolean | null = null;
+  private useLegacyTimeline: boolean | null = null;
 
   constructor() {
     super();
@@ -48,14 +48,14 @@ export class TimelineView extends LitElement {
   async connectedCallback() {
     super.connectedCallback();
     const settings = await getSettings();
-    this.isNewTimelineEnabled = settings.timeline.experimental.timeline;
+    this.useLegacyTimeline = settings.timeline.legacy;
   }
 
   render() {
     let timelineBody;
-    if (!this.timelineRoot || this.isNewTimelineEnabled === null) {
+    if (!this.timelineRoot || this.useLegacyTimeline === null) {
       timelineBody = html`<timeline-skeleton></timeline-skeleton>`;
-    } else if (this.isNewTimelineEnabled) {
+    } else if (!this.useLegacyTimeline) {
       timelineBody = html`<timeline-flame-chart
         .apexLog=${this.timelineRoot}
       ></timeline-flame-chart>`;