From 2da2641c2e2ec38e2a655e769306515445a990ea Mon Sep 17 00:00:00 2001
From: Patrick Tasse <patrick.tasse@ericsson.com>
Date: Tue, 26 Jan 2021 17:28:59 -0500
Subject: [PATCH] Make drawing of TimeGraphAxisScale labels optional and fix
 their y-pos

The y-pos of the text label should be just below the tick line height.

Drawing the labels should be optional and not done for a TimeGraphGrid
which only needs to draw tick (grid) lines at full height.

Fixes https://github.com/theia-ide/theia-trace-extension/issues/243
---
 timeline-chart/src/components/time-graph-axis-scale.ts | 8 ++++----
 timeline-chart/src/components/time-graph-grid.ts       | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/timeline-chart/src/components/time-graph-axis-scale.ts b/timeline-chart/src/components/time-graph-axis-scale.ts
index d92a8d7..86a184d 100644
--- a/timeline-chart/src/components/time-graph-axis-scale.ts
+++ b/timeline-chart/src/components/time-graph-axis-scale.ts
@@ -74,7 +74,7 @@ export class TimeGraphAxisScale extends TimeGraphComponent {
         return stepLength;
     }
 
-    protected renderVerticalLines(lineColor: number, lineStyle: (label: string | undefined) => { lineHeight: number }) {
+    protected renderVerticalLines(drawLabels: boolean, lineColor: number, lineStyle: (label: string | undefined) => { lineHeight: number }) {
         if (this.unitController.viewRangeLength > 0) {
             const stepLength = this.getStepLength();
             const canvasDisplayWidth = this.stateController.canvasDisplayWidth;
@@ -91,7 +91,7 @@ export class TimeGraphAxisScale extends TimeGraphComponent {
                         y: this._options.position.y
                     };
                     let label;
-                    if (this.unitController.numberTranslator) {
+                    if (drawLabels && this.unitController.numberTranslator) {
                         label = this.unitController.numberTranslator(absolutePosition);
                         if (label) {
                             const text = new PIXI.Text(label, {
@@ -99,7 +99,7 @@ export class TimeGraphAxisScale extends TimeGraphComponent {
                                 fill: lineColor
                             });
                             text.x = position.x + 5;
-                            text.y = this._options.height - (2 * lineStyle(label).lineHeight);
+                            text.y = position.y + lineStyle(label).lineHeight;
                             this.labels.push(text);
                             this._displayObject.addChild(text);
                         }
@@ -127,7 +127,7 @@ export class TimeGraphAxisScale extends TimeGraphComponent {
             width: this._options.width,
             position: this._options.position
         });
-        this.renderVerticalLines(this._options.lineColor || 0x000000, (l) => ({ lineHeight: l === '' || l === undefined ? 5 : 10 }));
+        this.renderVerticalLines(true, this._options.lineColor || 0x000000, (l) => ({ lineHeight: l === '' || l === undefined ? 5 : 10 }));
     }
 
     zoomAroundLeftViewBorder(zoomStep: number) {
diff --git a/timeline-chart/src/components/time-graph-grid.ts b/timeline-chart/src/components/time-graph-grid.ts
index 00e0678..06b7cb7 100644
--- a/timeline-chart/src/components/time-graph-grid.ts
+++ b/timeline-chart/src/components/time-graph-grid.ts
@@ -20,6 +20,6 @@ export class TimeGraphGrid extends TimeGraphAxisScale {
     protected addEvents() { }
 
     render(): void {
-        this.renderVerticalLines(this._options.lineColor || 0xdddddd, () => ({ lineHeight: this.stateController.canvasDisplayHeight }));
+        this.renderVerticalLines(false, this._options.lineColor || 0xdddddd, () => ({ lineHeight: this.stateController.canvasDisplayHeight }));
     }
 }
\ No newline at end of file