fix: only show non 0 numbers in change status bar

Author: Vinzent03

src/editor/signs/changesStatusBar.ts

@@ -11,6 +11,9 @@ export class ChangesStatusBar {
         if (plugin.settings.hunks.statusBar === "colored") {
             statusBarEl.addClass("git-changes-status-bar-colored");
         }
+
+        statusBarEl.setAttr("aria-label", "Git diff of the current editor");
+        this.statusBarEl.setAttribute("data-tooltip-position", "top");
         plugin.app.workspace.on("active-leaf-change", (leaf) => {
             if (
                 !leaf ||
@@ -23,41 +26,39 @@ export class ChangesStatusBar {
     }
 
     display(hunks: Hunk[], file: TFile | null): void {
-        if (this.plugin.gitReady) {
-            const mdView =
-                this.plugin.app.workspace.getActiveViewOfType(MarkdownView);
-            if (!mdView || mdView.file?.path !== file?.path) {
-                return;
-            }
+        const mdView =
+            this.plugin.app.workspace.getActiveViewOfType(MarkdownView);
+        if (!mdView || mdView.file?.path !== file?.path) {
+            return;
+        }
 
-            let added: number = 0,
-                changed: number = 0,
-                deleted: number = 0;
-            for (const hunk of hunks) {
-                added += Math.max(0, hunk.added.count - hunk.removed.count);
-                changed += Math.min(hunk.added.count, hunk.removed.count);
-                deleted += Math.max(0, hunk.removed.count - hunk.added.count);
-            }
-            // this.statusBarEl.setText(`+${added} ~${changed} -${deleted}`);
-            this.statusBarEl.empty();
+        let added: number = 0,
+            changed: number = 0,
+            deleted: number = 0;
+        for (const hunk of hunks) {
+            added += Math.max(0, hunk.added.count - hunk.removed.count);
+            changed += Math.min(hunk.added.count, hunk.removed.count);
+            deleted += Math.max(0, hunk.removed.count - hunk.added.count);
+        }
+        this.statusBarEl.empty();
+        if (added > 0) {
             this.statusBarEl.createSpan({
                 text: `+${added} `,
                 cls: "git-add",
             });
+        }
+        if (changed > 0) {
             this.statusBarEl.createSpan({
                 text: `~${changed} `,
                 cls: "git-change",
             });
+        }
+        if (deleted > 0) {
             this.statusBarEl.createSpan({
                 text: `-${deleted}`,
                 cls: "git-delete",
             });
-        } else {
-            this.statusBarEl.empty();
         }
-        // } else {
-        //     this.statusBarEl.empty();
-        // }
     }
 
     remove() {

src/editor/signs/signsProvider.ts

@@ -1,5 +1,4 @@
 import type { Extension } from "@codemirror/state";
-import { Prec } from "@codemirror/state";
 import type { TFile } from "obsidian";
 import { eventsPerFilePathSingleton } from "src/editor/eventsPerFilepath";
 import type ObsidianGit from "src/main";

styles.css

@@ -673,10 +673,6 @@ div:hover > .git-gutter-marker.git-changedelete {
     }
 }
 
-/* .git-changes-status-bar > * { */
-/*     margin-right: 4px; */
-/* } */
-
 .git-changes-status-bar-colored {
     .git-add {
         color: var(--color-green);
@@ -689,6 +685,10 @@ div:hover > .git-gutter-marker.git-changedelete {
     }
 }
 
+.git-changes-status-bar .git-add {
+    margin-right: 0.3em;
+}
+
 .git-changes-status-bar .git-change {
-    margin: 0 4px;
+    margin-right: 0.3em;
 }