Issue #141: Waybar subscription missed opportunity (#142)

* add notifications number in text field * include a mention about this in the README * fix the linting issue... hopefully * print the count as a uint directly

Issue #141: Waybar subscription missed opportunity (#142)
b455c634 · Carlos Ortega · GitHub · 440ca813 · b455c634 · b455c634
Unverified Commit b455c634 authored Jul 10, 2022 by Carlos Ortega Committed by GitHub Jul 10, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 11 additions and 2 deletions

README.md README.md +9 -0

client.vala src/client.vala +2 -2

No files found.
--- a/README.md
+++ b/README.md
@@ -215,6 +215,15 @@ Waybar css file
 }
 ```

+Alternatively, the number of notifications can be shown by adding `{}` anywhere in the `format` field in the Waybar config
+
+```json
+  "custom/notification": {
+    "format": "{} {icon}",
+    ...
+  },
+```
+
 ## Screenshots

 ![Screenshot of desktop notification](./assets/desktop.png)

--- a/src/client.vala
+++ b/src/client.vala
@@ -88,8 +88,8 @@ private void on_subscribe_waybar (uint count, bool dnd, bool cc_open) {
    }

    print (
-        "{\"text\": \"\", \"alt\": \"%s\", \"tooltip\": \"%s\", \"class\": %s}\n",
-        state, tooltip, _class);
+        "{\"text\": \"%u\", \"alt\": \"%s\", \"tooltip\": \"%s\", \"class\": %s}\n",
+        count, state, tooltip, _class);
 }

 private void print_subscribe_waybar () {