fix history panel not rendering when history is empty (#1215)

catch error to handle the log return when there are no past commits in the current branch

prettify and format correctly

change handled return to simply a non-null, non-zero code rather than 0 with an empty list of commits

Rebases to include changes from #13645

Update Playwright Snapshots

Updates selection to selectionBackground, docs

Update Playwright Snapshots

Add renderer

Fix UI test locator

Fix linter

Update Playwright Snapshots

Buffer message while waiting for the terminal

Update Playwright Snapshots

Co-authored-by: Shawn Esquivel <[email protected]>

Author: basokant

src/components/GitCommitGraph.tsx

@@ -95,12 +95,19 @@ export class GitCommitGraph extends React.Component<IGitCommitGraphProps> {
   }
 
   getHeight(): number {
-    return (
-      this._graphData[this._graphData.length - 1].yOffset +
-      this.props.getNodeHeight(
-        this.props.commits[this.props.commits.length - 1].sha
-      )
-    );
+    // if this.props.commits.length = 0, just pass 0
+    const numCommits = this.props.commits.length;
+    const lastNodeHeight =
+      numCommits === 0
+        ? 0
+        : this.props.getNodeHeight(this.props.commits[numCommits - 1].sha);
+
+    const numGraphNodes = this._graphData.length;
+    const lastYOffset =
+      numGraphNodes === 0
+        ? 0
+        : this._graphData[this._graphData.length - 1].yOffset;
+    return lastYOffset + lastNodeHeight;
   }
 
   renderRouteNode(svgPathDataAttribute: string, branch: number): JSX.Element {

src/model.ts

@@ -919,14 +919,18 @@ export class GitExtension implements IGitExtension {
     return await this._taskHandler.execute<Git.ILogResult>(
       'git:fetch:log',
       async () => {
-        return await requestAPI<Git.ILogResult>(
-          URLExt.join(path, 'log'),
-          'POST',
-          {
-            history_count: count,
-            follow_path: this.selectedHistoryFile?.to
-          }
-        );
+        try {
+          return await requestAPI<Git.ILogResult>(
+            URLExt.join(path, 'log'),
+            'POST',
+            {
+              history_count: count,
+              follow_path: this.selectedHistoryFile?.to
+            }
+          );
+        } catch (error) {
+          return { code: 1 };
+        }
       }
     );
   }