chore: v0.2.1

Module network ipaddr

.gitignore

@@ -4,7 +4,7 @@ vgcore.*
 /.vscode
 *.swp
 packagecache
-/subprojects/fmt-4.1.0
+/subprojects/**/
 /build
 /dist
 /meson.egg-info

.travis.yml

@@ -0,0 +1,17 @@
+sudo: false
+
+services:
+    - docker
+
+env:
+    - distro: debian
+    - distro: archlinux
+
+before_install:
+    - docker pull alexays/waybar:${distro}
+
+script:
+    - echo FROM alexays/waybar:${distro} > Dockerfile
+    - echo ADD . /root >> Dockerfile
+    - docker build -t waybar .
+    - docker run waybar /bin/sh -c "cd /root && git clone https://github.com/swaywm/wlroots subprojects/wlroots && meson build && ninja -C build"

Dockerfiles/archlinux

@@ -0,0 +1,4 @@
+FROM archlinux/base:latest
+
+RUN pacman -Syu --noconfirm && \
+    pacman -S git meson base-devel libinput wayland wayland-protocols pixman libxkbcommon mesa gtkmm3 jsoncpp --noconfirm

Dockerfiles/debian

@@ -0,0 +1,5 @@
+FROM debian:sid
+
+RUN apt-get update && \
+    apt-get install -y build-essential meson ninja-build git pkg-config libinput10 libinput-dev wayland-protocols libwayland-client0 libwayland-cursor0 libwayland-dev libegl1-mesa-dev libgles2-mesa-dev libgbm-dev libxkbcommon-dev libudev-dev libpixman-1-dev libgtkmm-3.0-dev libjsoncpp-dev && \
+    apt-get clean

README.md

@@ -1,11 +1,12 @@
-# Waybar [![Licence](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)<br>![Waybar](https://raw.githubusercontent.com/alexays/waybar/master/preview-2.png)
+# Waybar [![Travis](https://travis-ci.org/Alexays/Waybar.svg?branch=master)](https://travis-ci.org/Alexays/Waybar) [![Licence](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)<br>![Waybar](https://raw.githubusercontent.com/alexays/waybar/master/preview-2.png)
 **Proof of concept**
 
 > Highly customizable Wayland bar for Sway and Wlroots based compositors.<br>
 > Available on [AUR](https://aur.archlinux.org/packages/waybar-git/)
 
 **Current features**
-- Sway Workspaces
+- Sway (Workspaces, Binding mode, Focused window name)
+- Tray (Beta) [#21](https://github.com/Alexays/Waybar/issues/21)
 - Local time
 - Battery
 - Network
@@ -13,6 +14,7 @@
 - Memory
 - Cpu load average
 - Custom scripts
+- And much more customizations
 
 **Configuration and Customization**
 
@@ -22,12 +24,14 @@
 
 ```bash
 $ git clone https://github.com/Alexays/Waybar
+$ cd Waybar
 $ meson build
 $ ninja -C build
 $ ./build/waybar
 ```
 
-Contributions welcome! - have fun :)
+Contributions welcome! - have fun :)<br>
+The style guidelines is [Google's](https://google.github.io/styleguide/cppguide.html)
 
 ## License
 

include/ALabel.hpp

@@ -0,0 +1,30 @@
+#pragma once
+
+#include <json/json.h>
+#include "IModule.hpp"
+
+namespace waybar {
+
+class ALabel : public IModule {
+ public:
+  ALabel(const Json::Value&, const std::string format);
+  virtual ~ALabel() = default;
+  virtual auto update() -> void;
+  virtual std::string getIcon(uint16_t, const std::string& alt = "");
+  virtual operator Gtk::Widget&();
+
+ protected:
+  Gtk::EventBox event_box_;
+  Gtk::Label label_;
+  const Json::Value& config_;
+  std::string format_;
+  std::mutex mutex_;
+
+ private:
+  bool handleToggle(GdkEventButton* const& ev);
+  bool handleScroll(GdkEventScroll*);
+  bool alt = false;
+  const std::string default_format_;
+};
+
+}  // namespace waybar

include/IModule.hpp

@@ -1,10 +1,15 @@
 #pragma once
 
+#include <gtkmm.h>
+
 namespace waybar {
-  class IModule {
-	  public:
-      virtual ~IModule() {}
-      virtual auto update() -> void = 0;
-      virtual operator Gtk::Widget &() = 0;
-  };
+
+class IModule {
+  public:
+    virtual ~IModule() = default;
+    virtual auto update() -> void = 0;
+    virtual operator Gtk::Widget &() = 0;
+    Glib::Dispatcher dp; // Hmmm Maybe I should create an abstract class ?
+};
+
 }

include/bar.hpp

@@ -3,43 +3,57 @@
 #include <json/json.h>
 #include <gtkmm.h>
 #include "wlr-layer-shell-unstable-v1-client-protocol.h"
+#include "xdg-output-unstable-v1-client-protocol.h"
+#include "IModule.hpp"
 
 namespace waybar {
 
-  struct Client;
+class Client;
+class Factory;
 
-  struct Bar {
-    Bar(Client& client, std::unique_ptr<struct wl_output *>&& output);
+class Bar {
+  public:
+    Bar(const Client&, std::unique_ptr<struct wl_output *>&&, uint32_t);
     Bar(const Bar&) = delete;
-    Client& client;
+
+    auto toggle() -> void;
+
+    const Client& client;
     Gtk::Window window;
     struct wl_surface *surface;
-    struct zwlr_layer_surface_v1 *layerSurface;
+    struct zwlr_layer_surface_v1 *layer_surface;
     std::unique_ptr<struct wl_output *> output;
+    std::string output_name;
+    uint32_t wl_name;
     bool visible = true;
-    auto setWidth(int) -> void;
-    auto toggle() -> void;
   private:
-    static void _handleGeometry(void *data, struct wl_output *wl_output,
-      int32_t x, int32_t y, int32_t physical_width, int32_t physical_height,
-      int32_t subpixel, const char *make, const char *model, int32_t transform);
-    static void _handleMode(void *data, struct wl_output *wl_output,
-      uint32_t f, int32_t w, int32_t h, int32_t refresh);
-    static void _handleDone(void *data, struct wl_output *);
-    static void _handleScale(void *data, struct wl_output *wl_output,
-      int32_t factor);
-    static void _layerSurfaceHandleConfigure(void *data,
-      struct zwlr_layer_surface_v1 *surface, uint32_t serial, uint32_t width,
-      uint32_t height);
-    static void _layerSurfaceHandleClosed(void *data,
-      struct zwlr_layer_surface_v1 *surface);
-    auto _setupConfig() -> void;
-    auto _setupWidgets() -> void;
-    auto _setupCss() -> void;
-    int _width = 10;
-    Json::Value _config;
-    Glib::RefPtr<Gtk::StyleContext> _styleContext;
-    Glib::RefPtr<Gtk::CssProvider> _cssProvider;
-  };
+    static void handleLogicalPosition(void *, struct zxdg_output_v1 *, int32_t,
+      int32_t);
+    static void handleLogicalSize(void *, struct zxdg_output_v1 *, int32_t,
+      int32_t);
+    static void handleDone(void *, struct zxdg_output_v1 *);
+    static void handleName(void *, struct zxdg_output_v1 *, const char *);
+    static void handleDescription(void *, struct zxdg_output_v1 *,
+      const char *);
+    static void layerSurfaceHandleConfigure(void *,
+      struct zwlr_layer_surface_v1 *, uint32_t, uint32_t, uint32_t);
+    static void layerSurfaceHandleClosed(void *,
+      struct zwlr_layer_surface_v1 *);
+
+    auto setupConfig() -> void;
+    auto setupWidgets() -> void;
+    auto setupCss() -> void;
+    void getModules(const Factory&, const std::string&);
+
+    uint32_t width_ = 0;
+    uint32_t height_ = 30;
+    Json::Value config_;
+    Glib::RefPtr<Gtk::StyleContext> style_context_;
+    Glib::RefPtr<Gtk::CssProvider> css_provider_;
+    struct zxdg_output_v1 *xdg_output_;
+    std::vector<std::unique_ptr<waybar::IModule>> modules_left_;
+    std::vector<std::unique_ptr<waybar::IModule>> modules_center_;
+    std::vector<std::unique_ptr<waybar::IModule>> modules_right_;
+};
 
 }

include/client.hpp

@@ -2,49 +2,38 @@
 
 #include <unistd.h>
 #include <wordexp.h>
-
-#include <iostream>
-
 #include <fmt/format.h>
-
 #include <gdk/gdk.h>
-#include <gtkmm.h>
 #include <wayland-client.h>
-
-#include "wlr-layer-shell-unstable-v1-client-protocol.h"
-#include "idle-client-protocol.h"
-
-#include "util/ptr_vec.hpp"
-
 #include <gdk/gdkwayland.h>
-
 #include "bar.hpp"
 
 namespace waybar {
 
-  struct Client {
-    uint32_t height = 30;
-    std::string cssFile;
-    std::string configFile;
-
-    Gtk::Main gtk_main;
+class Client {
+  public:
+    Client(int argc, char *argv[]);
+    int main(int argc, char *argv[]);
 
+    Glib::RefPtr<Gtk::Application> gtk_app;
+    std::string css_file;
+    std::string config_file;
     Glib::RefPtr<Gdk::Display> gdk_display;
-    struct wl_display *wlDisplay;
-    struct wl_registry *registry;
-    struct zwlr_layer_shell_v1 *layer_shell;
-    struct org_kde_kwin_idle *idle_manager;
-    struct wl_seat *seat;
-    util::ptr_vec<Bar> bars;
+    struct wl_display *wl_display = nullptr;
+    struct wl_registry *registry = nullptr;
+    struct zwlr_layer_shell_v1 *layer_shell = nullptr;
+    struct zxdg_output_manager_v1 *xdg_output_manager = nullptr;
+    struct wl_seat *seat = nullptr;
+    std::vector<std::unique_ptr<Bar>> bars;
 
-    Client(int argc, char* argv[]);
-    void bind_interfaces();
-    auto setup_css();
-    int main(int argc, char* argv[]);
   private:
-    static void _handle_global(void *data, struct wl_registry *registry,
+    void bindInterfaces();
+    auto setupCss();
+
+    static void handleGlobal(void *data, struct wl_registry *registry,
       uint32_t name, const char *interface, uint32_t version);
-    static void _handle_global_remove(void *data,
+    static void handleGlobalRemove(void *data,
       struct wl_registry *registry, uint32_t name);
-  };
+};
+
 }

include/factory.hpp

@@ -2,23 +2,36 @@
 
 #include <json/json.h>
 #include "modules/clock.hpp"
-#include "modules/workspaces.hpp"
+#ifdef HAVE_SWAY
+#include "modules/sway/mode.hpp"
+#include "modules/sway/workspaces.hpp"
+#include "modules/sway/window.hpp"
+#endif
 #include "modules/battery.hpp"
 #include "modules/memory.hpp"
 #include "modules/cpu.hpp"
+#ifdef HAVE_DBUSMENU
+#include "modules/sni/tray.hpp"
+#endif
+#ifdef HAVE_LIBNL
 #include "modules/network.hpp"
+#endif
+#ifdef HAVE_LIBPULSE
 #include "modules/pulseaudio.hpp"
+#endif
 #include "modules/custom.hpp"
 
 namespace waybar {
 
-  class Factory {
-	  public:
-      Factory(Bar &bar, Json::Value config);
-      IModule &makeModule(std::string name);
-    private:
-      Bar &_bar;
-      Json::Value _config;
-  };
+class Bar;
+
+class Factory {
+  public:
+    Factory(Bar& bar, const Json::Value& config);
+    IModule* makeModule(const std::string &name) const;
+  private:
+    Bar& bar_;
+    const Json::Value& config_;
+};
 
 }

include/ipc/client.hpp

@@ -1,32 +0,0 @@
-#pragma once
-
-#include <iostream>
-#include "ipc.hpp"
-
-/**
- * IPC response including type of IPC response, size of payload and the json
- * encoded payload string.
- */
-struct ipc_response {
-	uint32_t size;
-	uint32_t type;
-	std::string payload;
-};
-
-/**
- * Gets the path to the IPC socket from sway.
- */
-std::string get_socketpath(void);
-/**
- * Opens the sway socket.
- */
-int ipc_open_socket(std::string socket_path);
-/**
- * Issues a single IPC command and returns the buffer. len will be updated with
- * the length of the buffer returned from sway.
- */
-std::string ipc_single_command(int socketfd, uint32_t type, const char *payload, uint32_t *len);
-/**
- * Receives a single IPC response and returns an ipc_response.
- */
-struct ipc_response ipc_recv_response(int socketfd);

include/modules/battery.hpp

@@ -1,29 +1,43 @@
 #pragma once
 
-#include <json/json.h>
-#include <filesystem>
+#ifdef FILESYSTEM_EXPERIMENTAL
+  #include <experimental/filesystem>
+#else
+  #include <filesystem>
+#endif
 #include <fstream>
-#include <gtkmm.h>
 #include <iostream>
 #include <fmt/format.h>
+#include <sys/inotify.h>
+#include <algorithm>
 #include "util/chrono.hpp"
-#include "IModule.hpp"
+#include "ALabel.hpp"
 
 namespace waybar::modules {
 
-  namespace fs = std::filesystem;
+#ifdef FILESYSTEM_EXPERIMENTAL
+namespace fs = std::experimental::filesystem;
+#else
+namespace fs = std::filesystem;
+#endif
 
-  class Battery : public IModule {
-    public:
-      Battery(Json::Value config);
-      auto update() -> void;
-      operator Gtk::Widget&();
-    private:
-      static inline const fs::path _data_dir = "/sys/class/power_supply/";
-      std::vector<fs::path> _batteries;
-      util::SleeperThread _thread;
-      Gtk::Label _label;
-      Json::Value _config;
-  };
+class Battery : public ALabel {
+  public:
+    Battery(const Json::Value&);
+    ~Battery();
+    auto update() -> void;
+  private:
+    static inline const fs::path data_dir_ = "/sys/class/power_supply/";
+  
+    void worker();
+    std::tuple<uint16_t, std::string> getInfos();
+    std::string getState(uint16_t);
+
+    util::SleeperThread thread_;
+    util::SleeperThread thread_timer_;
+    std::vector<fs::path> batteries_;
+    int fd_;
+    std::string old_status_;
+};
 
 }

include/modules/clock.hpp

@@ -1,23 +1,18 @@
 #pragma once
 
-#include <json/json.h>
-#include <gtkmm.h>
 #include <fmt/format.h>
-#include <thread>
+#include "fmt/time.h"
 #include "util/chrono.hpp"
-#include "IModule.hpp"
+#include "ALabel.hpp"
 
 namespace waybar::modules {
 
-  class Clock : public IModule {
-    public:
-      Clock(Json::Value config);
-      auto update() -> void;
-      operator Gtk::Widget &();
-    private:
-      Gtk::Label _label;
-      waybar::util::SleeperThread _thread;
-      Json::Value _config;
-  };
+class Clock : public ALabel {
+  public:
+    Clock(const Json::Value&);
+    auto update() -> void;
+  private:
+    waybar::util::SleeperThread thread_;
+};
 
 }

include/modules/cpu.hpp

@@ -1,24 +1,28 @@
 #pragma once
 
-#include <json/json.h>
-#include <gtkmm.h>
 #include <fmt/format.h>
 #include <sys/sysinfo.h>
-#include <thread>
+#include <fstream>
+#include <vector>
+#include <numeric>
+#include <iostream>
 #include "util/chrono.hpp"
-#include "IModule.hpp"
+#include "ALabel.hpp"
 
 namespace waybar::modules {
 
-  class Cpu : public IModule {
-    public:
-      Cpu(Json::Value config);
-      auto update() -> void;
-      operator Gtk::Widget &();
-    private:
-      Gtk::Label _label;
-      waybar::util::SleeperThread _thread;
-      Json::Value _config;
-  };
+class Cpu : public ALabel {
+  public:
+    Cpu(const Json::Value&);
+    auto update() -> void;
+  private:
+    static inline const std::string data_dir_ = "/proc/stat";
+    uint16_t getCpuLoad();
+    std::tuple<uint16_t, std::string> getCpuUsage();
+    std::vector<std::tuple<size_t, size_t>> parseCpuinfo();
+
+    std::vector<std::tuple<size_t, size_t>> prev_times_;
+    waybar::util::SleeperThread thread_;
+};
 
 }

include/modules/custom.hpp

@@ -1,24 +1,32 @@
 #pragma once
 
-#include <json/json.h>
-#include <gtkmm.h>
 #include <fmt/format.h>
-#include <thread>
+#include <iostream>
 #include "util/chrono.hpp"
-#include "IModule.hpp"
+#include "util/command.hpp"
+#include "util/json.hpp"
+#include "ALabel.hpp"
 
 namespace waybar::modules {
 
-  class Custom : public IModule {
-    public:
-      Custom(std::string name, Json::Value config);
-      auto update() -> void;
-      operator Gtk::Widget &();
-    private:
-      Gtk::Label _label;
-      waybar::util::SleeperThread _thread;
-      const std::string _name;
-      Json::Value _config;
-  };
+class Custom : public ALabel {
+  public:
+    Custom(const std::string, const Json::Value&);
+    auto update() -> void;
+  private:
+    void delayWorker();
+    void continuousWorker();
+    void parseOutputRaw();
+    void parseOutputJson();
+
+    const std::string name_;
+    std::string text_;
+    std::string tooltip_;
+    std::string class_;
+    std::string prevclass_;
+    waybar::util::SleeperThread thread_;
+    waybar::util::command::res output_;
+    waybar::util::JsonParser parser_;
+};
 
 }

include/modules/memory.hpp

@@ -1,24 +1,22 @@
 #pragma once
 
-#include <json/json.h>
-#include <gtkmm.h>
 #include <fmt/format.h>
-#include <sys/sysinfo.h>
-#include <thread>
+#include <fstream>
 #include "util/chrono.hpp"
-#include "IModule.hpp"
+#include "ALabel.hpp"
 
 namespace waybar::modules {
 
-  class Memory : public IModule {
-    public:
-      Memory(Json::Value config);
-      auto update() -> void;
-      operator Gtk::Widget &();
-    private:
-      Gtk::Label _label;
-      waybar::util::SleeperThread _thread;
-      Json::Value _config;
-  };
+class Memory : public ALabel {
+  public:
+    Memory(const Json::Value&);
+    auto update() -> void;
+  private:
+    static inline const std::string data_dir_ = "/proc/meminfo";
+    unsigned long memtotal_;
+    unsigned long memfree_;
+    void parseMeminfo();
+    waybar::util::SleeperThread thread_;
+};
 
 }

include/modules/network.hpp

@@ -1,38 +1,54 @@
 #pragma once
 
 #include <net/if.h>
+#include <arpa/inet.h>
+#include <ifaddrs.h>
 #include <netlink/netlink.h>
 #include <netlink/genl/genl.h>
 #include <netlink/genl/ctrl.h>
 #include <linux/nl80211.h>
-#include <iwlib.h> // TODO
-#include <json/json.h>
-#include <gtkmm.h>
 #include <fmt/format.h>
-#include <thread>
 #include "util/chrono.hpp"
-#include "IModule.hpp"
+#include "ALabel.hpp"
 
 namespace waybar::modules {
 
-  class Network : public IModule {
-    public:
-      Network(Json::Value config);
-      auto update() -> void;
-      operator Gtk::Widget &();
-    private:
-      void _parseEssid(struct nlattr **bss);
-      void _parseSignal(struct nlattr **bss);
-      bool _associatedOrJoined(struct nlattr **bss);
-      static int _scanCb(struct nl_msg *msg, void *data);
-      auto _getInfo() -> void;
-      Gtk::Label _label;
-      waybar::util::SleeperThread _thread;
-      Json::Value _config;
-      std::size_t _ifid;
-      std::string _essid;
-      int _signalStrengthdBm;
-      int _signalStrength;
-  };
+class Network : public ALabel {
+  public:
+    Network(const Json::Value&);
+    ~Network();
+    auto update() -> void;
+  private:
+    static int netlinkRequest(int, void*, uint32_t, uint32_t groups = 0);
+    static int netlinkResponse(int, void*, uint32_t, uint32_t groups = 0);
+    static int scanCb(struct nl_msg*, void*);
+
+    void worker();
+    void disconnected();
+    void initNL80211();
+    int getExternalInterface();
+    void getInterfaceAddress();
+    void parseEssid(struct nlattr**);
+    void parseSignal(struct nlattr**);
+    bool associatedOrJoined(struct nlattr**);
+    auto getInfo() -> void;
+
+    waybar::util::SleeperThread thread_;
+    waybar::util::SleeperThread thread_timer_;
+    int ifid_;
+    sa_family_t family_;
+    int sock_fd_;
+    struct sockaddr_nl nladdr_ = {0};
+    struct nl_sock* sk_ = nullptr;
+    int nl80211_id_;
+
+    std::string essid_;
+    std::string ifname_;
+    std::string ipaddr_;
+    std::string netmask_;
+    int cidr_;
+    int signal_strength_dbm_;
+    uint16_t signal_strength_;
+};
 
 }

include/modules/pulseaudio.hpp

@@ -1,35 +1,39 @@
 #pragma once
 
-#include <pulse/pulseaudio.h>
-#include <json/json.h>
-#include <gtkmm.h>
 #include <fmt/format.h>
-#include "IModule.hpp"
+#include <pulse/pulseaudio.h>
+#include <pulse/volume.h>
+#include <algorithm>
+#include "ALabel.hpp"
 
 namespace waybar::modules {
 
-  class Pulseaudio : public IModule {
-    public:
-      Pulseaudio(Json::Value config);
-      auto update() -> void;
-      operator Gtk::Widget &();
-    private:
-      static void _subscribeCb(pa_context *context,
-        pa_subscription_event_type_t type, uint32_t idx, void *data);
-      static void _contextStateCb(pa_context *c, void *data);
-      static void _sinkInfoCb(pa_context *context, const pa_sink_info *i,
-        int eol, void *data);
-      static void _serverInfoCb(pa_context *context, const pa_server_info *i,
-        void *data);
-      Gtk::Label _label;
-      Json::Value _config;
-      pa_threaded_mainloop *_mainloop;
-      pa_mainloop_api *_mainloop_api;
-      pa_context *_context;
-      uint32_t _sinkIdx{0};
-      int _volume;
-      bool _muted;
-      std::string _desc;
-  };
+class Pulseaudio : public ALabel {
+  public:
+    Pulseaudio(const Json::Value&);
+    ~Pulseaudio();
+    auto update() -> void;
+  private:
+    static void subscribeCb(pa_context*, pa_subscription_event_type_t,
+      uint32_t, void*);
+    static void contextStateCb(pa_context*, void*);
+    static void sinkInfoCb(pa_context*, const pa_sink_info*, int, void*);
+    static void serverInfoCb(pa_context*, const pa_server_info*, void*);
+    static void volumeModifyCb(pa_context*, int, void*);
+    bool handleScroll(GdkEventScroll* e);
 
-}
+    const std::string getPortIcon() const;
+
+    pa_threaded_mainloop* mainloop_;
+    pa_mainloop_api* mainloop_api_;
+    pa_context* context_;
+    uint32_t sink_idx_{0};
+    uint16_t volume_;
+    pa_cvolume pa_volume_;
+    bool muted_;
+    std::string port_name_;
+    std::string desc_;
+    bool scrolling_;
+};
+
+}  // namespace waybar::modules

include/modules/sni/snh.hpp

@@ -0,0 +1,38 @@
+#pragma once
+
+#include <gtkmm.h>
+#include <json/json.h>
+#include <tuple>
+#include <dbus-status-notifier-watcher.h>
+#include "modules/sni/sni.hpp"
+
+namespace waybar::modules::SNI {
+
+class Host {
+  public:
+    Host(Glib::Dispatcher*, const Json::Value&);
+    std::vector<Item> items;
+  private:
+    static void busAcquired(GDBusConnection*, const gchar*, gpointer);
+    static void nameAppeared(GDBusConnection*, const gchar*, const gchar*,
+      gpointer);
+    static void nameVanished(GDBusConnection*, const gchar*, gpointer);
+    static void proxyReady(GObject*, GAsyncResult*, gpointer);
+    static void registerHost(GObject*, GAsyncResult*, gpointer);
+    static void itemRegistered(SnWatcher*, const gchar*, gpointer);
+    static void itemUnregistered(SnWatcher*, const gchar*, gpointer);
+
+    std::tuple<std::string, std::string> getBusNameAndObjectPath(const gchar*);
+    void addRegisteredItem(const gchar* service);
+
+    uint32_t bus_name_id_;
+    uint32_t watcher_id_;
+    std::string bus_name_;
+    std::string object_path_;
+    Glib::Dispatcher* dp_;
+    GCancellable* cancellable_ = nullptr;
+    SnWatcher* watcher_ = nullptr;
+    const Json::Value &config_;
+};
+
+}

include/modules/sni/sni.hpp

@@ -0,0 +1,59 @@
+#pragma once
+
+#include <dbus-status-notifier-item.h>
+#include <gtkmm.h>
+#include <json/json.h>
+#ifdef FILESYSTEM_EXPERIMENTAL
+  #include <experimental/filesystem>
+#else
+  #include <filesystem>
+#endif
+
+namespace waybar::modules::SNI {
+
+class Item {
+public:
+  Item(std::string, std::string, Glib::Dispatcher*, Json::Value);
+
+  std::string bus_name;
+  std::string object_path;
+  Gtk::EventBox event_box;
+
+  int icon_size;
+  int effective_icon_size;
+  Gtk::Image *image;
+  Gtk::Menu *gtk_menu = nullptr;
+  std::string category;
+  std::string id;
+  std::string status;
+
+  std::string title;
+  int32_t window_id;
+  std::string icon_name;
+  Glib::RefPtr<Gdk::Pixbuf> icon_pixmap;
+  std::string overlay_icon_name;
+  std::string attention_icon_name;
+  std::string attention_movie_name;
+  std::string icon_theme_path;
+  std::string menu;
+  bool item_is_menu;
+
+private:
+  static void proxyReady(GObject *obj, GAsyncResult *res, gpointer data);
+  static void getAll(GObject *obj, GAsyncResult *res, gpointer data);
+  static void handleActivate(GObject *, GAsyncResult *, gpointer);
+  static void handleSecondaryActivate(GObject *, GAsyncResult *, gpointer);
+
+  void updateImage();
+  void updateMenu();
+  Glib::RefPtr<Gdk::Pixbuf> extractPixBuf(GVariant *variant);
+  Glib::RefPtr<Gdk::Pixbuf> getIconByName(std::string name, int size);
+  bool handleClick(GdkEventButton *const & /*ev*/);
+
+  Glib::Dispatcher *dp_;
+  GCancellable *cancellable_ = nullptr;
+  SnItem *proxy_ = nullptr;
+  Json::Value config_;
+};
+
+} // namespace waybar::modules::SNI

include/modules/sni/snw.hpp

@@ -0,0 +1,47 @@
+#pragma once
+
+#include <gtkmm.h>
+#include <dbus-status-notifier-watcher.h>
+
+namespace waybar::modules::SNI {
+
+class Watcher {
+public:
+  Watcher();
+  ~Watcher();
+
+private:
+  typedef enum { GF_WATCH_TYPE_HOST, GF_WATCH_TYPE_ITEM } GfWatchType;
+
+  typedef struct {
+    GfWatchType type;
+    Watcher *watcher;
+    gchar *service;
+    gchar *bus_name;
+    gchar *object_path;
+    guint watch_id;
+  } GfWatch;
+
+  static void busAcquired(GDBusConnection *, const gchar *, gpointer);
+  static gboolean handleRegisterHost(Watcher *, GDBusMethodInvocation *,
+                                     const gchar *);
+  static gboolean handleRegisterItem(Watcher *, GDBusMethodInvocation *,
+                                     const gchar *);
+  static GfWatch *gfWatchFind(GSList *list, const gchar *bus_name,
+                              const gchar *object_path);
+  static GfWatch *gfWatchNew(GfWatchType, const gchar *, const gchar *,
+                             const gchar *, Watcher *);
+  static void nameVanished(GDBusConnection *connection, const char *name,
+                           gpointer data);
+
+  void updateRegisteredItems(SnWatcher *obj);
+
+  uint32_t bus_name_id_;
+  uint32_t watcher_id_;
+  GSList *hosts_ = nullptr;
+  GSList *items_ = nullptr;
+  SnWatcher *watcher_ = nullptr;
+};
+
+} // namespace waybar::modules::SNI
+

include/modules/sni/tray.hpp

@@ -0,0 +1,25 @@
+#pragma once
+
+#include <fmt/format.h>
+#include <thread>
+#include "util/json.hpp"
+#include "IModule.hpp"
+#include "modules/sni/snw.hpp"
+#include "modules/sni/snh.hpp"
+
+namespace waybar::modules::SNI {
+
+class Tray : public IModule {
+  public:
+    Tray(const Json::Value&);
+    auto update() -> void;
+    operator Gtk::Widget &();
+  private:
+    std::thread thread_;
+    const Json::Value& config_;
+    Gtk::Box box_;
+    SNI::Watcher watcher_ ;
+    SNI::Host host_;
+};
+
+}

include/modules/sway/ipc/client.hpp

@@ -0,0 +1,42 @@
+#pragma once
+
+#include <iostream>
+#include <cstring>
+#include <unistd.h>
+#include <sys/socket.h>
+#include <sys/un.h>
+#include "ipc.hpp"
+
+namespace waybar::modules::sway {
+
+class Ipc {
+  public:
+    Ipc();
+    ~Ipc();
+
+		struct ipc_response {
+			uint32_t size;
+			uint32_t type;
+			std::string payload;
+		};
+
+		void connect();
+		struct ipc_response sendCmd(uint32_t type,
+			const std::string& payload = "") const;
+		void subscribe(const std::string& payload) const;
+		struct ipc_response handleEvent() const;
+  protected:
+		static inline const std::string ipc_magic_ = "i3-ipc";
+		static inline const size_t ipc_header_size_ = ipc_magic_.size() + 8;
+
+		const std::string getSocketPath() const;
+		int open(const std::string&) const;
+		struct ipc_response send(int fd, uint32_t type,
+			const std::string& payload = "") const;
+		struct ipc_response recv(int fd) const;
+
+		int fd_;
+		int fd_event_;
+};
+
+}

include/modules/sway/ipc/ipc.hpp

@@ -1,6 +1,6 @@
 #pragma once
 
-#define event_mask(ev) (1 << (ev & 0x7F))
+#define event_mask(ev) (1u << (ev & 0x7F))
 
 enum ipc_command_type {
 	// i3 command types - see i3's I3_REPLY_TYPE constants

include/modules/sway/mode.hpp

@@ -0,0 +1,27 @@
+#pragma once
+
+#include <fmt/format.h>
+#include "bar.hpp"
+#include "client.hpp"
+#include "util/chrono.hpp"
+#include "util/json.hpp"
+#include "ALabel.hpp"
+#include "modules/sway/ipc/client.hpp"
+
+namespace waybar::modules::sway {
+
+class Mode : public ALabel {
+  public:
+    Mode(waybar::Bar&, const Json::Value&);
+    auto update() -> void;
+  private:
+    void worker();
+    
+    Bar& bar_;
+    waybar::util::SleeperThread thread_;
+    util::JsonParser parser_;
+    Ipc ipc_;
+    std::string mode_;
+};
+
+}

include/modules/sway/window.hpp

@@ -0,0 +1,31 @@
+#pragma once
+
+#include <fmt/format.h>
+#include <tuple>
+#include "bar.hpp"
+#include "client.hpp"
+#include "util/chrono.hpp"
+#include "util/json.hpp"
+#include "ALabel.hpp"
+#include "modules/sway/ipc/client.hpp"
+
+namespace waybar::modules::sway {
+
+class Window : public ALabel {
+  public:
+    Window(waybar::Bar&, const Json::Value&);
+    auto update() -> void;
+  private:
+    void worker();
+    std::tuple<int, std::string> getFocusedNode(Json::Value nodes);
+    void getFocusedWindow();
+
+    Bar& bar_;
+    waybar::util::SleeperThread thread_;
+    util::JsonParser parser_;
+    Ipc ipc_;
+    std::string window_;
+    int windowId_;
+};
+
+}

include/modules/sway/workspaces.hpp

@@ -0,0 +1,38 @@
+#pragma once
+
+#include <fmt/format.h>
+#include "bar.hpp"
+#include "client.hpp"
+#include "util/chrono.hpp"
+#include "util/json.hpp"
+#include "IModule.hpp"
+#include "modules/sway/ipc/client.hpp"
+
+namespace waybar::modules::sway {
+
+class Workspaces : public IModule {
+  public:
+    Workspaces(waybar::Bar&, const Json::Value&);
+    auto update() -> void;
+    operator Gtk::Widget &();
+  private:
+    void worker();
+    void addWorkspace(Json::Value);
+    std::string getIcon(std::string, Json::Value);
+    bool handleScroll(GdkEventScroll*);
+    std::string getPrevWorkspace();
+    std::string getNextWorkspace();
+
+    Bar& bar_;
+    const Json::Value& config_;
+    waybar::util::SleeperThread thread_;
+    Gtk::Box box_;
+    util::JsonParser parser_;
+    std::mutex mutex_;
+    bool scrolling_;
+    std::unordered_map<std::string, Gtk::Button> buttons_;
+    Json::Value workspaces_;
+    Ipc ipc_;
+};
+
+}

include/modules/workspaces.hpp

@@ -1,33 +0,0 @@
-#pragma once
-
-#include <fmt/format.h>
-#include "bar.hpp"
-#include "client.hpp"
-#include "util/chrono.hpp"
-#include "IModule.hpp"
-
-namespace waybar::modules {
-
-  class Workspaces : public IModule {
-    public:
-      Workspaces(waybar::Bar &bar);
-      auto update() -> void;
-      operator Gtk::Widget &();
-    private:
-      void _updateThread();
-      static void _handle_idle(void *data,
-        struct org_kde_kwin_idle_timeout *timer);
-      static void _handle_resume(void *data,
-        struct org_kde_kwin_idle_timeout *timer);
-      void _addWorkspace(Json::Value node);
-      Json::Value _getWorkspaces();
-      Bar &_bar;
-      util::SleeperThread *_thread;
-      Gtk::Box *_box;
-      std::unordered_map<int, Gtk::Button> _buttons;
-      int _ipcSocketfd;
-      int _ipcEventSocketfd;
-      struct org_kde_kwin_idle_timeout *_idle_timer;
-  };
-
-}

include/util/algorithm.hpp

@@ -1,823 +0,0 @@
-#pragma once
-
-#include <algorithm>
-#include <functional>
-#include <initializer_list>
-#include <numeric>
-#include <string>
-#include <string_view>
-
-namespace waybar::util {
-
-  /// Joins a sequence of strings, separating them using `js`
-  template<class StrIterator> // Models InputIterator<std::string>
-  std::string join_strings(StrIterator b, StrIterator e, std::string_view js = ", ")
-  {
-    std::string result;
-    std::for_each(b, e, [&](auto&& s) {
-      if (!result.empty()) {
-        result.append(js);
-      }
-      result.append(s);
-    });
-    return result;
-  }
-
-  inline const char* nonull(const char* str) {
-    if (str == nullptr) return "";
-    return str;
-  };
-
-  inline bool iequals(std::string_view a, std::string_view b)
-  {
-    return std::equal(a.begin(), a.end(), b.begin(), b.end(),
-                      [](char a, char b) { return tolower(a) == tolower(b); });
-  }
-
-  inline bool starts_with(std::string_view prefix, std::string_view a)
-  {
-    return a.compare(0, prefix.size(), prefix) == 0;
-  }
-
-  inline bool ends_with(std::string_view prefix, std::string_view a)
-  {
-    return a.compare(a.size() - prefix.size(), prefix.size(), prefix) == 0;
-  }
-
-  /// Return a closure which compares the adress any reference to T to the address of t
-  template<typename T>
-  constexpr auto addr_eq(T&& t) {
-    return [&t] (auto&& t2) {
-      return &t == &t2;
-    };
-  }
-
-  template<typename T>
-  bool erase_this(std::vector<T>& cont, T* el)
-  {
-    if (el < cont.data() && el >= cont.data() + cont.size()) return false;
-    cont.erase(cont.begin() + (el - cont.data()));
-    return true;
-  }
-
-  template<typename T>
-  bool erase_this(std::vector<T>& cont, T& el)
-  {
-    return erase_this(cont, &el);
-  }
-
-  namespace detail {
-    template<class Func, int... ns>
-    constexpr auto generate_array_impl(std::integer_sequence<int, ns...>&&, Func&& gen)
-    {
-      return std::array<std::decay_t<decltype(std::invoke(gen, std::declval<int>()))>,
-                        sizeof...(ns)>{{std::invoke(gen, ns)...}};
-    }
-  } // namespace detail
-
-  template<int n, class Func>
-  constexpr auto generate_array(Func&& gen)
-  {
-    auto intseq = std::make_integer_sequence<int, n>();
-    return detail::generate_array_impl(std::move(intseq), std::forward<Func>(gen));
-  }
-
-  namespace view {
-
-    namespace detail {
-      template<typename T>
-      using store_or_ref_t = std::conditional_t<std::is_rvalue_reference_v<T>, std::decay_t<T>, T&>;
-    }
-
-    template<typename Cont>
-    struct reverse {
-
-      reverse(Cont&& cont) noexcept : _container(std::forward<Cont>(cont)) {}
-
-      auto begin()
-      {
-        return std::rbegin(_container);
-      }
-
-      auto end()
-      {
-        return std::rend(_container);
-      }
-
-      auto begin() const
-      {
-        return std::rbegin(_container);
-      }
-
-      auto end() const
-      {
-        return std::rend(_container);
-      }
-
-      auto cbegin() const
-      {
-        return std::crbegin(_container);
-      }
-
-      auto cend() const
-      {
-        return std::crend(_container);
-      }
-
-      detail::store_or_ref_t<Cont&&> _container;
-    };
-
-    template<typename ContRef>
-    reverse(ContRef&& cont) -> reverse<ContRef&&>;
-
-    template<typename Cont>
-    struct constant {
-      constant(Cont&& cont) noexcept : _container(std::forward<Cont>(cont)){};
-
-      auto begin() const
-      {
-        return std::cbegin(_container);
-      }
-
-      auto end() const
-      {
-        return std::cend(_container);
-      }
-
-      auto cbegin() const
-      {
-        return std::cbegin(_container);
-      }
-
-      auto cend() const
-      {
-        return std::cend(_container);
-      }
-
-      detail::store_or_ref_t<Cont&&> _container;
-    };
-
-    template<typename ContRef>
-    constant(ContRef&& cont) -> constant<ContRef&&>;
-
-  } // namespace view
-
-
-  /*
-   * Range algorithms
-   */
-
-  template<typename InputIt, typename Size, typename F>
-  constexpr InputIt for_each_n(InputIt&& first, Size n, F&& f)
-  {
-    for (Size i = 0; i < n; ++first, ++i) {
-      std::invoke(f, *first);
-    }
-    return first;
-  }
-
-  /// `for_each` with access to an index value. Function called as `f(*it, i)`
-  ///
-  /// For each item in range `[first, last)`, invoke `f` with args
-  /// `*iter, i` where `iter` is the current iterator, and `i` is
-  /// an incrementing value, starting at zero. Use this instead of
-  /// raw indexed loops wherever possible.
-  ///
-  /// \param first Input iterator to the begining of the range
-  /// \param last Input iterator to the end of the range
-  /// \param f Must be invocable with arguments `value_type`, `std::size_t`
-  /// \returns The number of iterations performed
-  template<typename InputIt, typename F>
-  constexpr std::size_t indexed_for(InputIt&& first, InputIt&& last, F&& f)
-  {
-    std::size_t i = 0;
-    std::for_each(std::forward<InputIt>(first), std::forward<InputIt>(last), [&](auto&& a) {
-      std::invoke(f, a, i);
-      i++;
-    });
-    return i;
-  }
-
-  template<typename Rng, typename F>
-  constexpr std::size_t indexed_for(Rng&& rng, F&& f)
-  {
-    return indexed_for(std::begin(rng), std::end(rng), std::forward<F>(f));
-  }
-
-
-  /// `for_each_n` with access to an index value. Function called as `f(*it, i)`
-  ///
-  /// for `n` iterations, invoke `f` with args `*iter, i`
-  /// where `iter` is the current iterator starting with `first`,
-  /// and `i` is an incrementing value, starting at zero.
-  /// Use this instead of raw indexed loops wherever possible.
-  ///
-  /// \param first Input iterator to the begining of the range
-  /// \param n Number of iterations to go through
-  /// \param f Must be invocable with arguments `value_type`, `std::size_t`
-  /// \returns An iterator one past the last one visited
-  template<class InputIt, class Size, class F>
-  constexpr InputIt indexed_for_n(InputIt first, Size n, F&& f)
-  {
-    for (Size i = 0; i < n; ++first, ++i) {
-      std::invoke(f, *first, i);
-    }
-    return first;
-  }
-
-  template<class Rng, class Size, class F>
-  constexpr std::size_t indexed_for_n(Rng&& rng, Size n, F&& f)
-  {
-    return indexed_for_n(std::begin(rng), std::end(rng), n, std::forward<F>(f));
-  }
-
-  template<typename Iter1, typename Iter2, typename F>
-  constexpr void for_both(Iter1&& f1, Iter1&& l1, Iter2&& f2, Iter2&& l2, F&& f)
-  {
-    Iter1 i1 = std::forward<Iter1>(f1);
-    Iter2 i2 = std::forward<Iter2>(f2);
-    for (; i1 != l1 && i2 != l2; i1++, i2++) {
-      std::invoke(f, *i1, *i2);
-    }
-  }
-
-  template<typename Rng1, typename Rng2, typename F>
-  constexpr void for_both(Rng1&& r1, Rng2&& r2, F&& f)
-  {
-    for_both(std::begin(r1), std::end(r1), std::begin(r2), std::end(r2), std::forward<F>(f));
-  }
-
-  /*
-   * Range based standard algorithms
-   *
-   * Thanks, chris from SO!
-   */
-
-  template<typename Cont, typename T>
-  constexpr auto accumulate(Cont&& cont, T&& init)
-  {
-    // TODO C++20: std::accumulate is constexpr
-    using std::begin, std::end;
-    auto first = begin(cont);
-    auto last = end(cont);
-    for (; first != last; ++first) init = init + *first;
-    return init;
-  }
-
-  template<typename Cont, typename T, typename BinaryOperation>
-  constexpr auto accumulate(Cont&& cont, T&& init, BinaryOperation&& op)
-  {
-    // TODO C++20: std::accumulate is constexpr
-    using std::begin, std::end;
-    auto first = begin(cont);
-    auto last = end(cont);
-    for (; first != last; ++first) init = op(init, *first);
-    return init;
-  }
-
-  template<typename Cont, typename OutputIterator>
-  decltype(auto) adjacent_difference(Cont&& cont, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::adjacent_difference(begin(cont), end(cont), std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont>
-  decltype(auto) prev_permutation(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::prev_permutation(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename Compare>
-  decltype(auto) prev_permutation(Cont&& cont, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::prev_permutation(begin(cont), end(cont), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont>
-  decltype(auto) push_heap(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::push_heap(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename Compare>
-  decltype(auto) push_heap(Cont&& cont, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::push_heap(begin(cont), end(cont), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename T>
-  decltype(auto) remove(Cont&& cont, T&& value)
-  {
-    using std::begin;
-    using std::end;
-    return std::remove(begin(cont), end(cont), std::forward<T>(value));
-  }
-
-  template<typename Cont, typename OutputIterator, typename T>
-  decltype(auto) remove_copy(Cont&& cont, OutputIterator&& first, T&& value)
-  {
-    using std::begin;
-    using std::end;
-    return std::remove_copy(begin(cont), end(cont), std::forward<OutputIterator>(first),
-                            std::forward<T>(value));
-  }
-
-  template<typename Cont, typename OutputIterator, typename UnaryPredicate>
-  decltype(auto) remove_copy_if(Cont&& cont, OutputIterator&& first, UnaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::remove_copy_if(begin(cont), end(cont), std::forward<OutputIterator>(first),
-                               std::forward<UnaryPredicate>(p));
-  }
-
-  template<typename Cont, typename UnaryPredicate>
-  decltype(auto) remove_if(Cont&& cont, UnaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::remove_if(begin(cont), end(cont), std::forward<UnaryPredicate>(p));
-  }
-
-  template<typename Cont, typename T, typename T2>
-  decltype(auto) replace(Cont&& cont, T&& old_value, T2&& new_value)
-  {
-    using std::begin;
-    using std::end;
-    return std::replace(begin(cont), end(cont), std::forward<T>(old_value),
-                        std::forward<T2>(new_value));
-  }
-
-  template<typename Cont, typename OutputIterator, typename T, typename T2>
-  decltype(auto) replace_copy(Cont&& cont, OutputIterator&& first, T&& old_value, T2&& new_value)
-  {
-    using std::begin;
-    using std::end;
-    return std::replace_copy(begin(cont), end(cont), std::forward<OutputIterator>(first),
-                             std::forward<T>(old_value), std::forward<T2>(old_value));
-  }
-
-  template<typename Cont, typename OutputIterator, typename UnaryPredicate, typename T>
-  decltype(auto) replace_copy_if(Cont&& cont,
-                                 OutputIterator&& first,
-                                 UnaryPredicate&& p,
-                                 T&& new_value)
-  {
-    using std::begin;
-    using std::end;
-    return std::replace_copy(begin(cont), end(cont), std::forward<OutputIterator>(first),
-                             std::forward<UnaryPredicate>(p), std::forward<T>(new_value));
-  }
-
-  template<typename Cont, typename UnaryPredicate, typename T>
-  decltype(auto) replace_if(Cont&& cont, UnaryPredicate&& p, T&& new_value)
-  {
-    using std::begin;
-    using std::end;
-    return std::replace_if(begin(cont), end(cont), std::forward<UnaryPredicate>(p),
-                           std::forward<T>(new_value));
-  }
-
-  template<typename Cont>
-  decltype(auto) reverse(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::reverse(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename OutputIterator>
-  decltype(auto) reverse_copy(Cont&& cont, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::reverse_copy(begin(cont), end(cont), std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename ForwardIterator>
-  decltype(auto) rotate(Cont&& cont, ForwardIterator&& new_first)
-  {
-    using std::begin;
-    using std::end;
-    return std::rotate(begin(cont), std::forward<ForwardIterator>(new_first), end(cont));
-  }
-
-  template<typename Cont, typename ForwardIterator, typename OutputIterator>
-  decltype(auto) rotate_copy(Cont&& cont, ForwardIterator&& new_first, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::rotate_copy(begin(cont), std::forward<ForwardIterator>(new_first), end(cont),
-                            std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename Cont2>
-  decltype(auto) search(Cont&& cont, Cont2&& cont2)
-  {
-    using std::begin;
-    using std::end;
-    return std::search(begin(cont), end(cont), begin(cont2), end(cont2));
-  }
-
-  template<typename Cont, typename Cont2, typename BinaryPredicate>
-  decltype(auto) search(Cont&& cont, Cont2&& cont2, BinaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::search(begin(cont), end(cont), begin(cont2), end(cont2),
-                       std::forward<BinaryPredicate>(p));
-  }
-
-  template<typename Cont, typename Size, typename T>
-  decltype(auto) search_n(Cont&& cont, Size count, T&& value)
-  {
-    using std::begin;
-    using std::end;
-    return std::search_n(begin(cont), end(cont), count, std::forward<T>(value));
-  }
-
-  template<typename Cont, typename Size, typename T, typename BinaryPredicate>
-  decltype(auto) search_n(Cont&& cont, Size count, T&& value, BinaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::search_n(begin(cont), end(cont), count, std::forward<T>(value),
-                         std::forward<BinaryPredicate>(p));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator>
-  decltype(auto) set_difference(Cont&& cont, Cont2&& cont2, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_difference(begin(cont), end(cont), begin(cont2), end(cont2),
-                               std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator, typename Compare>
-  decltype(auto) set_difference(Cont&& cont, Cont2&& cont2, OutputIterator&& first, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_difference(begin(cont), end(cont), begin(cont2), end(cont2),
-                               std::forward<OutputIterator>(first), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator>
-  decltype(auto) set_intersection(Cont&& cont, Cont2&& cont2, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_intersection(begin(cont), end(cont), begin(cont2), end(cont2),
-                                 std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator, typename Compare>
-  decltype(auto) set_intersection(Cont&& cont,
-                                  Cont2&& cont2,
-                                  OutputIterator&& first,
-                                  Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_intersection(begin(cont), end(cont), begin(cont2), end(cont2),
-                                 std::forward<OutputIterator>(first), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator>
-  decltype(auto) set_symmetric_difference(Cont&& cont, Cont2&& cont2, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_symmetric_difference(begin(cont), end(cont), begin(cont2), end(cont2),
-                                         std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator, typename Compare>
-  decltype(auto) set_symmetric_difference(Cont&& cont,
-                                          Cont2&& cont2,
-                                          OutputIterator&& first,
-                                          Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_symmetric_difference(begin(cont), end(cont), begin(cont2), end(cont2),
-                                         std::forward<OutputIterator>(first),
-                                         std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator>
-  decltype(auto) set_union(Cont&& cont, Cont2&& cont2, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_union(begin(cont), end(cont), begin(cont2), end(cont2),
-                          std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename Cont2, typename OutputIterator, typename Compare>
-  decltype(auto) set_union(Cont&& cont, Cont2&& cont2, OutputIterator&& first, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::set_union(begin(cont), end(cont), begin(cont2), end(cont2),
-                          std::forward<OutputIterator>(first), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename UniformRandomNumberGenerator>
-  decltype(auto) shuffle(Cont&& cont, UniformRandomNumberGenerator&& g)
-  {
-    using std::begin;
-    using std::end;
-    return std::shuffle(begin(cont), end(cont), std::forward<UniformRandomNumberGenerator>(g));
-  }
-
-  template<typename Cont>
-  decltype(auto) sort(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::sort(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename Compare>
-  decltype(auto) sort(Cont&& cont, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::sort(begin(cont), end(cont), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont>
-  decltype(auto) sort_heap(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::sort_heap(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename Compare>
-  decltype(auto) sort_heap(Cont&& cont, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::sort_heap(begin(cont), end(cont), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename UnaryPredicate>
-  decltype(auto) stable_partition(Cont&& cont, UnaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::stable_partition(begin(cont), end(cont), std::forward<UnaryPredicate>(p));
-  }
-
-  template<typename Cont>
-  decltype(auto) stable_sort(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::stable_sort(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename Compare>
-  decltype(auto) stable_sort(Cont&& cont, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::stable_sort(begin(cont), end(cont), std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename ForwardIterator>
-  decltype(auto) swap_ranges(Cont&& cont, ForwardIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::swap_ranges(begin(cont), end(cont), std::forward<ForwardIterator>(first));
-  }
-
-  template<typename Cont, typename Cont2, typename F>
-  auto transform(Cont&& cont, Cont2&& cont2, F&& f) -> decltype(begin(cont2))
-  {
-    using std::begin;
-    using std::end;
-    return std::transform(begin(cont), end(cont), begin(cont2), std::forward<F>(f));
-  }
-
-  template<typename Cont, typename Iter, typename F>
-  decltype(auto) transform(Cont&& cont, Iter&& iter, F&& f)
-  {
-    using std::begin;
-    using std::end;
-    return std::transform(begin(cont), end(cont), std::forward<Iter>(iter), std::forward<F>(f));
-  }
-
-  template<typename Cont, typename Cont2, typename Cont3, typename BinaryPredicate>
-  auto transform(Cont&& cont, Cont2&& cont2, Cont3&& cont3, BinaryPredicate&& f)
-    -> decltype(begin(cont2), begin(cont3))
-  {
-    using std::begin;
-    using std::end;
-    return std::transform(begin(cont), end(cont), begin(cont2), begin(cont3),
-                          std::forward<BinaryPredicate>(f));
-  }
-
-  template<typename Cont, typename InputIterator, typename Cont3, typename BinaryPredicate>
-  auto transform(Cont&& cont, InputIterator&& iter, Cont3&& cont3, BinaryPredicate&& f)
-    -> decltype(begin(cont), begin(cont3))
-  {
-    using std::begin;
-    using std::end;
-    return std::transform(begin(cont), end(cont), std::forward<InputIterator>(iter), begin(cont3),
-                          std::forward<BinaryPredicate>(f));
-  }
-
-  template<typename Cont, typename Cont2, typename InputIterator, typename BinaryPredicate>
-  auto transform(Cont&& cont, Cont2&& cont2, InputIterator&& iter, BinaryPredicate&& f)
-    -> decltype(begin(cont), begin(cont2), iter)
-  {
-    using std::begin;
-    using std::end;
-    return std::transform(begin(cont), end(cont), begin(cont2), std::forward<InputIterator>(iter),
-                          std::forward<BinaryPredicate>(f));
-  }
-
-  template<typename Cont, typename InputIterator, typename OutputIterator, typename BinaryOperation>
-  decltype(auto) transform(Cont&& cont,
-                           InputIterator&& firstIn,
-                           OutputIterator&& firstOut,
-                           BinaryOperation&& op)
-  {
-    using std::begin;
-    using std::end;
-    return std::transform(begin(cont), end(cont), std::forward<InputIterator>(firstIn),
-                          std::forward<OutputIterator>(firstOut),
-                          std::forward<BinaryOperation>(op));
-  }
-
-  template<typename Cont>
-  decltype(auto) unique(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::unique(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename BinaryPredicate>
-  decltype(auto) unique(Cont&& cont, BinaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::unique(begin(cont), end(cont), std::forward<BinaryPredicate>(p));
-  }
-
-  template<typename Cont, typename OutputIterator>
-  decltype(auto) unique_copy(Cont&& cont, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::unique_copy(begin(cont), end(cont), std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename OutputIterator, typename BinaryPredicate>
-  decltype(auto) unique_copy(Cont&& cont, OutputIterator&& first, BinaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::unique_copy(begin(cont), end(cont), std::forward<OutputIterator>(first),
-                            std::forward<BinaryPredicate>(p));
-  }
-
-  template<typename Cont, typename T>
-  decltype(auto) upper_bound(Cont&& cont, T&& value)
-  {
-    using std::begin;
-    using std::end;
-    return std::upper_bound(begin(cont), end(cont), std::forward<T>(value));
-  }
-
-  template<typename Cont, typename T, typename Compare>
-  decltype(auto) upper_bound(Cont&& cont, T&& value, Compare&& comp)
-  {
-    using std::begin;
-    using std::end;
-    return std::upper_bound(begin(cont), end(cont), std::forward<T>(value),
-                            std::forward<Compare>(comp));
-  }
-
-  template<typename Cont, typename OutputIterator>
-  decltype(auto) copy(Cont&& cont, OutputIterator&& first)
-  {
-    using std::begin;
-    using std::end;
-    return std::copy(begin(cont), end(cont), std::forward<OutputIterator>(first));
-  }
-
-  template<typename Cont, typename OutputIterator, typename UnaryPredicate>
-  decltype(auto) copy_if(Cont&& cont, OutputIterator&& first, UnaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::copy_if(begin(cont), end(cont), std::forward<OutputIterator>(first),
-                        std::forward<UnaryPredicate>(p));
-  }
-
-  template<typename Cont, typename T>
-  decltype(auto) fill(Cont&& cont, T&& value)
-  {
-    using std::begin;
-    using std::end;
-    return std::fill(begin(cont), end(cont), std::forward<T>(value));
-  }
-
-  template<typename Cont, typename T>
-  decltype(auto) fill_n(Cont&& cont, std::size_t n, T&& value)
-  {
-    using std::begin;
-    using std::end;
-    return std::fill_n(begin(cont), n, std::forward<T>(value));
-  }
-
-  template<typename Cont, typename UnaryPredicate>
-  decltype(auto) any_of(Cont&& cont, UnaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::any_of(begin(cont), end(cont), std::forward<UnaryPredicate>(p));
-  }
-
-  template<typename Cont, typename UnaryPredicate>
-  decltype(auto) all_of(Cont&& cont, UnaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::all_of(begin(cont), end(cont), std::forward<UnaryPredicate>(p));
-  }
-
-  template<typename Cont, typename UnaryPredicate>
-  decltype(auto) none_of(Cont&& cont, UnaryPredicate&& p)
-  {
-    using std::begin;
-    using std::end;
-    return std::none_of(begin(cont), end(cont), std::forward<UnaryPredicate>(p));
-  }
-
-  template<typename Cont>
-  decltype(auto) max_element(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::max_element(begin(cont), end(cont));
-  }
-
-  template<typename Cont>
-  decltype(auto) min_element(Cont&& cont)
-  {
-    using std::begin;
-    using std::end;
-    return std::min_element(begin(cont), end(cont));
-  }
-
-  template<typename Cont, typename Compare>
-  decltype(auto) min_element(Cont&& cont, Compare&& f)
-  {
-    using std::begin;
-    using std::end;
-    return std::min_element(begin(cont), end(cont), std::forward<Compare>(f));
-  }
-
-  template<typename Cont, typename Compare>
-  decltype(auto) max_element(Cont&& cont, Compare&& f)
-  {
-    using std::begin;
-    using std::end;
-    return std::max_element(begin(cont), end(cont), std::forward<Compare>(f));
-  }
-
-  template<typename Cont, typename T>
-  decltype(auto) find(Cont&& cont, T&& t)
-  {
-    using std::begin;
-    using std::end;
-    return std::find(begin(cont), end(cont), std::forward<T>(t));
-  }
-
-  template<typename Cont, typename UnaryPredicate>
-  decltype(auto) find_if(Cont&& cont, UnaryPredicate&& f)
-  {
-    using std::begin;
-    using std::end;
-    return std::find_if(begin(cont), end(cont), std::forward<UnaryPredicate>(f));
-  }
-
-} // namespace waybar::util

include/util/chrono.hpp

@@ -5,82 +5,90 @@
 #include <functional>
 #include <condition_variable>
 #include <thread>
+#include <gtkmm.h>
 
 namespace waybar::chrono {
 
-    using namespace std::chrono;
+using namespace std::chrono;
 
-    using clock = std::chrono::system_clock;
-    using duration = clock::duration;
-    using time_point = std::chrono::time_point<clock, duration>;
-
-    inline struct timespec to_timespec(time_point t) noexcept
-    {
-      long secs = duration_cast<seconds>(t.time_since_epoch()).count();
-      long nsc = duration_cast<nanoseconds>(t.time_since_epoch() % seconds(1)).count();
-      return {secs, nsc};
-    }
-
-    inline time_point to_time_point(struct timespec t) noexcept
-    {
-      return time_point(duration_cast<duration>(seconds(t.tv_sec) + nanoseconds(t.tv_nsec)));
-    }
+using clock = std::chrono::system_clock;
+using duration = clock::duration;
+using time_point = std::chrono::time_point<clock, duration>;
 
 }
 
 namespace waybar::util {
 
-  struct SleeperThread {
-    SleeperThread() = default;
+struct SleeperThread {
+  SleeperThread() = default;
 
-    SleeperThread(std::function<void()> func)
-      : thread{[this, func] {
-          do {
-            func();
-          } while (do_run);
-        }}
-    {}
-
-    SleeperThread& operator=(std::function<void()> func)
-    {
-      thread = std::thread([this, func] {
-        do {
+  SleeperThread(std::function<void()> func)
+    : thread_{[this, func] {
+        while(true) {
+          {
+            std::lock_guard<std::mutex> lock(mutex_);
+            if (!do_run_) {
+              break;
+            }
+          }
           func();
-        } while (do_run);
-      });
-      return *this;
+        }
+      }}
+  {}
+
+  SleeperThread& operator=(std::function<void()> func)
+  {
+    thread_ = std::thread([this, func] {
+      while(true) {
+        {
+          std::lock_guard<std::mutex> lock(mutex_);
+          if (!do_run_) {
+            break;
+          }
+        }
+        func();
+      }
+    });
+    return *this;
+  }
+
+
+  auto sleep_for(chrono::duration dur)
+  {
+    auto lock = std::unique_lock(mutex_);
+    return condvar_.wait_for(lock, dur);
+  }
+
+  auto sleep_until(chrono::time_point time)
+  {
+    auto lock = std::unique_lock(mutex_);
+    return condvar_.wait_until(lock, time);
+  }
+
+  auto wake_up()
+  {
+    condvar_.notify_all();
+  }
+
+  auto stop()
+  {
+    do_run_ = false;
+    condvar_.notify_all();
+    if (thread_.joinable()) {
+      thread_.detach();
     }
+  }
 
+  ~SleeperThread()
+  {
+    stop();
+  }
 
-    auto sleep_for(chrono::duration dur)
-    {
-      auto lock = std::unique_lock(mutex);
-      return condvar.wait_for(lock, dur);
-    }
-
-    auto sleep_until(chrono::time_point time)
-    {
-      auto lock = std::unique_lock(mutex);
-      return condvar.wait_until(lock, time);
-    }
-
-    auto wake_up()
-    {
-      condvar.notify_all();
-    }
-
-    ~SleeperThread()
-    {
-      do_run = false;
-      condvar.notify_all();
-      thread.join();
-    }
-
-  private:
-    std::thread thread;
-    std::condition_variable condvar;
-    std::mutex mutex;
-    bool do_run = true;
-  };
+private:
+  std::thread thread_;
+  std::condition_variable condvar_;
+  std::mutex mutex_;
+  bool do_run_ = true;
+};
 
 }

include/util/command.hpp

@@ -0,0 +1,51 @@
+#pragma once
+
+#include <sys/wait.h>
+
+namespace waybar::util::command {
+
+struct res {
+  int exit_code;
+  std::string out;
+};
+
+inline struct res exec(const std::string cmd)
+{
+  FILE* fp(popen(cmd.c_str(), "r"));
+  if (!fp) {
+    return { -1, "" };
+  }
+
+  std::array<char, 128> buffer = {0};
+  std::string output;
+  while (feof(fp) == 0) {
+    if (fgets(buffer.data(), 128, fp) != nullptr) {
+      output += buffer.data();
+    }
+  }
+
+  // Remove last newline
+  if (!output.empty() && output[output.length()-1] == '\n') {
+    output.erase(output.length()-1);
+  }
+  int exit_code = WEXITSTATUS(pclose(fp));
+  return {exit_code, output};
+}
+
+inline bool forkExec(std::string cmd) {
+  if (cmd == "") return true;
+
+  int32_t pid = fork();
+
+  if (pid < 0) {
+    printf("Unable to exec cmd %s, error %s", cmd.c_str(), strerror(errno));
+    return false;
+  }
+
+  // Child executes the command
+  if (!pid) execl("/bin/sh", "sh", "-c", cmd.c_str(), (char*)0);
+
+  return true;
+}
+
+}  // namespace waybar::util::command

include/util/json.hpp

@@ -0,0 +1,31 @@
+#pragma once
+
+#include <json/json.h>
+
+namespace waybar::util {
+
+struct JsonParser {
+
+  JsonParser()
+    : reader_(builder_.newCharReader())
+  {}
+
+  const Json::Value parse(const std::string data) const
+  {
+    Json::Value root;
+    std::string err;
+    bool res =
+      reader_->parse(data.c_str(), data.c_str() + data.size(), &root, &err);
+    if (!res)
+      throw std::runtime_error(err);
+    return root;
+  }
+
+  ~JsonParser() = default;
+
+private:
+  Json::CharReaderBuilder builder_;
+  std::unique_ptr<Json::CharReader> const reader_;
+};
+
+}

include/util/ptr_vec.hpp

@@ -1,581 +0,0 @@
-#pragma once
-
-#include <cassert>
-#include <memory>
-#include <type_traits>
-#include <vector>
-#include "algorithm.hpp"
-
-namespace waybar::util {
-
-  /// An iterator wrapper that dereferences twice.
-  template<typename Iter>
-  struct double_iterator {
-    using wrapped = Iter;
-
-    using value_type = std::decay_t<decltype(*std::declval<typename wrapped::value_type>())>;
-    using difference_type = typename wrapped::difference_type;
-    using reference = value_type&;
-    using pointer = value_type*;
-    using iterator_category = std::random_access_iterator_tag;
-
-    using self_t = double_iterator<Iter>;
-
-    double_iterator(wrapped w) : _iter(std::move(w)) {}
-    double_iterator() : _iter() {}
-
-    reference operator*() const
-    {
-      return (**_iter);
-    }
-    pointer operator->() const
-    {
-      return &(**_iter);
-    }
-
-    self_t& operator++()
-    {
-      _iter.operator++();
-      return *this;
-    }
-    self_t operator++(int i)
-    {
-      return _iter.operator++(i);
-    }
-    self_t& operator--()
-    {
-      _iter.operator--();
-      return *this;
-    }
-    self_t operator--(int i)
-    {
-      return _iter.operator--(i);
-    }
-
-    auto operator==(const self_t& rhs) const noexcept
-    {
-      return _iter == rhs._iter;
-    }
-    auto operator!=(const self_t& rhs) const noexcept
-    {
-      return _iter != rhs._iter;
-    }
-    auto operator<(const self_t& rhs) const noexcept
-    {
-      return _iter < rhs._iter;
-    }
-    auto operator>(const self_t& rhs) const noexcept
-    {
-      return _iter > rhs._iter;
-    }
-    auto operator<=(const self_t& rhs) const noexcept
-    {
-      return _iter <= rhs._iter;
-    }
-    auto operator>=(const self_t& rhs) const noexcept
-    {
-      return _iter >= rhs._iter;
-    }
-
-    self_t operator+(difference_type d) const noexcept
-    {
-      return _iter + d;
-    }
-    self_t operator-(difference_type d) const noexcept
-    {
-      return _iter - d;
-    }
-    auto operator-(const self_t& rhs) const noexcept
-    {
-      return _iter - rhs._iter;
-    }
-
-    self_t& operator+=(difference_type d)
-    {
-      _iter += d;
-      return *this;
-    }
-    self_t& operator-=(difference_type d)
-    {
-      _iter -= d;
-      return *this;
-    }
-
-    operator wrapped&()
-    {
-      return _iter;
-    }
-    operator const wrapped&() const
-    {
-      return _iter;
-    }
-
-    wrapped& data()
-    {
-      return _iter;
-    }
-    const wrapped& data() const
-    {
-      return _iter;
-    }
-
-  private:
-    wrapped _iter;
-  };
-
-  template<typename Iter>
-  auto operator+(typename double_iterator<Iter>::difference_type diff, double_iterator<Iter> iter)
-  {
-    return iter + diff;
-  }
-
-  /// To avoid clients being moved, they are stored in unique_ptrs, which are
-  /// moved around in a vector. This class is purely for convenience, to still
-  /// have iterator semantics, and a few other utility functions
-  template<typename T>
-  struct ptr_vec {
-    using value_type = T;
-
-    std::vector<std::unique_ptr<value_type>> _order;
-
-    using iterator = double_iterator<typename decltype(_order)::iterator>;
-    using const_iterator = double_iterator<typename decltype(_order)::const_iterator>;
-
-    using reverse_iterator = double_iterator<typename decltype(_order)::reverse_iterator>;
-    using const_reverse_iterator =
-      double_iterator<typename decltype(_order)::const_reverse_iterator>;
-
-    value_type& push_back(const value_type& v)
-    {
-      auto ptr = std::make_unique<value_type>(v);
-      auto res = ptr.get();
-      _order.push_back(std::move(ptr));
-      return *res;
-    }
-
-    value_type& push_back(value_type&& v)
-    {
-      auto ptr = std::make_unique<value_type>(std::move(v));
-      auto res = ptr.get();
-      _order.push_back(std::move(ptr));
-      return *res;
-    }
-
-    value_type& push_back(std::unique_ptr<value_type> ptr)
-    {
-      auto res = ptr.get();
-      _order.push_back(std::move(ptr));
-      return *res;
-    }
-
-    template<typename... Args>
-    value_type& emplace_back(Args&&... args)
-    {
-      return push_back(std::make_unique<value_type>(std::forward<Args>(args)...));
-    }
-
-    std::unique_ptr<value_type> erase(const value_type& v)
-    {
-      auto iter =
-        std::find_if(_order.begin(), _order.end(), [&v](auto&& uptr) { return uptr.get() == &v; });
-      if (iter != _order.end()) {
-        auto uptr = std::move(*iter);
-        _order.erase(iter);
-        return uptr;
-      }
-      return nullptr;
-    }
-
-    iterator rotate_to_back(const value_type& v)
-    {
-      auto iter =
-        std::find_if(_order.begin(), _order.end(), [&v](auto&& uptr) { return uptr.get() == &v; });
-      return rotate_to_back(iter);
-    }
-
-    iterator rotate_to_back(iterator iter)
-    {
-      if (iter != _order.end()) {
-        {
-          return std::rotate(iter.data(), iter.data() + 1, _order.end());
-        }
-      }
-      return end();
-    }
-
-    iterator rotate_to_front(const value_type& v)
-    {
-      auto iter =
-        std::find_if(_order.begin(), _order.end(), [&v](auto&& uptr) { return uptr.get() == &v; });
-      return rotate_to_front(iter);
-    }
-
-    iterator rotate_to_front(iterator iter)
-    {
-      if (iter != _order.end()) {
-        {
-          return std::rotate(_order.begin(), iter.data(), iter.data() + 1);
-        }
-      }
-      return end();
-    }
-
-    std::size_t size() const noexcept
-    {
-      return _order.size();
-    }
-
-    bool empty() const noexcept
-    {
-      return _order.empty();
-    }
-
-    std::size_t capacity() const noexcept
-    {
-      return _order.capacity();
-    }
-
-    std::size_t max_size() const noexcept
-    {
-      return _order.max_size();
-    }
-
-    void reserve(std::size_t new_cap) 
-    {
-      _order.reserve(new_cap);
-    }
-
-    void shrink_to_fit()
-    {
-      _order.shrink_to_fit();
-    }
-
-    value_type& operator[](std::size_t n)
-    {
-      return *_order[n];
-    }
-
-    const value_type& operator[](std::size_t n) const
-    {
-      return *_order[n];
-    }
-
-    value_type& at(std::size_t n)
-    {
-      return *_order.at(n);
-    }
-
-    const value_type& at(std::size_t n) const
-    {
-      return *_order.at(n);
-    }
-
-    iterator begin()
-    {
-      return _order.begin();
-    }
-    iterator end()
-    {
-      return _order.end();
-    }
-    const_iterator begin() const
-    {
-      return _order.begin();
-    }
-    const_iterator end() const
-    {
-      return _order.end();
-    }
-
-    reverse_iterator rbegin()
-    {
-      return _order.rbegin();
-    }
-    reverse_iterator rend()
-    {
-      return _order.rend();
-    }
-    const_reverse_iterator rbegin() const
-    {
-      return _order.rbegin();
-    }
-    const_reverse_iterator rend() const
-    {
-      return _order.rend();
-    }
-
-    value_type& front()
-    {
-      return *_order.front();
-    }
-
-    value_type& back()
-    {
-      return *_order.back();
-    }
-
-    const value_type& front() const
-    {
-      return *_order.front();
-    }
-
-    const value_type& back() const
-    {
-      return *_order.back();
-    }
-
-    std::vector<std::unique_ptr<value_type>>& underlying() {
-      return _order;
-    }
-  };
-
-  template<typename T, typename T2>
-  std::unique_ptr<T> erase_this(ptr_vec<T>& vec, T2* el)
-  {
-    return vec.erase(*el);
-  }
-
-  template<typename T, typename T2>
-  std::unique_ptr<T> erase_this(ptr_vec<T>& vec, T2& el)
-  {
-    return vec.erase(el);
-  }
-
-  template<typename T>
-  struct non_null_ptr {
-    non_null_ptr() = delete;
-    constexpr non_null_ptr(T* ptr) : _ptr(ptr)
-    {
-      assert(ptr != nullptr);
-    }
-    non_null_ptr(std::nullptr_t) = delete;
-
-    constexpr non_null_ptr(const non_null_ptr&) = default;
-    constexpr non_null_ptr(non_null_ptr&&) = default;
-    constexpr non_null_ptr& operator=(const non_null_ptr&) = default;
-    constexpr non_null_ptr& operator=(non_null_ptr&&) = default;
-
-    constexpr T& operator*() const noexcept
-    {
-      return *_ptr;
-    }
-
-    constexpr T* operator->() const noexcept
-    {
-      return _ptr;
-    }
-
-    constexpr operator T*() noexcept
-    {
-      return _ptr;
-    }
-
-    constexpr operator T* const() const noexcept
-    {
-      return _ptr;
-    }
-
-  private:
-    T* _ptr;
-  };
-
-  template<typename T>
-  struct ref_vec {
-    using value_type = T;
-
-    std::vector<value_type*> _order;
-
-    using iterator = double_iterator<typename decltype(_order)::iterator>;
-    using const_iterator = double_iterator<typename decltype(_order)::const_iterator>;
-
-    using reverse_iterator = double_iterator<typename decltype(_order)::reverse_iterator>;
-    using const_reverse_iterator =
-      double_iterator<typename decltype(_order)::const_reverse_iterator>;
-
-    ref_vec() = default;
-
-    ref_vec(std::initializer_list<value_type*> lst) : _order {lst} { };
-
-    template<typename InputIter, typename = std::enable_if_t<std::is_same_v<decltype(*std::declval<InputIter>()), value_type&>>>
-    ref_vec(InputIter iter1, InputIter iter2) {
-      _order.reserve(std::distance(iter1, iter2));
-      std::transform(iter1, iter2, std::back_inserter(_order), [] (auto& v) {return &v; });      
-    }
-
-    template<typename Range, typename  = std::enable_if_t<std::is_same_v<decltype(*std::declval<Range>().begin()), value_type&>>>
-    ref_vec(Range&& rng) : ref_vec (std::begin(rng), std::end(rng)) { }
-
-    value_type& push_back(value_type& v)
-    {
-      _order.push_back(&v);
-      return v;
-    }
-
-    value_type& push_back(non_null_ptr<value_type> ptr)
-    {
-      _order.push_back(ptr);
-      return *ptr;
-    }
-
-    value_type& emplace_back(value_type& v)
-    {
-      return push_back(v);
-    }
-
-    std::unique_ptr<value_type> erase(const value_type& v)
-    {
-      auto iter =
-        std::find_if(_order.begin(), _order.end(), [&v](auto&& ptr) { return ptr == &v; });
-      if (iter != _order.end()) {
-        auto uptr = std::move(*iter);
-        _order.erase(iter);
-        return uptr;
-      }
-      return nullptr;
-    }
-
-    iterator rotate_to_back(const value_type& v)
-    {
-      auto iter =
-        std::find_if(_order.begin(), _order.end(), [&v](auto&& ptr) { return ptr == &v; });
-      return rotate_to_back(iter);
-    }
-
-    iterator rotate_to_back(iterator iter)
-    {
-      if (iter != _order.end()) {
-        {
-          return std::rotate(iter.data(), iter.data() + 1, _order.end());
-        }
-      }
-      return end();
-    }
-
-    iterator rotate_to_front(const value_type& v)
-    {
-      auto iter =
-        std::find_if(_order.begin(), _order.end(), [&v](auto&& ptr) { return ptr == &v; });
-      return rotate_to_front(iter);
-    }
-
-    iterator rotate_to_front(iterator iter)
-    {
-      if (iter != _order.end()) {
-        {
-          return std::rotate(_order.begin(), iter.data(), iter.data() + 1);
-        }
-      }
-      return end();
-    }
-
-    std::size_t size() const noexcept
-    {
-      return _order.size();
-    }
-
-    bool empty() const noexcept
-    {
-      return _order.empty();
-    }
-
-    std::size_t capacity() const noexcept
-    {
-      return _order.capacity();
-    }
-
-    std::size_t max_size() const noexcept
-    {
-      return _order.max_size();
-    }
-
-    void reserve(std::size_t new_cap) 
-    {
-      _order.reserve(new_cap);
-    }
-
-    void shrink_to_fit()
-    {
-      _order.shrink_to_fit();
-    }
-
-    value_type& operator[](std::size_t n)
-    {
-      return *_order[n];
-    }
-
-    const value_type& operator[](std::size_t n) const
-    {
-      return *_order[n];
-    }
-
-    value_type& at(std::size_t n)
-    {
-      return *_order.at(n);
-    }
-
-    const value_type& at(std::size_t n) const
-    {
-      return *_order.at(n);
-    }
-
-    iterator begin()
-    {
-      return _order.begin();
-    }
-    iterator end()
-    {
-      return _order.end();
-    }
-    const_iterator begin() const
-    {
-      return _order.begin();
-    }
-    const_iterator end() const
-    {
-      return _order.end();
-    }
-
-    reverse_iterator rbegin()
-    {
-      return _order.rbegin();
-    }
-    reverse_iterator rend()
-    {
-      return _order.rend();
-    }
-    const_reverse_iterator rbegin() const
-    {
-      return _order.rbegin();
-    }
-    const_reverse_iterator rend() const
-    {
-      return _order.rend();
-    }
-
-    value_type& front()
-    {
-      return *_order.front();
-    }
-
-    value_type& back()
-    {
-      return *_order.back();
-    }
-
-    const value_type& front() const
-    {
-      return *_order.front();
-    }
-
-    const value_type& back() const
-    {
-      return *_order.back();
-    }
-
-    std::vector<value_type*>& underlying() {
-      return _order;
-    }
-  };
-
-
-} // namespace waybar::util

meson.build

@@ -1,8 +1,12 @@
 project(
     'waybar', 'cpp', 'c',
-    version: '0.0.1',
+    version: '0.2.1',
     license: 'MIT',
-    default_options : ['cpp_std=c++17'],
+    default_options : [
+        'cpp_std=c++17',
+        'buildtype=release',
+        'default_library=static'
+    ],
 )
 
 cpp_args = []
@@ -14,32 +18,82 @@ if false # libc++
 
     cpp_link_args += ['-lc++fs']
 else
-    # TODO: For std::filesystem in libstdc++. Still unstable? Or why is it not in libstdc++ proper yet?
     cpp_link_args += ['-lstdc++fs']
 endif
 
+compiler = meson.get_compiler('cpp')
+
+if not compiler.has_header('filesystem')
+    add_project_arguments('-DFILESYSTEM_EXPERIMENTAL', language: 'cpp')
+endif
+
 add_global_arguments(cpp_args, language : 'cpp')
 add_global_link_arguments(cpp_link_args, language : 'cpp')
 
 thread_dep = dependency('threads')
 libinput = dependency('libinput')
-fmt = dependency('fmt', fallback: ['fmtlib', 'fmt_dep'])
+fmt = dependency('fmt', version : ['>=5.2.1'], fallback : ['fmt', 'fmt_dep'])
 wayland_client = dependency('wayland-client')
 wayland_cursor = dependency('wayland-cursor')
 wayland_protos = dependency('wayland-protocols')
 wlroots = dependency('wlroots', fallback: ['wlroots', 'wlroots'])
 gtkmm = dependency('gtkmm-3.0')
+dbusmenu_gtk = dependency('dbusmenu-gtk3-0.4', required: get_option('dbusmenu-gtk'))
+giounix = dependency('gio-unix-2.0', required: get_option('dbusmenu-gtk'))
 jsoncpp = dependency('jsoncpp')
 sigcpp = dependency('sigc++-2.0')
-libnl = dependency('libnl-3.0')
-libnlgen = dependency('libnl-genl-3.0')
-libpulse = dependency('libpulse')
+libnl = dependency('libnl-3.0', required: get_option('libnl'))
+libnlgen = dependency('libnl-genl-3.0', required: get_option('libnl'))
+libpulse = dependency('libpulse', required: get_option('pulseaudio'))
+
+src_files = files(
+    'src/factory.cpp',
+    'src/ALabel.cpp',
+    'src/modules/memory.cpp',
+    'src/modules/battery.cpp',
+    'src/modules/clock.cpp',
+    'src/modules/custom.cpp',
+    'src/modules/cpu.cpp',
+    'src/main.cpp',
+    'src/bar.cpp',
+    'src/client.cpp'
+)
+
+if find_program('sway', required : false).found()
+    add_project_arguments('-DHAVE_SWAY', language: 'cpp')
+    src_files += [
+        'src/modules/sway/ipc/client.cpp',
+        'src/modules/sway/mode.cpp',
+        'src/modules/sway/window.cpp',
+        'src/modules/sway/workspaces.cpp'
+    ]
+endif
+
+if libnl.found() and libnlgen.found()
+    add_project_arguments('-DHAVE_LIBNL', language: 'cpp')
+    src_files += 'src/modules/network.cpp'
+endif
+
+if libpulse.found()
+    add_project_arguments('-DHAVE_LIBPULSE', language: 'cpp')
+    src_files += 'src/modules/pulseaudio.cpp'
+endif
+
+if dbusmenu_gtk.found()
+    add_project_arguments('-DHAVE_DBUSMENU', language: 'cpp')
+    src_files += files(
+        'src/modules/sni/tray.cpp',
+        'src/modules/sni/snw.cpp',
+        'src/modules/sni/snh.cpp',
+        'src/modules/sni/sni.cpp'
+    )
+endif
 
 subdir('protocol')
 
 executable(
     'waybar',
-    run_command('find', './src', '-name', '*.cpp').stdout().strip().split('\n'),
+    src_files,
     dependencies: [
         thread_dep,
         wlroots,
@@ -51,9 +105,11 @@ executable(
         libinput,
         wayland_cursor,
         gtkmm,
+        dbusmenu_gtk,
+        giounix,
         libnl,
         libnlgen,
-        libpulse,
+        libpulse
     ],
     include_directories: [include_directories('include')],
     install: true,
@@ -62,5 +118,17 @@ executable(
 install_data(
     './resources/config',
     './resources/style.css',
-    install_dir: '/etc/xdg/waybar',
+    install_dir: join_paths(get_option('out'), 'etc/xdg/waybar')
 )
+
+clangtidy = find_program('clang-tidy', required: false)
+
+if clangtidy.found()
+    run_target(
+        'tidy',
+        command: [
+            clangtidy,
+            '-checks=*,-fuchsia-default-arguments',
+            '-p', meson.build_root()
+        ] + src_files)
+endif

meson_options.txt

@@ -0,0 +1,4 @@
+option('libnl', type: 'feature', value: 'auto', description: 'Enable libnl support for network related features')
+option('pulseaudio', type: 'feature', value: 'auto', description: 'Enable support for pulseaudio')
+option('dbusmenu-gtk', type: 'feature', value: 'auto', description: 'Enable support for tray')
+option('out', type: 'string', value : '/', description: 'output prefix directory')

protocol/dbus-menu.xml

@@ -0,0 +1,69 @@
+<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
+"http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
+<node>
+<interface name="com.canonical.dbusmenu">
+    <!-- Properties -->
+    <property name="Version" type="u" access="read" />
+    <property name="TextDirection" type="s" access="read" />
+    <property name="Status" type="s" access="read" />
+    <property name="IconThemePath" type="as" access="read" />
+
+    <!-- Functions -->
+    <method name="GetLayout">
+        <arg type="i" name="parentId" direction="in" />
+        <arg type="i" name="recursionDepth" direction="in" />
+        <arg type="as" name="propertyNames" direction="in" />
+        <arg type="u" name="revision" direction="out" />
+        <arg type="(ia{sv}av)" name="layout" direction="out" />
+    </method>
+
+    <method name="GetGroupProperties">
+        <arg type="ai" name="ids" direction="in" />
+        <arg type="as" name="propertyNames" direction="in" />
+        <arg type="a(ia{sv})" name="properties" direction="out" />
+    </method>
+
+    <method name="GetProperty">
+        <arg type="i" name="id" direction="in" />
+        <arg type="s" name="name" direction="in" />
+        <arg type="v" name="value" direction="out" />
+    </method>
+
+    <method name="Event">
+        <arg type="i" name="id" direction="in" />
+        <arg type="s" name="eventId" direction="in" />
+        <arg type="v" name="data" direction="in" />
+        <arg type="u" name="timestamp" direction="in" />
+    </method>
+
+    <method name="EventGroup">
+        <arg type="a(isvu)" name="events" direction="in" />
+        <arg type="ai" name="idErrors" direction="out" />
+    </method>
+
+    <method name="AboutToShow">
+        <arg type="i" name="id" direction="in" />
+        <arg type="b" name="needUpdate" direction="out" />
+    </method>
+
+    <method name="AboutToShowGroup">
+        <arg type="ai" name="ids" direction="in" />
+        <arg type="ai" name="updatesNeeded" direction="out" />
+        <arg type="ai" name="idErrors" direction="out" />
+    </method>
+
+    <!-- Signals -->
+    <signal name="ItemsPropertiesUpdated">
+        <arg type="a(ia{sv})" name="updatedProps" direction="out" />
+        <arg type="a(ias)" name="removedProps" direction="out" />
+    </signal>
+    <signal name="LayoutUpdated">
+        <arg type="u" name="revision" direction="out" />
+        <arg type="i" name="parent" direction="out" />
+    </signal>
+    <signal name="ItemActivationRequested">
+        <arg type="i" name="id" direction="out" />
+        <arg type="u" name="timestamp" direction="out" />
+    </signal>
+</interface>
+</node>

protocol/dbus-status-notifier-item.xml

@@ -0,0 +1,78 @@
+<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
+"http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
+<node>
+<interface name="org.kde.StatusNotifierItem">
+    <annotation name="org.gtk.GDBus.C.Name" value="Item" />
+    <property name="Category" type="s" access="read"/>
+    <property name="Id" type="s" access="read"/>
+    <property name="Title" type="s" access="read"/>
+    <property name="Status" type="s" access="read"/>
+    <property name="WindowId" type="i" access="read"/>
+    <property name="Menu" type="o" access="read" />
+
+    <!-- main icon -->
+    <!-- names are preferred over pixmaps -->
+    <property name="IconName" type="s" access="read" />
+    <property name="IconThemePath" type="s" access="read" />
+
+    <!-- struct containing width, height and image data-->
+    <!-- implementation has been dropped as of now -->
+    <property name="IconPixmap" type="a(iiay)" access="read" />
+
+    <!-- not used in ayatana code, no test case so far -->
+    <property name="OverlayIconName" type="s" access="read"/>
+    <property name="OverlayIconPixmap" type="a(iiay)" access="read" />
+
+    <!-- Requesting attention icon -->
+    <property name="AttentionIconName" type="s" access="read"/>
+
+    <!--same definition as image-->
+    <property name="AttentionIconPixmap" type="a(iiay)" access="read" />
+
+    <!-- tooltip data -->
+    <!-- unimplemented as of now -->
+    <!--(iiay) is an image-->
+    <property name="ToolTip" type="(sa(iiay)ss)" access="read" />
+
+
+    <!-- interaction: actually, we do not use them. -->
+    <method name="Activate">
+        <arg name="x" type="i" direction="in"/>
+        <arg name="y" type="i" direction="in"/>
+    </method>
+    <method name="SecondaryActivate">
+        <arg name="x" type="i" direction="in"/>
+        <arg name="y" type="i" direction="in"/>
+    </method>
+    <method name="Scroll">
+        <arg name="delta" type="i" direction="in"/>
+        <arg name="dir"   type="s" direction="in"/>
+    </method>
+
+    <!-- Signals: the client wants to change something in the status-->
+    <signal name="NewTitle"></signal>
+    <signal name="NewIcon"></signal>
+    <signal name="NewIconThemePath">
+        <arg type="s" name="icon_theme_path" direction="out" />
+    </signal>
+    <signal name="NewAttentionIcon"></signal>
+    <signal name="NewOverlayIcon"></signal>
+    <signal name="NewToolTip"></signal>
+    <signal name="NewStatus">
+        <arg name="status" type="s" />
+    </signal>
+
+    <!-- ayatana labels -->
+    <!-- These are commented out because GDBusProxy would otherwise require them,
+         but they are not available for KDE indicators
+    -->
+    <!--<signal name="XAyatanaNewLabel">
+        <arg type="s" name="label" direction="out" />
+        <arg type="s" name="guide" direction="out" />
+    </signal>
+    <property name="XAyatanaLabel" type="s" access="read" />
+    <property name="XAyatanaLabelGuide" type="s" access="read" />-->
+
+
+</interface>
+</node>

protocol/dbus-status-notifier-watcher.xml

@@ -0,0 +1,52 @@
+<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN" "http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
+<node>
+  <interface name="org.kde.StatusNotifierWatcher">
+    <annotation name="org.gtk.GDBus.C.Name" value="Watcher" />
+
+    <!-- methods -->
+    <method name="RegisterStatusNotifierItem">
+      <annotation name="org.gtk.GDBus.C.Name" value="RegisterItem" />
+      <arg name="service" type="s" direction="in"/>
+    </method>
+
+    <method name="RegisterStatusNotifierHost">
+        <annotation name="org.gtk.GDBus.C.Name" value="RegisterHost" />
+        <arg name="service" type="s" direction="in"/>
+    </method>
+
+
+    <!-- properties -->
+
+    <property name="RegisteredStatusNotifierItems" type="as" access="read">
+      <annotation name="org.gtk.GDBus.C.Name" value="RegisteredItems" />
+      <annotation name="org.qtproject.QtDBus.QtTypeName.Out0" value="QStringList"/>
+    </property>
+
+    <property name="IsStatusNotifierHostRegistered" type="b" access="read">
+      <annotation name="org.gtk.GDBus.C.Name" value="IsHostRegistered" />
+    </property>
+
+    <property name="ProtocolVersion" type="i" access="read"/>
+
+
+    <!-- signals -->
+
+    <signal name="StatusNotifierItemRegistered">
+      <annotation name="org.gtk.GDBus.C.Name" value="ItemRegistered" />
+      <arg type="s" direction="out" name="service" />
+    </signal>
+
+    <signal name="StatusNotifierItemUnregistered">
+      <annotation name="org.gtk.GDBus.C.Name" value="ItemUnregistered" />
+      <arg type="s" direction="out" name="service" />
+    </signal>
+
+    <signal name="StatusNotifierHostRegistered">
+      <annotation name="org.gtk.GDBus.C.Name" value="HostRegistered" />
+    </signal>
+
+    <signal name="StatusNotifierHostUnregistered">
+      <annotation name="org.gtk.GDBus.C.Name" value="HostUnregistered" />
+    </signal>
+  </interface>
+</node>

protocol/idle.xml

@@ -1,49 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<protocol name="idle">
-  <copyright><![CDATA[
-    Copyright (C) 2015 Martin Gräßlin
-
-    This program is free software: you can redistribute it and/or modify
-    it under the terms of the GNU Lesser General Public License as published by
-    the Free Software Foundation, either version 2.1 of the License, or
-    (at your option) any later version.
-
-    This program is distributed in the hope that it will be useful,
-    but WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU Lesser General Public License for more details.
-
-    You should have received a copy of the GNU Lesser General Public License
-    along with this program.  If not, see <http://www.gnu.org/licenses/>.
-  ]]></copyright>
-  <interface  name="org_kde_kwin_idle" version="1">
-      <description summary="User idle time manager">
-        This interface allows to monitor user idle time on a given seat. The interface
-        allows to register timers which trigger after no user activity was registered
-        on the seat for a given interval. It notifies when user activity resumes.
-
-        This is useful for applications wanting to perform actions when the user is not
-        interacting with the system, e.g. chat applications setting the user as away, power
-        management features to dim screen, etc..
-      </description>
-      <request name="get_idle_timeout">
-        <arg name="id" type="new_id" interface="org_kde_kwin_idle_timeout"/>
-        <arg name="seat" type="object" interface="wl_seat"/>
-        <arg name="timeout" type="uint" summary="The idle timeout in msec"/>
-      </request>
-  </interface>
-  <interface name="org_kde_kwin_idle_timeout" version="1">
-      <request name="release" type="destructor">
-        <description summary="release the timeout object"/>
-      </request>
-      <request name="simulate_user_activity">
-          <description summary="Simulates user activity for this timeout, behaves just like real user activity on the seat"/>
-      </request>
-      <event name="idle">
-          <description summary="Triggered when there has not been any user activity in the requested idle time interval"/>
-      </event>
-      <event name="resumed">
-          <description summary="Triggered on the first user activity after an idle event"/>
-      </event>
-  </interface>
-</protocol>

protocol/meson.build

@@ -23,8 +23,8 @@ wayland_scanner_client = generator(
 
 client_protocols = [
 	[wl_protocol_dir, 'stable/xdg-shell/xdg-shell.xml'],
+	[wl_protocol_dir, 'unstable/xdg-output/xdg-output-unstable-v1.xml'],
 	['wlr-layer-shell-unstable-v1.xml'],
-	['idle.xml'],
 ]
 
 client_protos_src = []
@@ -36,10 +36,66 @@ foreach p : client_protocols
 	client_protos_headers += wayland_scanner_client.process(xml)
 endforeach
 
+gdbus_codegen = find_program('gdbus-codegen')
+
+r = run_command(gdbus_codegen, '--body', '--output', '/dev/null')
+if r.returncode() != 0
+  gdbus_code_dsnw = custom_target(
+      'dbus-status-notifier-watcher.[ch]',
+      output: ['@BASENAME@.c','@BASENAME@.h'],
+      input: './dbus-status-notifier-watcher.xml',
+      command: [gdbus_codegen,'--c-namespace', 'Sn', '--generate-c-code', 'protocol/@BASENAME@', '@INPUT@'],
+  )
+
+  gdbus_code_dsni = custom_target(
+      'dbus-status-notifier-item.[ch]',
+      output: ['@BASENAME@.c','@BASENAME@.h'],
+      input: './dbus-status-notifier-item.xml',
+      command: [gdbus_codegen,'--c-namespace', 'Sn', '--generate-c-code', 'protocol/@BASENAME@', '@INPUT@'],
+  )
+
+  gdbus_code_dm = custom_target(
+      'dbus-menu.[ch]',
+      output: ['@BASENAME@.c','@BASENAME@.h'],
+      input: './dbus-menu.xml',
+      command: [gdbus_codegen,'--c-namespace', 'Sn', '--generate-c-code', 'protocol/@BASENAME@', '@INPUT@'],
+  )
+
+  client_protos_src += gdbus_code_dsnw[0]
+  client_protos_headers += gdbus_code_dsnw[1]
+  client_protos_src += gdbus_code_dsni[0]
+  client_protos_headers += gdbus_code_dsni[1]
+  client_protos_src += gdbus_code_dm[0]
+  client_protos_headers += gdbus_code_dm[1]
+else
+  gdbus_code = generator(
+      gdbus_codegen,
+      output: '@BASENAME@.c',
+      arguments: ['--c-namespace', 'Sn', '--body', '--output', '@OUTPUT@', '@INPUT@']
+  )
+
+  gdbus_header = generator(
+      gdbus_codegen,
+      output: '@BASENAME@.h',
+      arguments: ['--c-namespace', 'Sn', '--header', '--output', '@OUTPUT@', '@INPUT@']
+  )
+
+  client_protos_src += gdbus_code.process('./dbus-status-notifier-watcher.xml')
+  client_protos_headers += gdbus_header.process('./dbus-status-notifier-watcher.xml')
+
+  client_protos_src += gdbus_code.process('./dbus-status-notifier-item.xml')
+  client_protos_headers += gdbus_header.process('./dbus-status-notifier-item.xml')
+
+  client_protos_src += gdbus_code.process('./dbus-menu.xml')
+  client_protos_headers += gdbus_header.process('./dbus-menu.xml')
+endif
+
+
 lib_client_protos = static_library(
 	'client_protos',
 	client_protos_src + client_protos_headers,
-	dependencies: [wayland_client]
+	dependencies: [wayland_client, gtkmm, giounix],
+	include_directories: include_directories('..'),
 ) # for the include directory
 
 client_protos = declare_dependency(

resources/config

@@ -1,25 +1,88 @@
 {
-    "modules-left": ["workspaces", "custom/spotify"],
-    "modules-right": ["pulseaudio", "network", "cpu", "memory", "battery", "clock"],
+    "layer": "top", // Waybar at top layer
+    // "position": "bottom", // Waybar at the bottom of your screen
+    // "height": 30, // Waybar height
+    // "width": 1280, // Waybar width
+    // Choose the order of the modules
+    "modules-left": ["sway/workspaces", "sway/mode", "custom/spotify"],
+    "modules-center": ["sway/window"],
+    "modules-right": ["pulseaudio", "network", "cpu", "memory", "battery", "battery#bat2", "clock", "tray"],
+    // Modules configuration
+    // "sway/workspaces": {
+    //     "disable-scroll": true,
+    //     "all-outputs": true,
+    //     "format": "{name}: {icon}",
+    //     "format-icons": {
+    //         "1": "",
+    //         "2": "",
+    //         "3": "",
+    //         "4": "",
+    //         "5": "",
+    //         "urgent": "",
+    //         "focused": "",
+    //         "default": ""
+    //     }
+    // },
+    "sway/mode": {
+        "format": "{}"
+    },
+    "sway/window": {
+        "max-length": 50
+    },
+    "tray": {
+        // "icon-size": 21,
+        "spacing": 10
+    },
+    "clock": {
+        "format-alt": "{:%Y-%m-%d}"
+    },
     "cpu": {
-        "format": "{}% "
+        "format": "{usage}% "
     },
     "memory": {
         "format": "{}% "
     },
     "battery": {
-        "format": "{}% "
+        "states": {
+            // "good": 95,
+            "warning": 30,
+            "critical": 15
+        },
+        "format": "{capacity}% {icon}",
+        // "format-good": "", // An empty format will hide the module
+        // "format-full": "",
+        "format-icons": ["", "", "", "", ""]
+    },
+    "battery#bat2": {
+        "bat": "BAT2"
     },
     "network": {
-        "interface": "wlp2s0",
-        "format": "{essid} ({signalStrength}%) "
+        // "interface": "wlp2s0", // (Optional) To force the use of this interface
+        "format-wifi": "{essid} ({signalStrength}%) ",
+        "format-ethernet": "{ifname}: {ipaddr}/{cidr} ",
+        "format-disconnected": "Disconnected ⚠"
     },
     "pulseaudio": {
-        "format": "{}% ",
-        "format-muted": ""
+        //"scroll-step": 1,
+        "format": "{volume}% {icon}",
+        "format-bluetooth": "{volume}% {icon}",
+        "format-muted": "",
+        "format-icons": {
+            "headphones": "",
+            "handsfree": "",
+            "headset": "",
+            "phone": "",
+            "portable": "",
+            "car": "",
+            "default": ["", ""]
+        },
+        "on-click": "pavucontrol"
     },
     "custom/spotify": {
         "format": " {}",
-        "exec": "$HOME/.bin/mediaplayer.sh"
+        "max-length": 40,
+        "interval": 30, // Remove this if your script is endless and write in loop
+        "exec": "$HOME/.config/waybar/mediaplayer.sh 2> /dev/null", // Script in resources folder
+        "exec-if": "pgrep spotify"
     }
 }

resources/custom_modules/mediaplayer.sh

@@ -0,0 +1,7 @@
+#!/bin/sh
+player_status=$(playerctl status 2> /dev/null)
+if [ "$player_status" = "Playing" ]; then
+    echo "$(playerctl metadata artist) - $(playerctl metadata title)"
+elif [ "$player_status" = "Paused" ]; then
+    echo " $(playerctl metadata artist) - $(playerctl metadata title)"
+fi

resources/style.css

@@ -3,69 +3,100 @@
     border-radius: 0;
     font-family: Roboto, Helvetica, Arial, sans-serif;
     font-size: 13px;
+    min-height: 0;
 }
 
-window {
+window#waybar {
     background: rgba(43, 48, 59, 0.5);
     border-bottom: 3px solid rgba(100, 114, 125, 0.5);
     color: white;
 }
 
-.workspaces button {
+#workspaces button {
     padding: 0 5px;
     background: transparent;
     color: white;
     border-bottom: 3px solid transparent;
 }
 
-.workspaces button.current {
+#workspaces button.focused {
     background: #64727D;
     border-bottom: 3px solid white;
 }
 
-.clock, .battery, .cpu, .memory, .network, .pulseaudio, .custom-spotify {
+#mode {
+    background: #64727D;
+    border-bottom: 3px solid white;
+}
+
+#clock, #battery, #cpu, #memory, #network, #pulseaudio, #custom-spotify, #tray, #mode {
     padding: 0 10px;
     margin: 0 5px;
 }
 
-.clock {
+#clock {
     background-color: #64727D;
 }
 
-.battery {
+#battery {
     background-color: #ffffff;
     color: black;
 }
 
-.battery.charging {
+#battery.charging {
     color: white;
     background-color: #26A65B;
 }
 
-.cpu {
+@keyframes blink {
+    to {
+        background-color: #ffffff;
+        color: black;
+    }
+}
+
+#battery.warning:not(.charging) {
+    background: #f53c3c;
+    color: white;
+    animation-name: blink;
+    animation-duration: 0.5s;
+    animation-timing-function: linear;
+    animation-iteration-count: infinite;
+    animation-direction: alternate;
+}
+
+#cpu {
     background: #2ecc71;
     color: #000000;
 }
 
-.memory {
+#memory {
     background: #9b59b6;
 }
 
-.network {
+#network {
     background: #2980b9;
 }
 
-.pulseaudio {
+#network.disconnected {
+    background: #f53c3c;
+}
+
+#pulseaudio {
     background: #f1c40f;
     color: black;
 }
 
-.pulseaudio.muted {
+#pulseaudio.muted {
     background: #90b1b1;
     color: #2a5c45;
 }
 
-.custom-spotify {
+#custom-spotify {
     background: #66cc99;
     color: #2a5c45;
 }
+
+#tray {
+    background-color: #2980b9;
+}

src/ALabel.cpp

@@ -0,0 +1,112 @@
+#include "ALabel.hpp"
+#include <util/command.hpp>
+
+#include <iostream>
+
+waybar::ALabel::ALabel(const Json::Value& config, const std::string format)
+  : config_(config),
+    format_(config_["format"].isString() ? config_["format"].asString() : format),
+    default_format_(format_)
+{
+  event_box_.add(label_);
+	if (config_["max-length"].isUInt()) {
+    label_.set_max_width_chars(config_["max-length"].asUInt());
+    label_.set_ellipsize(Pango::EllipsizeMode::ELLIPSIZE_END);
+  }
+  if (config_["format-alt"].isString()) {
+    event_box_.add_events(Gdk::BUTTON_PRESS_MASK);
+    event_box_.signal_button_press_event().connect(
+        sigc::mem_fun(*this, &ALabel::handleToggle));
+  }
+
+  // configure events' user commands
+  if (config_["on-click"].isString()) {
+    event_box_.add_events(Gdk::BUTTON_PRESS_MASK);
+    event_box_.signal_button_press_event().connect(
+      sigc::mem_fun(*this, &ALabel::handleToggle));
+  }
+  if (config_["on-scroll-up"].isString()) {
+    event_box_.add_events(Gdk::SCROLL_MASK);
+    event_box_.signal_scroll_event().connect(
+      sigc::mem_fun(*this, &ALabel::handleScroll));
+  }
+  if (config_["on-scroll-down"].isString()) {
+    event_box_.add_events(Gdk::SCROLL_MASK);
+    event_box_.signal_scroll_event().connect(
+      sigc::mem_fun(*this, &ALabel::handleScroll));
+  }
+}
+
+auto waybar::ALabel::update() -> void {
+  // Nothing here
+}
+
+bool waybar::ALabel::handleToggle(GdkEventButton* const& e) {
+  if (config_["on-click"].isString() && e->button == 1) {
+    waybar::util::command::forkExec(config_["on-click"].asString());
+  } else {
+    alt = !alt;
+    if (alt) {
+      format_ = config_["format-alt"].asString();
+    } else {
+      format_ = default_format_;
+    }
+  }
+
+  dp.emit();
+  return true;
+}
+
+bool waybar::ALabel::handleScroll(GdkEventScroll* e) {
+
+  // Avoid concurrent scroll event
+  std::lock_guard<std::mutex> lock(mutex_);
+  bool direction_up = false;
+
+  if (e->direction == GDK_SCROLL_UP) {
+    direction_up = true;
+  }
+  if (e->direction == GDK_SCROLL_DOWN) {
+    direction_up = false;
+  }
+  if (e->direction == GDK_SCROLL_SMOOTH) {
+    gdouble delta_x, delta_y;
+    gdk_event_get_scroll_deltas(reinterpret_cast<const GdkEvent*>(e),
+                                &delta_x, &delta_y);
+    if (delta_y < 0) {
+      direction_up = true;
+    } else if (delta_y > 0) {
+      direction_up = false;
+    }
+  }
+  if (direction_up && config_["on-scroll-up"].isString()) {
+    waybar::util::command::forkExec(config_["on-scroll-up"].asString());
+  } else if (config_["on-scroll-down"].isString()) {
+    waybar::util::command::forkExec(config_["on-scroll-down"].asString());
+  }
+  dp.emit();
+  return true;
+}
+
+std::string waybar::ALabel::getIcon(uint16_t percentage,
+                                    const std::string& alt) {
+  auto format_icons = config_["format-icons"];
+  if (format_icons.isObject()) {
+    if (!alt.empty() && format_icons[alt].isString()) {
+      format_icons = format_icons[alt];
+    } else {
+      format_icons = format_icons["default"];
+    }
+  }
+  if (format_icons.isArray()) {
+    auto size = format_icons.size();
+    auto idx = std::clamp(percentage / (100 / size), 0U, size - 1);
+    format_icons = format_icons[idx];
+  }
+  if (format_icons.isString()) {
+    return format_icons.asString();
+  }
+  return "";
+}
+
+waybar::ALabel::operator Gtk::Widget&() { return event_box_; }

src/bar.cpp

@@ -1,185 +1,221 @@
-#include <condition_variable>
-#include <gdk/gdkwayland.h>
-#include <thread>
-#include <fstream>
 #include "bar.hpp"
 #include "client.hpp"
 #include "factory.hpp"
-#include "util/chrono.hpp"
+#include "util/json.hpp"
 
-waybar::Bar::Bar(Client &client, std::unique_ptr<struct wl_output *> &&p_output)
+waybar::Bar::Bar(const Client& client,
+  std::unique_ptr<struct wl_output *> &&p_output, uint32_t p_wl_name)
   : client(client), window{Gtk::WindowType::WINDOW_TOPLEVEL},
-    output(std::move(p_output))
+    surface(nullptr), layer_surface(nullptr),
+    output(std::move(p_output)), wl_name(p_wl_name)
 {
-  static const struct wl_output_listener outputListener = {
-    .geometry = _handleGeometry,
-    .mode = _handleMode,
-    .done = _handleDone,
-    .scale = _handleScale,
+  static const struct zxdg_output_v1_listener xdgOutputListener = {
+    .logical_position = handleLogicalPosition,
+    .logical_size = handleLogicalSize,
+    .done = handleDone,
+    .name = handleName,
+    .description = handleDescription,
   };
-
-  wl_output_add_listener(*output, &outputListener, this);
+  xdg_output_ =
+    zxdg_output_manager_v1_get_xdg_output(client.xdg_output_manager, *output);
+	zxdg_output_v1_add_listener(xdg_output_, &xdgOutputListener, this);
   window.set_title("waybar");
+  window.set_name("waybar");
   window.set_decorated(false);
-  _setupConfig();
-  _setupCss();
-  _setupWidgets();
-  bool positionBottom = (_config["position"] == "bottom");
-  gtk_widget_realize(GTK_WIDGET(window.gobj()));
-  GdkWindow *gdkWindow = gtk_widget_get_window(GTK_WIDGET(window.gobj()));
-  gdk_wayland_window_set_use_custom_surface(gdkWindow);
-  surface = gdk_wayland_window_get_wl_surface(gdkWindow);
-  layerSurface = zwlr_layer_shell_v1_get_layer_surface(
-    client.layer_shell, surface, *output,
-    (positionBottom ? ZWLR_LAYER_SHELL_V1_LAYER_BOTTOM : ZWLR_LAYER_SHELL_V1_LAYER_TOP),
-    "waybar");
-  zwlr_layer_surface_v1_set_anchor(layerSurface,
-    ZWLR_LAYER_SURFACE_V1_ANCHOR_LEFT | ZWLR_LAYER_SURFACE_V1_ANCHOR_RIGHT |
-    (positionBottom ? ZWLR_LAYER_SURFACE_V1_ANCHOR_BOTTOM : ZWLR_LAYER_SURFACE_V1_ANCHOR_TOP));
-  zwlr_layer_surface_v1_set_size(layerSurface, _width, client.height);
-  static const struct zwlr_layer_surface_v1_listener layerSurfaceListener = {
-    .configure = _layerSurfaceHandleConfigure,
-    .closed = _layerSurfaceHandleClosed,
+  window.set_resizable(false);
+  setupConfig();
+  setupCss();
+
+  auto wrap = reinterpret_cast<GtkWidget*>(window.gobj());
+  gtk_widget_realize(wrap);
+  GdkWindow *gdk_window = gtk_widget_get_window(wrap);
+  gdk_wayland_window_set_use_custom_surface(gdk_window);
+  surface = gdk_wayland_window_get_wl_surface(gdk_window);
+
+  std::size_t layer_top = config_["layer"] == "top"
+    ? ZWLR_LAYER_SHELL_V1_LAYER_TOP : ZWLR_LAYER_SHELL_V1_LAYER_BOTTOM;
+  layer_surface = zwlr_layer_shell_v1_get_layer_surface(
+    client.layer_shell, surface, *output, layer_top, "waybar");
+
+  static const struct zwlr_layer_surface_v1_listener layer_surface_listener = {
+    .configure = layerSurfaceHandleConfigure,
+    .closed = layerSurfaceHandleClosed,
   };
-  zwlr_layer_surface_v1_add_listener(layerSurface, &layerSurfaceListener,
-    this);
+  zwlr_layer_surface_v1_add_listener(layer_surface,
+    &layer_surface_listener, this);
+
+  std::size_t anchor = ZWLR_LAYER_SURFACE_V1_ANCHOR_LEFT
+    | ZWLR_LAYER_SURFACE_V1_ANCHOR_RIGHT;
+  if (config_["position"] == "bottom") {
+    anchor |= ZWLR_LAYER_SURFACE_V1_ANCHOR_BOTTOM;
+  } else {
+    anchor |= ZWLR_LAYER_SURFACE_V1_ANCHOR_TOP;
+  }
+
+  auto height = config_["height"].isUInt() ? config_["height"].asUInt() : height_;
+  auto width = config_["width"].isUInt() ? config_["width"].asUInt() : width_;
+  zwlr_layer_surface_v1_set_anchor(layer_surface, anchor);
+  zwlr_layer_surface_v1_set_exclusive_zone(layer_surface, height);
+  zwlr_layer_surface_v1_set_size(layer_surface, width, height);
+
   wl_surface_commit(surface);
+
+  setupWidgets();
 }
 
-void waybar::Bar::_handleGeometry(void *data, struct wl_output *wl_output,
-  int32_t x, int32_t y, int32_t physical_width, int32_t physical_height,
-  int32_t subpixel, const char *make, const char *model, int32_t transform)
+void waybar::Bar::handleLogicalPosition(void* /*data*/,
+  struct zxdg_output_v1* /*zxdg_output_v1*/, int32_t /*x*/, int32_t /*y*/)
 {
   // Nothing here
 }
 
-void waybar::Bar::_handleMode(void *data, struct wl_output *wl_output,
-  uint32_t f, int32_t w, int32_t h, int32_t refresh)
-{
-  auto o = reinterpret_cast<waybar::Bar *>(data);
-  o->setWidth(w);
-}
-
-void waybar::Bar::_handleDone(void *data, struct wl_output *)
+void waybar::Bar::handleLogicalSize(void* /*data*/,
+  struct zxdg_output_v1* /*zxdg_output_v1*/, int32_t /*width*/,
+  int32_t /*height*/)
 {
   // Nothing here
 }
 
-void waybar::Bar::_handleScale(void *data, struct wl_output *wl_output,
-  int32_t factor)
+void waybar::Bar::handleDone(void* /*data*/,
+  struct zxdg_output_v1* /*zxdg_output_v1*/)
 {
   // Nothing here
 }
 
-void waybar::Bar::_layerSurfaceHandleConfigure(
-  void *data, struct zwlr_layer_surface_v1 *surface, uint32_t serial,
-  uint32_t width, uint32_t height)
+void waybar::Bar::handleName(void* data, struct zxdg_output_v1* /*xdg_output*/,
+  const char* name)
 {
-  auto o = reinterpret_cast<waybar::Bar *>(data);
+	auto o = static_cast<waybar::Bar *>(data);
+  o->output_name = name;
+}
+
+void waybar::Bar::handleDescription(void* /*data*/,
+  struct zxdg_output_v1* /*zxdg_output_v1*/, const char* /*description*/)
+{
+  // Nothing here
+}
+
+void waybar::Bar::layerSurfaceHandleConfigure(void* data,
+  struct zwlr_layer_surface_v1* surface, uint32_t serial, uint32_t width,
+  uint32_t height)
+{
+  auto o = static_cast<waybar::Bar *>(data);
   o->window.show_all();
   zwlr_layer_surface_v1_ack_configure(surface, serial);
-  if (o->client.height != height)
-  {
-    height = o->client.height;
-    std::cout << fmt::format("New Height: {}", height) << std::endl;
-    zwlr_layer_surface_v1_set_size(surface, width, height);
-    zwlr_layer_surface_v1_set_exclusive_zone(surface, o->visible ? height : 0);
+  if (width != o->width_ || height != o->height_) {
+    o->width_ = width;
+    o->height_ = height;
+    o->window.set_size_request(o->width_, o->height_);
+    o->window.resize(o->width_, o->height_);
+
+    int dummy_width, min_height;
+    o->window.get_size(dummy_width, min_height);
+    if (o->height_ < static_cast<uint32_t>(min_height)) {
+      std::cout << fmt::format("Requested height: {} exceeds the minimum \
+height: {} required by the modules", o->height_, min_height) << std::endl;
+      o->height_ = min_height;
+    }
+    std::cout << fmt::format(
+      "Bar configured (width: {}, height: {}) for output: {}",
+      o->width_, o->height_, o->output_name) << std::endl;
+
     wl_surface_commit(o->surface);
   }
 }
 
-void waybar::Bar::_layerSurfaceHandleClosed(void *data,
-  struct zwlr_layer_surface_v1 *surface)
+void waybar::Bar::layerSurfaceHandleClosed(void* data,
+  struct zwlr_layer_surface_v1* /*surface*/)
 {
-  auto o = reinterpret_cast<waybar::Bar *>(data);
-  zwlr_layer_surface_v1_destroy(o->layerSurface);
-  o->layerSurface = nullptr;
-  wl_surface_destroy(o->surface);
-  o->surface = nullptr;
-  o->window.close();
-}
-
-auto waybar::Bar::setWidth(int width) -> void
-{
-  if (width == this->_width) return;
-  std::cout << fmt::format("Bar width configured: {}", width) << std::endl;
-  this->_width = width;
-  window.set_size_request(width);
-  window.resize(width, client.height);
-  zwlr_layer_surface_v1_set_size(layerSurface, width, 40);
-  wl_surface_commit(surface);
+  auto o = static_cast<waybar::Bar *>(data);
+  std::cout << "Bar removed from output: " + o->output_name << std::endl;
+  zwlr_layer_surface_v1_destroy(o->layer_surface);
+  wl_output_destroy(*o->output);
+  zxdg_output_v1_destroy(o->xdg_output_);
+  o->modules_left_.clear();
+  o->modules_center_.clear();
+  o->modules_right_.clear();
 }
 
 auto waybar::Bar::toggle() -> void
 {
   visible = !visible;
-  auto zone = visible ? client.height : 0;
-  zwlr_layer_surface_v1_set_exclusive_zone(layerSurface, zone);
+  auto zone = visible ? height_ : 0;
+  zwlr_layer_surface_v1_set_exclusive_zone(layer_surface, zone);
   wl_surface_commit(surface);
 }
 
-auto waybar::Bar::_setupConfig() -> void
+auto waybar::Bar::setupConfig() -> void
 {
-  Json::Value root;
-  Json::CharReaderBuilder builder;
-  Json::CharReader* reader = builder.newCharReader();
-  std::string err;
-  std::ifstream file(client.configFile);
-  if (!file.is_open())
+  std::ifstream file(client.config_file);
+  if (!file.is_open()) {
     throw std::runtime_error("Can't open config file");
+  }
   std::string str((std::istreambuf_iterator<char>(file)),
     std::istreambuf_iterator<char>());
-  bool res = reader->parse(str.c_str(), str.c_str() + str.size(), &_config, &err);
-  delete reader;
-  if (!res)
-    throw std::runtime_error(err);
+  util::JsonParser parser;
+  config_ = parser.parse(str);
 }
 
-auto waybar::Bar::_setupCss() -> void
+auto waybar::Bar::setupCss() -> void
 {
-  _cssProvider = Gtk::CssProvider::create();
-  _styleContext = Gtk::StyleContext::create();
+  css_provider_ = Gtk::CssProvider::create();
+  style_context_ = Gtk::StyleContext::create();
 
-  // load our css file, wherever that may be hiding
-  if (_cssProvider->load_from_path(client.cssFile)) {
+  // Load our css file, wherever that may be hiding
+  if (css_provider_->load_from_path(client.css_file)) {
     Glib::RefPtr<Gdk::Screen> screen = window.get_screen();
-    _styleContext->add_provider_for_screen(screen, _cssProvider,
+    style_context_->add_provider_for_screen(screen, css_provider_,
       GTK_STYLE_PROVIDER_PRIORITY_USER);
   }
 }
 
-auto waybar::Bar::_setupWidgets() -> void
+void waybar::Bar::getModules(const Factory& factory, const std::string& pos)
+{
+  if (config_[pos].isArray()) {
+    for (const auto &name : config_[pos]) {
+      try {
+        auto module = factory.makeModule(name.asString());
+        if (pos == "modules-left") {
+          modules_left_.emplace_back(module);
+        }
+        if (pos == "modules-center") {
+          modules_center_.emplace_back(module);
+        }
+        if (pos == "modules-right") {
+          modules_right_.emplace_back(module);
+        }
+        module->dp.connect([module] { module->update(); });
+      } catch (const std::exception& e) {
+        std::cerr << e.what() << std::endl;
+      }
+    }
+  }
+}
+
+auto waybar::Bar::setupWidgets() -> void
 {
   auto &left = *Gtk::manage(new Gtk::Box(Gtk::ORIENTATION_HORIZONTAL, 0));
   auto &center = *Gtk::manage(new Gtk::Box(Gtk::ORIENTATION_HORIZONTAL, 0));
   auto &right = *Gtk::manage(new Gtk::Box(Gtk::ORIENTATION_HORIZONTAL, 0));
 
-  auto &box1 = *Gtk::manage(new Gtk::Box(Gtk::ORIENTATION_HORIZONTAL, 0));
-  window.add(box1);
-  box1.set_homogeneous(true);
-  box1.pack_start(left, true, true);
-  box1.pack_start(center, false, false);
-  box1.pack_end(right, true, true);
+  auto &box = *Gtk::manage(new Gtk::Box(Gtk::ORIENTATION_HORIZONTAL, 0));
+  window.add(box);
+  box.pack_start(left, true, true);
+  box.set_center_widget(center);
+  box.pack_end(right, true, true);
 
-  Factory factory(*this, _config);
-
-  if (_config["modules-left"]) {
-    for (auto name : _config["modules-left"]) {
-      auto &module = factory.makeModule(name.asString());
-      left.pack_start(module, false, true, 0);
-    }
+  Factory factory(*this, config_);
+  getModules(factory, "modules-left");
+  getModules(factory, "modules-center");
+  getModules(factory, "modules-right");
+  for (auto const& module : modules_left_) {
+    left.pack_start(*module, false, true, 0);
   }
-  if (_config["modules-center"]) {
-    for (auto name : _config["modules-center"]) {
-      auto &module = factory.makeModule(name.asString());
-      center.pack_start(module, true, false, 10);
-    }
+  for (auto const& module : modules_center_) {
+    center.pack_start(*module, true, true, 0);
   }
-  if (_config["modules-right"]) {
-    std::reverse(_config["modules-right"].begin(), _config["modules-right"].end());
-    for (auto name : _config["modules-right"]) {
-      auto &module = factory.makeModule(name.asString());
-      right.pack_end(module, false, false, 0);
-    }
+  std::reverse(modules_right_.begin(), modules_right_.end());
+  for (auto const& module : modules_right_) {
+    right.pack_end(*module, false, false, 0);
   }
 }

src/client.cpp

@@ -1,84 +1,111 @@
 #include "client.hpp"
+#include <iostream>
 
 waybar::Client::Client(int argc, char* argv[])
-  : gtk_main(argc, argv),
-    gdk_display(Gdk::Display::get_default()),
-    wlDisplay(gdk_wayland_display_get_wl_display(gdk_display->gobj()))
+  : gtk_app(Gtk::Application::create(argc, argv, "fr.arouillard.waybar")),
+    gdk_display(Gdk::Display::get_default())
 {
+  if (!gdk_display) {
+    throw std::runtime_error("Can't find display");
+  }
+  if (!GDK_IS_WAYLAND_DISPLAY(gdk_display->gobj())) {
+    throw std::runtime_error("Bar need to run under Wayland");
+  }
+  wl_display = gdk_wayland_display_get_wl_display(gdk_display->gobj());
   auto getFirstValidPath = [] (std::vector<std::string> possiblePaths) {
     wordexp_t p;
 
-    for (std::string path: possiblePaths) {
+    for (const std::string &path: possiblePaths) {
       if (wordexp(path.c_str(), &p, 0) == 0) {
-        if (access(p.we_wordv[0], F_OK) == 0) {
-          std::string result = p.we_wordv[0];
+        if (access(*p.we_wordv, F_OK) == 0) {
+          std::string result = *p.we_wordv;
           wordfree(&p);
           return result;
-        } else {
-          wordfree(&p);
         }
+        wordfree(&p);
       }
     }
 
     return std::string();
   };
 
-  configFile = getFirstValidPath({
+  config_file = getFirstValidPath({
     "$XDG_CONFIG_HOME/waybar/config",
     "$HOME/waybar/config",
     "/etc/xdg/waybar/config",
     "./resources/config",
   });
-  cssFile = getFirstValidPath({
+  css_file = getFirstValidPath({
     "$XDG_CONFIG_HOME/waybar/style.css",
     "$HOME/waybar/style.css",
     "/etc/xdg/waybar/style.css",
     "./resources/style.css",
   });
-
+  std::cout << "Resources files: " + config_file + ", " + css_file << std::endl;
 }
 
-void waybar::Client::_handle_global(void *data, struct wl_registry *registry,
+void waybar::Client::handleGlobal(void *data, struct wl_registry *registry,
   uint32_t name, const char *interface, uint32_t version)
 {
-  auto o = reinterpret_cast<waybar::Client *>(data);
-  if (!strcmp(interface, zwlr_layer_shell_v1_interface.name)) {
-    o->layer_shell = (zwlr_layer_shell_v1 *)wl_registry_bind(registry, name,
-      &zwlr_layer_shell_v1_interface, version);
-  } else if (!strcmp(interface, wl_output_interface.name)) {
+  auto o = static_cast<waybar::Client *>(data);
+  if (strcmp(interface, zwlr_layer_shell_v1_interface.name) == 0) {
+    o->layer_shell = static_cast<struct zwlr_layer_shell_v1 *>(
+      wl_registry_bind(registry, name, &zwlr_layer_shell_v1_interface, version));
+  } else if (strcmp(interface, wl_output_interface.name) == 0) {
     auto output = std::make_unique<struct wl_output *>();
-    *output = (struct wl_output *)wl_registry_bind(registry, name,
-      &wl_output_interface, version);
-    o->bars.emplace_back(*o, std::move(output));
-  } else if (!strcmp(interface, org_kde_kwin_idle_interface.name)) {
-    o->idle_manager = (org_kde_kwin_idle *)wl_registry_bind(registry, name,
-      &org_kde_kwin_idle_interface, version);
-  } else if (!strcmp(interface, wl_seat_interface.name)) {
-    o->seat = (struct wl_seat *)wl_registry_bind(registry, name,
-      &wl_seat_interface, version);
+    *output = static_cast<struct wl_output *>(wl_registry_bind(registry, name,
+      &wl_output_interface, version));
+    if (o->xdg_output_manager != nullptr) {
+      o->bars.emplace_back(std::make_unique<Bar>(*o, std::move(output), name));
+    }
+  } else if (strcmp(interface, wl_seat_interface.name) == 0) {
+    o->seat = static_cast<struct wl_seat *>(wl_registry_bind(registry, name,
+      &wl_seat_interface, version));
+  } else if (strcmp(interface, zxdg_output_manager_v1_interface.name) == 0
+    && version >= ZXDG_OUTPUT_V1_NAME_SINCE_VERSION) {
+      o->xdg_output_manager = static_cast<struct zxdg_output_manager_v1 *>(
+        wl_registry_bind(registry, name,
+        &zxdg_output_manager_v1_interface, ZXDG_OUTPUT_V1_NAME_SINCE_VERSION));
   }
 }
 
-void waybar::Client::_handle_global_remove(void *data,
-  struct wl_registry *registry, uint32_t name)
+void waybar::Client::handleGlobalRemove(void* data,
+  struct wl_registry* /*registry*/, uint32_t name)
 {
-    // TODO
+  auto o = static_cast<waybar::Client *>(data);
+  for (auto it = o->bars.begin(); it != o->bars.end(); ++it) {
+    if ((**it).wl_name == name) {
+      o->bars.erase(it);
+      break;
+    }
+  }
 }
 
-void waybar::Client::bind_interfaces()
+void waybar::Client::bindInterfaces()
 {
-  registry = wl_display_get_registry(wlDisplay);
+  registry = wl_display_get_registry(wl_display);
   static const struct wl_registry_listener registry_listener = {
-    .global = _handle_global,
-    .global_remove = _handle_global_remove,
+    .global = handleGlobal,
+    .global_remove = handleGlobalRemove,
   };
   wl_registry_add_listener(registry, &registry_listener, this);
-  wl_display_roundtrip(wlDisplay);
+  wl_display_roundtrip(wl_display);
+  if (!layer_shell || !seat || !xdg_output_manager) {
+    throw std::runtime_error("Failed to acquire required resources.");
+  }
+  wl_display_roundtrip(wl_display);
 }
 
-int waybar::Client::main(int argc, char* argv[])
+int waybar::Client::main(int /*argc*/, char* /*argv*/[])
 {
-  bind_interfaces();
-  gtk_main.run();
+  bindInterfaces();
+  gtk_app->hold();
+  gtk_app->run();
+  bars.clear();
+  zxdg_output_manager_v1_destroy(xdg_output_manager);
+  zwlr_layer_shell_v1_destroy(layer_shell);
+  wl_registry_destroy(registry);
+  wl_seat_destroy(seat);
+  wl_display_disconnect(wl_display);
   return 0;
 }

src/factory.cpp

@@ -1,26 +1,60 @@
 #include "factory.hpp"
 
-waybar::Factory::Factory(Bar &bar, Json::Value config)
-  : _bar(bar), _config(config)
+waybar::Factory::Factory(Bar& bar, const Json::Value& config)
+  : bar_(bar), config_(config)
 {}
 
-waybar::IModule &waybar::Factory::makeModule(std::string name)
+waybar::IModule* waybar::Factory::makeModule(const std::string &name) const
 {
-  if (name == "battery")
-    return *new waybar::modules::Battery(_config[name]);
-  if (name == "workspaces")
-    return *new waybar::modules::Workspaces(_bar);
-  if (name == "memory")
-    return *new waybar::modules::Memory(_config[name]);
-  if (name == "cpu")
-    return *new waybar::modules::Cpu(_config[name]);
-  if (name == "clock")
-    return *new waybar::modules::Clock(_config[name]);
-  if (name == "network")
-    return *new waybar::modules::Network(_config[name]);
-  if (name == "pulseaudio")
-    return *new waybar::modules::Pulseaudio(_config[name]);
-  if (!name.compare(0, 7, "custom/") && name.size() > 7)
-    return *new waybar::modules::Custom(name.substr(7), _config[name]);
+  try {
+    auto ref = name.substr(0, name.find("#"));
+    if (ref == "battery") {
+      return new waybar::modules::Battery(config_[name]);
+    }
+    #ifdef HAVE_SWAY
+    if (ref == "sway/mode") {
+      return new waybar::modules::sway::Mode(bar_, config_[name]);
+    }
+    if (ref == "sway/workspaces") {
+      return new waybar::modules::sway::Workspaces(bar_, config_[name]);
+    }
+    if (ref == "sway/window") {
+      return new waybar::modules::sway::Window(bar_, config_[name]);
+    }
+    #endif
+    if (ref == "memory") {
+      return new waybar::modules::Memory(config_[name]);
+    }
+    if (ref == "cpu") {
+      return new waybar::modules::Cpu(config_[name]);
+    }
+    if (ref == "clock") {
+      return new waybar::modules::Clock(config_[name]);
+    }
+    #ifdef HAVE_DBUSMENU
+    if (ref == "tray") {
+      return new waybar::modules::SNI::Tray(config_[name]);
+    }
+    #endif
+    #ifdef HAVE_LIBNL
+    if (ref == "network") {
+      return new waybar::modules::Network(config_[name]);
+    }
+    #endif
+    #ifdef HAVE_LIBPULSE
+    if (ref == "pulseaudio") {
+      return new waybar::modules::Pulseaudio(config_[name]);
+    }
+    #endif
+    if (ref.compare(0, 7, "custom/") == 0 && ref.size() > 7) {
+      return new waybar::modules::Custom(ref.substr(7), config_[name]);
+    }
+  } catch (const std::exception& e) {
+    auto err = fmt::format("Disabling module \"{}\", {}", name, e.what());
+    throw std::runtime_error(err);
+  } catch (...) {
+    auto err = fmt::format("Disabling module \"{}\", Unknown reason", name);
+    throw std::runtime_error(err);
+  }
   throw std::runtime_error("Unknown module: " + name);
 }

src/ipc/client.cpp

@@ -1,101 +0,0 @@
-#define _POSIX_C_SOURCE 200809L
-#include <string>
-#include <stdio.h>
-#include <sys/socket.h>
-#include <sys/un.h>
-#include "ipc/client.hpp"
-
-static const char ipc_magic[] = {'i', '3', '-', 'i', 'p', 'c'};
-static const size_t ipc_header_size = sizeof(ipc_magic)+8;
-
-std::string get_socketpath(void) {
-  const char *env = getenv("SWAYSOCK");
-  if (env) return std::string(env);
-	std::string str;
-  {
-		std::string str_buf;
-		FILE*  in;
-		char  buf[512] = { 0 };
-		if (!(in = popen("sway --get-socketpath 2>/dev/null", "r"))) {
-			throw std::runtime_error("Failed to get socket path");
-		}
-		while (fgets(buf, sizeof(buf), in) != nullptr) {
-			str_buf.append(buf, sizeof(buf));
-		}
-		pclose(in);
-		str = str_buf;
-	}
-	if (str.back() == '\n') {
-		str.pop_back();
-	}
-	return str;
-}
-
-int ipc_open_socket(std::string socket_path) {
-	struct sockaddr_un addr;
-	int socketfd;
-	if ((socketfd = socket(AF_UNIX, SOCK_STREAM, 0)) == -1) {
-		throw std::runtime_error("Unable to open Unix socket");
-	}
-	addr.sun_family = AF_UNIX;
-	strncpy(addr.sun_path, socket_path.c_str(), sizeof(addr.sun_path) - 1);
-	addr.sun_path[sizeof(addr.sun_path) - 1] = 0;
-	int l = sizeof(struct sockaddr_un);
-	if (connect(socketfd, (struct sockaddr *)&addr, l) == -1) {
-		throw std::runtime_error("Unable to connect to " + socket_path);
-	}
-	return socketfd;
-}
-
-struct ipc_response ipc_recv_response(int socketfd) {
-	char data[ipc_header_size];
-	uint32_t *data32 = (uint32_t *)(data + sizeof(ipc_magic));
-
-	size_t total = 0;
-	while (total < ipc_header_size) {
-		ssize_t received = recv(socketfd, data + total, ipc_header_size - total, 0);
-		if (received <= 0) {
-			throw std::runtime_error("Unable to receive IPC response");
-		}
-		total += received;
-	}
-
-	struct ipc_response response;
-
-	total = 0;
-	response.size = data32[0];
-	response.type = data32[1];
-  char payload[response.size + 1];
-
-	while (total < response.size) {
-		ssize_t received = recv(socketfd, payload + total, response.size - total, 0);
-		if (received < 0) {
-			throw std::runtime_error("Unable to receive IPC response");
-		}
-		total += received;
-	}
-	payload[response.size] = '\0';
-	response.payload = std::string(payload);
-	return response;
-}
-
-std::string ipc_single_command(int socketfd, uint32_t type, const char *payload, uint32_t *len) {
-	char data[ipc_header_size];
-	uint32_t *data32 = (uint32_t *)(data + sizeof(ipc_magic));
-	memcpy(data, ipc_magic, sizeof(ipc_magic));
-	data32[0] = *len;
-	data32[1] = type;
-
-	if (send(socketfd, data, ipc_header_size, 0) == -1) {
-		throw std::runtime_error("Unable to send IPC header");
-	}
-
-	if (send(socketfd, payload, *len, 0) == -1) {
-		throw std::runtime_error("Unable to send IPC payload");
-	}
-
-	struct ipc_response resp = ipc_recv_response(socketfd);
-	std::string response = resp.payload;
-	*len = resp.size;
-	return response;
-}

src/main.cpp

@@ -1,21 +1,21 @@
-#include <gtkmm.h>
-#include <wayland-client.hpp>
-#include <gdk/gdkwayland.h>
-#include <csignal>
 #include "client.hpp"
+#include <csignal>
+#include <iostream>
 
 namespace waybar {
-  static Client* client;
-}
+
+static Client* client;
+
+} // namespace waybar
 
 int main(int argc, char* argv[])
 {
   try {
     waybar::Client c(argc, argv);
     waybar::client = &c;
-    std::signal(SIGUSR1, [] (int signal) {
+    std::signal(SIGUSR1, [] (int /*signal*/) {
       for (auto& bar : waybar::client->bars) {
-        bar.toggle();
+        (*bar).toggle();
       }
     });
 

src/modules/battery.cpp

@@ -1,56 +1,142 @@
 #include "modules/battery.hpp"
 
-waybar::modules::Battery::Battery(Json::Value config)
-  : _config(config)
+waybar::modules::Battery::Battery(const Json::Value& config)
+  : ALabel(config, "{capacity}%")
 {
   try {
-    for (auto &node : fs::directory_iterator(_data_dir)) {
-      if (fs::is_directory(node) && fs::exists(node / "capacity")
-        && fs::exists(node / "status")) {
-        _batteries.push_back(node);
+    if (config_["bat"].isString()) {
+      auto dir = data_dir_ / config_["bat"].asString();
+      if (fs::is_directory(dir) && fs::exists(dir / "capacity")
+        && fs::exists(dir / "status") && fs::exists(dir / "uevent")) {
+        batteries_.push_back(dir);
+      }
+    } else {
+      for (auto const& node : fs::directory_iterator(data_dir_)) {
+        if (fs::is_directory(node) && fs::exists(node / "capacity")
+          && fs::exists(node / "status") && fs::exists(node / "uevent")) {
+          batteries_.push_back(node);
+        }
       }
     }
   } catch (fs::filesystem_error &e) {
-    std::cerr << e.what() << std::endl;
+    throw std::runtime_error(e.what());
   }
+  if (batteries_.empty()) {
+    if (config_["bat"].isString()) {
+      throw std::runtime_error("No battery named " + config_["bat"].asString());
+    }
+    throw std::runtime_error("No batteries.");
+  }
+  fd_ = inotify_init1(IN_CLOEXEC);
+  if (fd_ == -1) {
+    throw std::runtime_error("Unable to listen batteries.");
+  }
+  for (auto const& bat : batteries_) {
+    inotify_add_watch(fd_, (bat / "uevent").c_str(), IN_ACCESS);
+  }
+  worker();
+}
 
-  _label.get_style_context()->add_class("battery");
+waybar::modules::Battery::~Battery()
+{
+  close(fd_);
+}
 
-  _thread = [this] {
-    update();
-    _thread.sleep_for(chrono::minutes(1));
+void waybar::modules::Battery::worker()
+{
+  // Trigger first values
+  update();
+  uint32_t interval = config_["interval"].isUInt() ? config_["interval"].asUInt() : 60;
+  thread_timer_ = [this, interval] {
+    thread_.sleep_for(chrono::seconds(interval));
+    dp.emit();
   };
+  thread_ = [this] {
+    struct inotify_event event = {0};
+    int nbytes = read(fd_, &event, sizeof(event));
+    if (nbytes != sizeof(event)) {
+      return;
+    }
+    // TODO: don't stop timer for now since there is some bugs :?
+    // thread_timer_.stop();
+    dp.emit();
+  };
+}
+
+std::tuple<uint16_t, std::string> waybar::modules::Battery::getInfos()
+{
+  try {
+    uint16_t total = 0;
+    std::string status = "Unknown";
+    for (auto const& bat : batteries_) {
+      uint16_t capacity;
+      std::string _status;
+      std::ifstream(bat / "capacity") >> capacity;
+      std::ifstream(bat / "status") >> _status;
+      if (_status != "Unknown") {
+        status = _status;
+      }
+      total += capacity;
+    }
+    uint16_t capacity = total / batteries_.size();
+    return {capacity, status};
+  } catch (const std::exception& e) {
+    std::cerr << e.what() << std::endl;
+    return {0, "Unknown"};
+  }
+}
+
+std::string waybar::modules::Battery::getState(uint16_t capacity)
+{
+  // Get current state
+  std::vector<std::pair<std::string, uint16_t>> states;
+  if (config_["states"].isObject()) {
+    for (auto it = config_["states"].begin(); it != config_["states"].end(); ++it) {
+      if (it->isUInt() && it.key().isString()) {
+        states.push_back({it.key().asString(), it->asUInt()});
+      }
+    }
+  }
+  // Sort states
+  std::sort(states.begin(), states.end(), [](auto &a, auto &b) {
+    return a.second < b.second;
+  });
+  std::string validState = "";
+  for (auto state : states) {
+    if (capacity <= state.second && validState.empty()) {
+      label_.get_style_context()->add_class(state.first);
+      validState = state.first;
+    } else {
+      label_.get_style_context()->remove_class(state.first);
+    }
+  }
+  return validState;
 }
 
 auto waybar::modules::Battery::update() -> void
 {
-  try {
-    int total = 0;
-    bool charging = false;
-    for (auto &bat : _batteries) {
-      int capacity;
-      std::string status;
-      std::ifstream(bat / "capacity") >> capacity;
-      total += capacity;
-      std::ifstream(bat / "status") >> status;
-      if (status == "Charging") {
-        charging = true;
-      }
-    }
-    if (charging == true) {
-      _label.get_style_context()->add_class("charging");
-    } else {
-      _label.get_style_context()->remove_class("charging");
-    }
-    auto format = _config["format"] ? _config["format"].asString() : "{}%";
-    _label.set_text(fmt::format(format, total / _batteries.size()));
-    _label.set_tooltip_text(charging ? "Charging" : "Discharging");
-  } catch (std::exception &e) {
-    std::cerr << e.what() << std::endl;
+  auto [capacity, status] = getInfos();
+  label_.set_tooltip_text(status);
+  std::transform(status.begin(), status.end(), status.begin(), ::tolower);
+  auto format = format_;
+  auto state = getState(capacity);
+  label_.get_style_context()->remove_class(old_status_);
+  label_.get_style_context()->add_class(status);
+  old_status_ = status;
+  if (!state.empty() && config_["format-" + status + "-" + state].isString()) {
+    format = config_["format-" + status + "-" + state].asString();
+  } else if (config_["format-" + status].isString()) {
+    format = config_["format-" + status].asString();
+  } else if (!state.empty() && config_["format-" + state].isString()) {
+    format = config_["format-" + state].asString();
+  }
+  if (format.empty()) {
+    event_box_.hide();
+    label_.set_name("");
+  } else {
+    event_box_.show();
+    label_.set_name("battery");
+    label_.set_text(fmt::format(format, fmt::arg("capacity", capacity),
+      fmt::arg("icon", getIcon(capacity))));
   }
 }
-
-waybar::modules::Battery::operator Gtk::Widget &()
-{
-  return _label;
-}

src/modules/clock.cpp

@@ -1,27 +1,21 @@
 #include "modules/clock.hpp"
 
-waybar::modules::Clock::Clock(Json::Value config)
-  : _config(config)
+waybar::modules::Clock::Clock(const Json::Value& config)
+  : ALabel(config, "{:%H:%M}")
 {
-  _label.get_style_context()->add_class("clock");
-  _thread = [this] {
-    update();
+  label_.set_name("clock");
+  uint32_t interval = config_["interval"].isUInt() ? config_["interval"].asUInt() : 60;
+  thread_ = [this, interval] {
     auto now = waybar::chrono::clock::now();
-    auto timeout =
-      std::chrono::floor<std::chrono::minutes>(now + std::chrono::minutes(1));
-    _thread.sleep_until(timeout);
+    dp.emit();
+    auto timeout = std::chrono::floor<std::chrono::seconds>(now
+      + std::chrono::seconds(interval));
+    thread_.sleep_until(timeout);
   };
-};
+}
 
 auto waybar::modules::Clock::update() -> void
 {
-  auto t = std::time(nullptr);
-  auto localtime = std::localtime(&t);
-  auto format =
-    _config["format"] ? _config["format"].asString() : "{:02}:{:02}";
-  _label.set_text(fmt::format(format, localtime->tm_hour, localtime->tm_min));
-}
-
-waybar::modules::Clock::operator Gtk::Widget &() {
-  return _label;
+  auto localtime = fmt::localtime(std::time(nullptr));
+  label_.set_text(fmt::format(format_, localtime));
 }

src/modules/cpu.cpp

@@ -1,27 +1,90 @@
 #include "modules/cpu.hpp"
 
-waybar::modules::Cpu::Cpu(Json::Value config)
-  : _config(config)
+waybar::modules::Cpu::Cpu(const Json::Value& config)
+  : ALabel(config, "{usage}%")
 {
-  _label.get_style_context()->add_class("cpu");
-  _thread = [this] {
-    update();
-    int interval = _config["interval"] ? _config["inveral"].asInt() : 10;
-    _thread.sleep_for(chrono::seconds(interval));
+  label_.set_name("cpu");
+  uint32_t interval = config_["interval"].isUInt() ? config_["interval"].asUInt() : 10;
+  thread_ = [this, interval] {
+    dp.emit();
+    thread_.sleep_for(chrono::seconds(interval));
   };
-};
+}
 
 auto waybar::modules::Cpu::update() -> void
 {
-  struct sysinfo info;
-  if (!sysinfo(&info)) {
-    float f_load = 1.f / (1 << SI_LOAD_SHIFT);
-    int load = info.loads[0] * f_load * 100 / get_nprocs();
-    auto format = _config["format"] ? _config["format"].asString() : "{}%";
-    _label.set_text(fmt::format(format, load));
+  try {
+    // TODO: as creating dynamic fmt::arg arrays is buggy we have to calc both
+    auto cpu_load = getCpuLoad();
+    auto [cpu_usage, tooltip] = getCpuUsage();
+    label_.set_tooltip_text(tooltip);
+    label_.set_text(fmt::format(format_,
+      fmt::arg("load", cpu_load), fmt::arg("usage", cpu_usage)));
+  } catch (const std::exception& e) {
+    std::cerr << e.what() << std::endl;
   }
 }
 
-waybar::modules::Cpu::operator Gtk::Widget &() {
-  return _label;
+uint16_t waybar::modules::Cpu::getCpuLoad()
+{
+  struct sysinfo info = {0};
+  if (sysinfo(&info) == 0) {
+    float f_load = 1.f / (1u << SI_LOAD_SHIFT);
+    uint16_t load = info.loads[0] * f_load * 100 / get_nprocs();
+    return load;
+  }
+  throw std::runtime_error("Can't get Cpu load");
+}
+
+std::tuple<uint16_t, std::string> waybar::modules::Cpu::getCpuUsage()
+{
+  if (prev_times_.empty()) {
+    prev_times_ = parseCpuinfo();
+    std::this_thread::sleep_for(chrono::milliseconds(100));
+  }
+  std::vector<std::tuple<size_t, size_t>> curr_times = parseCpuinfo();
+  std::string tooltip;
+  uint16_t usage = 0;
+  for (size_t i = 0; i < curr_times.size(); ++i) {
+    auto [curr_idle, curr_total] = curr_times[i];
+    auto [prev_idle, prev_total] = prev_times_[i];
+    const float delta_idle = curr_idle - prev_idle;
+    const float delta_total = curr_total - prev_total;
+    uint16_t tmp = 100 * (1 - delta_idle / delta_total);
+    if (i == 0) {
+      usage = tmp;
+      tooltip = fmt::format("Total: {}%", tmp);
+    } else {
+      tooltip = tooltip + fmt::format("\nCore{}: {}%", i - 1, tmp);
+    }
+  }
+  prev_times_ = curr_times;
+  return {usage, tooltip};
+}
+
+std::vector<std::tuple<size_t, size_t>> waybar::modules::Cpu::parseCpuinfo()
+{
+  std::ifstream info(data_dir_);
+  if (!info.is_open()) {
+    throw std::runtime_error("Can't open " + data_dir_);
+  }
+  std::vector< std::tuple<size_t, size_t> > cpuinfo;
+  std::string line;
+  while (getline(info, line)) {    
+    if (line.substr(0,3).compare("cpu") != 0) {
+      break;
+    }
+    std::stringstream sline(line.substr(5));
+    std::vector<size_t> times;
+    for (size_t time; sline >> time; times.push_back(time));
+
+    size_t idle_time = 0;
+    size_t total_time = 0;
+    if (times.size() >= 4) {
+      idle_time = times[3];
+      total_time = std::accumulate(times.begin(), times.end(), 0);
+    }
+    cpuinfo.push_back({idle_time, total_time});
+  }
+  return cpuinfo;
 }

src/modules/custom.cpp

@@ -1,50 +1,136 @@
 #include "modules/custom.hpp"
-#include <iostream>
 
-waybar::modules::Custom::Custom(std::string name, Json::Value config)
-  : _name(name), _config(config)
+waybar::modules::Custom::Custom(const std::string name,
+  const Json::Value& config)
+  : ALabel(config, "{}"), name_(name)
 {
-  _label.get_style_context()->add_class("custom-" + name);
-  if (!_config["exec"]) {
-    std::cerr << name + " has no exec path." << std::endl;
-    return;
+  if (!config_["exec"].isString()) {
+    throw std::runtime_error(name_ + " has no exec path.");
   }
-  _thread = [this] {
-    update();
-    int interval = _config["interval"] ? _config["inveral"].asInt() : 30;
-    _thread.sleep_for(chrono::seconds(interval));
+  if (config_["interval"].isUInt()) {
+    delayWorker();
+  } else {
+    continuousWorker();
+  }
+}
+
+void waybar::modules::Custom::delayWorker()
+{
+  auto interval = config_["interval"].asUInt();
+  thread_ = [this, interval] {
+    bool can_update = true;
+    if (config_["exec-if"].isString()) {
+      auto res = waybar::util::command::exec(config_["exec-if"].asString());
+      if (res.exit_code != 0) {
+        can_update = false;
+        label_.hide();
+        label_.set_name("");
+      }
+    }
+    if (can_update) {
+      output_ = waybar::util::command::exec(config_["exec"].asString());
+      dp.emit();
+    }
+    thread_.sleep_for(chrono::seconds(interval));
   };
-};
+}
+
+void waybar::modules::Custom::continuousWorker()
+{
+  auto cmd = config_["exec"].asString();
+  FILE* fp(popen(cmd.c_str(), "r"));
+  if (!fp) {
+    throw std::runtime_error("Unable to open " + cmd);
+  }
+  thread_ = [this, fp] {
+    char* buff = nullptr;
+    size_t len = 0;
+    if (getline(&buff, &len, fp) == -1) {
+      pclose(fp);
+      thread_.stop();
+      output_ = { 1, "" };
+      dp.emit();
+      return;
+    }
+
+    std::string output = buff;
+
+    // Remove last newline
+    if (!output.empty() && output[output.length()-1] == '\n') {
+      output.erase(output.length()-1);
+    }
+    output_ = { 0, output };
+    dp.emit();
+  };
+}
 
 auto waybar::modules::Custom::update() -> void
 {
-  std::array<char, 128> buffer;
-  std::string output;
-  std::shared_ptr<FILE> fp(popen(_config["exec"].asCString(), "r"), pclose);
-  if (!fp) {
-    std::cerr << _name + " can't exec " + _config["exec"].asString() << std::endl;
-    return;
-  }
-  while (!feof(fp.get())) {
-    if (fgets(buffer.data(), 128, fp.get()) != nullptr)
-        output += buffer.data();
-  }
-
-  // Remove last newline
-  if (!output.empty() && output[output.length()-1] == '\n') {
-    output.erase(output.length()-1);
-  }
-
   // Hide label if output is empty
-  if (output.empty())
-    _label.hide();
-  else {
-    auto format = _config["format"] ? _config["format"].asString() : "{}";
-    _label.set_text(fmt::format(format, output));
-    _label.show();
+  if (output_.out.empty() || output_.exit_code != 0) {
+    label_.hide();
+    label_.set_name("");
+  } else {
+    label_.set_name("custom-" + name_);
+
+    if (config_["return-type"].asString() == "json") {
+      parseOutputJson();
+    } else {
+      parseOutputRaw();
+    }
+
+    auto str = fmt::format(format_, text_);
+    label_.set_text(str);
+    if (text_ == tooltip_) {
+      label_.set_tooltip_text(str);
+    } else {
+      label_.set_tooltip_text(tooltip_);
+    }
+    if (class_ != "") {
+      if (prevclass_ != "") {
+        label_.get_style_context()->remove_class(prevclass_);
+      }
+      label_.get_style_context()->add_class(class_);
+      prevclass_ = class_;
+    } else {
+      label_.get_style_context()->remove_class(prevclass_);
+      prevclass_ = "";
+    }
+
+    label_.show();
   }
 }
 
-waybar::modules::Custom::operator Gtk::Widget &() {
-  return _label;
+void waybar::modules::Custom::parseOutputRaw()
+{
+  std::istringstream output(output_.out);
+  std::string line;
+  int i = 0;
+  while (getline(output, line)) {
+    if (i == 0) {
+      text_ = line;
+      tooltip_ = line;
+      class_ = "";
+    } else if (i == 1) {
+      tooltip_ = line;
+    } else if (i == 2) {
+      class_ = line;
+    } else {
+      break;
+    }
+    i++;
+  }
 }
+
+void waybar::modules::Custom::parseOutputJson()
+{
+  std::istringstream output(output_.out);
+  std::string line;
+  while (getline(output, line)) {
+    auto parsed = parser_.parse(line);
+    text_ = parsed["text"].asString();
+    tooltip_ = parsed["tooltip"].asString();
+    class_ = parsed["class"].asString();
+    break;
+  }
+}

src/modules/memory.cpp

@@ -1,30 +1,62 @@
 #include "modules/memory.hpp"
 
-waybar::modules::Memory::Memory(Json::Value config)
-  : _config(config)
+waybar::modules::Memory::Memory(const Json::Value& config)
+  : ALabel(config, "{}%")
 {
-  _label.get_style_context()->add_class("memory");
-  _thread = [this] {
-    update();
-    int interval = _config["interval"] ? _config["inveral"].asInt() : 30;
-    _thread.sleep_for(chrono::seconds(interval));
+  uint32_t interval = config_["interval"].isUInt() ? config_["interval"].asUInt() : 30;
+  thread_ = [this, interval] {
+    dp.emit();
+    thread_.sleep_for(chrono::seconds(interval));
   };
-};
+}
 
 auto waybar::modules::Memory::update() -> void
 {
-  struct sysinfo info;
-  if (!sysinfo(&info)) {
-    auto total = info.totalram * info.mem_unit;
-    auto freeram = info.freeram * info.mem_unit;
-    int used_ram_percentage = 100 * (total - freeram) / total;
-    auto format = _config["format"] ? _config["format"].asString() : "{}%";
-    _label.set_text(fmt::format(format, used_ram_percentage));
-    auto used_ram_gigabytes = (total - freeram) / std::pow(1024, 3);
-    _label.set_tooltip_text(fmt::format("{:.{}f}Gb used", used_ram_gigabytes, 1));
+  parseMeminfo();
+  if (memtotal_ > 0 && memfree_ >= 0) {
+    int used_ram_percentage = 100 * (memtotal_ - memfree_) / memtotal_;
+    label_.set_text(fmt::format(format_, used_ram_percentage));
+    auto used_ram_gigabytes = (memtotal_ - memfree_) / std::pow(1024, 2);
+    label_.set_tooltip_text(fmt::format("{:.{}f}Gb used", used_ram_gigabytes, 1));
+    label_.set_name("memory");
+    label_.show();
+  } else {
+    label_.set_name("");
+    label_.hide();
   }
 }
 
-waybar::modules::Memory::operator Gtk::Widget &() {
-  return _label;
+void waybar::modules::Memory::parseMeminfo()
+{
+  long memfree = -1, membuffer = -1, memcache = -1, memavail = -1;
+  std::ifstream info(data_dir_);
+  if (!info.is_open()) {
+    throw std::runtime_error("Can't open " + data_dir_);
+  }
+  std::string line;
+  while (getline(info, line)) {
+    auto posDelim = line.find(":");
+    if (posDelim == std::string::npos) {
+      continue;
+    }
+    std::string name = line.substr(0, posDelim);
+    long value = std::stol(line.substr(posDelim + 1));
+
+    if (name.compare("MemTotal") == 0) {
+      memtotal_ = value;
+    } else if (name.compare("MemAvailable") == 0) {
+      memavail = value;
+    } else if (name.compare("MemFree") == 0) {
+      memfree = value;
+    } else if (name.compare("Buffers") == 0) {
+      membuffer = value;
+    } else if (name.compare("Cached") == 0) {
+      memcache = value;
+    }
+    if (memtotal_ > 0 &&
+      (memavail >= 0 || (memfree > -1  && membuffer > -1 && memcache > -1))) {
+      break;
+    }
+  }
+  memfree_ = memavail >= 0 ? memavail : memfree + membuffer + memcache;
 }

src/modules/network.cpp

@@ -1,59 +1,375 @@
 #include "modules/network.hpp"
 
-waybar::modules::Network::Network(Json::Value config)
-  : _config(config), _ifid(if_nametoindex(config["interface"].asString().c_str()))
+waybar::modules::Network::Network(const Json::Value& config)
+  : ALabel(config, "{ifname}"), family_(AF_INET),
+    signal_strength_dbm_(0), signal_strength_(0)
 {
-  _label.get_style_context()->add_class("network");
-  _thread = [this] {
-    update();
-    _thread.sleep_for(chrono::minutes(1));
+  sock_fd_ = socket(AF_NETLINK, SOCK_RAW, NETLINK_ROUTE);
+  if (sock_fd_ < 0) {
+    throw std::runtime_error("Can't open network socket");
+  }
+  nladdr_.nl_family = AF_NETLINK;
+  nladdr_.nl_groups = RTMGRP_LINK | RTMGRP_IPV4_IFADDR;
+  if (bind(sock_fd_, reinterpret_cast<struct sockaddr *>(&nladdr_),
+    sizeof(nladdr_)) != 0) {
+    throw std::runtime_error("Can't bind network socket");
+  }
+  if (config_["interface"].isString()) {
+    ifid_ = if_nametoindex(config_["interface"].asCString());
+    ifname_ = config_["interface"].asString();
+    if (ifid_ <= 0) {
+      throw std::runtime_error("Can't found network interface");
+    }
+  } else {
+    ifid_ = getExternalInterface();
+    if (ifid_ > 0) {
+      char ifname[IF_NAMESIZE];
+      if_indextoname(ifid_, ifname);
+      ifname_ = ifname;
+      getInterfaceAddress();
+    }
+  }
+  initNL80211();
+  label_.set_name("network");
+  // Trigger first values
+  getInfo();
+  update();
+  worker();
+}
+
+waybar::modules::Network::~Network()
+{
+  close(sock_fd_);
+  nl_socket_free(sk_);
+}
+
+void waybar::modules::Network::worker()
+{
+  thread_ = [this] {
+    char buf[4096];
+    uint64_t len = netlinkResponse(sock_fd_, buf, sizeof(buf),
+      RTMGRP_LINK | RTMGRP_IPV4_IFADDR);
+    bool need_update = false;
+    for (auto nh = reinterpret_cast<struct nlmsghdr *>(buf); NLMSG_OK(nh, len);
+      nh = NLMSG_NEXT(nh, len)) {
+      if (nh->nlmsg_type == NLMSG_DONE) {
+        break;
+      }
+      if (nh->nlmsg_type == NLMSG_ERROR) {
+        continue;
+      }
+      if (nh->nlmsg_type < RTM_NEWADDR) {
+        auto rtif = static_cast<struct ifinfomsg *>(NLMSG_DATA(nh));
+        if (rtif->ifi_index == static_cast<int>(ifid_)) {
+          need_update = true;
+          if (!(rtif->ifi_flags & IFF_RUNNING)) {
+            disconnected();
+          }
+        }
+      }
+    }
+    if (ifid_ <= 0 && !config_["interface"].isString()) {
+      // Need to wait before get external interface
+      thread_.sleep_for(std::chrono::seconds(1));
+      ifid_ = getExternalInterface();
+      if (ifid_ > 0) {
+        char ifname[IF_NAMESIZE];
+        if_indextoname(ifid_, ifname);
+        ifname_ = ifname;
+        getInterfaceAddress();
+        need_update = true;
+      }
+    }
+    if (need_update) {
+      getInfo();
+      dp.emit();
+    }
   };
-};
+  uint32_t interval = config_["interval"].isUInt() ? config_["interval"].asUInt() : 60;
+  thread_timer_ = [this, interval] {
+    thread_.sleep_for(std::chrono::seconds(interval));
+    if (ifid_ > 0) {
+      getInfo();
+      dp.emit();
+    }
+  };
+}
 
 auto waybar::modules::Network::update() -> void
 {
-  _getInfo();
-  auto format = _config["format"] ? _config["format"].asString() : "{}";
-  _label.set_text(fmt::format(format,
-    fmt::arg("essid", _essid),
-    fmt::arg("signaldBm", _signalStrengthdBm),
-    fmt::arg("signalStrength", _signalStrength)
+  auto format = format_;
+  if (ifid_ <= 0) {
+    format = config_["format-disconnected"].isString()
+      ? config_["format-disconnected"].asString() : format;
+    label_.get_style_context()->add_class("disconnected");
+  } else {
+    if (essid_.empty()) {
+      format = config_["format-ethernet"].isString()
+        ? config_["format-ethernet"].asString() : format;
+    } else {
+      format = config_["format-wifi"].isString()
+        ? config_["format-wifi"].asString() : format;
+    }
+    label_.get_style_context()->remove_class("disconnected");
+  }
+  label_.set_text(fmt::format(format,
+    fmt::arg("essid", essid_),
+    fmt::arg("signaldBm", signal_strength_dbm_),
+    fmt::arg("signalStrength", signal_strength_),
+    fmt::arg("ifname", ifname_),
+    fmt::arg("netmask", netmask_),
+    fmt::arg("ipaddr", ipaddr_),
+    fmt::arg("cidr", cidr_)
   ));
 }
 
-int waybar::modules::Network::_scanCb(struct nl_msg *msg, void *data) {
-    auto net = static_cast<waybar::modules::Network *>(data);
-    auto gnlh = static_cast<genlmsghdr *>(nlmsg_data(nlmsg_hdr(msg)));
-    struct nlattr* tb[NL80211_ATTR_MAX + 1];
-    struct nlattr* bss[NL80211_BSS_MAX + 1];
-    struct nla_policy bss_policy[NL80211_BSS_MAX + 1]{};
-    bss_policy[NL80211_BSS_TSF].type = NLA_U64;
-    bss_policy[NL80211_BSS_FREQUENCY].type = NLA_U32;
-    bss_policy[NL80211_BSS_BSSID].type = NLA_UNSPEC;
-    bss_policy[NL80211_BSS_BEACON_INTERVAL].type = NLA_U16;
-    bss_policy[NL80211_BSS_CAPABILITY].type = NLA_U16;
-    bss_policy[NL80211_BSS_INFORMATION_ELEMENTS].type = NLA_UNSPEC;
-    bss_policy[NL80211_BSS_SIGNAL_MBM].type = NLA_U32;
-    bss_policy[NL80211_BSS_SIGNAL_UNSPEC].type = NLA_U8;
-    bss_policy[NL80211_BSS_STATUS].type = NLA_U32;
-
-    if (nla_parse(tb, NL80211_ATTR_MAX, genlmsg_attrdata(gnlh, 0), genlmsg_attrlen(gnlh, 0), nullptr) < 0)
-      return NL_SKIP;
-    if (!tb[NL80211_ATTR_BSS])
-      return NL_SKIP;
-    if (nla_parse_nested(bss, NL80211_BSS_MAX, tb[NL80211_ATTR_BSS], bss_policy))
-      return NL_SKIP;
-    if (!net->_associatedOrJoined(bss))
-      return NL_SKIP;
-    net->_parseEssid(bss);
-    net->_parseSignal(bss);
-    // TODO: parse quality
-    return NL_SKIP;
+void waybar::modules::Network::disconnected()
+{
+  essid_.clear();
+  signal_strength_dbm_ = 0;
+  signal_strength_ = 0;
+  ipaddr_.clear();
+  netmask_.clear();
+  cidr_ = 0;
+  ifname_.clear();
+  ifid_ = -1;
 }
 
-void waybar::modules::Network::_parseEssid(struct nlattr **bss)
+void waybar::modules::Network::initNL80211()
 {
-  _essid.clear();
+  sk_ = nl_socket_alloc();
+	if (genl_connect(sk_) != 0) {
+    nl_socket_free(sk_);
+    throw std::runtime_error("Can't connect to netlink socket");
+  }
+  if (nl_socket_modify_cb(sk_, NL_CB_VALID, NL_CB_CUSTOM, scanCb, this) < 0) {
+    nl_socket_free(sk_);
+    throw std::runtime_error("Can't connect to netlink socket");
+  }
+  nl80211_id_ = genl_ctrl_resolve(sk_, "nl80211");
+  if (nl80211_id_ < 0) {
+    nl_socket_free(sk_);
+    throw std::runtime_error("Can't resolve nl80211 interface");
+  }
+}
+
+// Based on https://gist.github.com/Yawning/c70d804d4b8ae78cc698
+int waybar::modules::Network::getExternalInterface()
+{
+  static const uint32_t route_buffer_size = 8192;
+  struct nlmsghdr *hdr = nullptr;
+  struct rtmsg *rt = nullptr;
+  char resp[route_buffer_size] = {0};
+  int ifidx = -1;
+
+  /* Prepare request. */
+  constexpr uint32_t reqlen = NLMSG_SPACE(sizeof(*rt));
+  char req[reqlen] = {0};
+
+  /* Build the RTM_GETROUTE request. */
+  hdr = reinterpret_cast<struct nlmsghdr *>(req);
+  hdr->nlmsg_len = NLMSG_LENGTH(sizeof(*rt));
+  hdr->nlmsg_flags = NLM_F_REQUEST | NLM_F_DUMP;
+  hdr->nlmsg_type = RTM_GETROUTE;
+  rt = static_cast<struct rtmsg *>(NLMSG_DATA(hdr));
+  rt->rtm_family = family_;
+  rt->rtm_table = RT_TABLE_MAIN;
+
+  /* Issue the query. */
+  if (netlinkRequest(sock_fd_, req, reqlen) < 0) {
+    goto out;
+  }
+
+  /* Read the response(s).
+   *
+   * WARNING: All the packets generated by the request must be consumed (as in,
+   * consume responses till NLMSG_DONE/NLMSG_ERROR is encountered).
+   */
+  do {
+    auto len = netlinkResponse(sock_fd_, resp, route_buffer_size);
+    if (len < 0) {
+      goto out;
+    }
+
+    /* Parse the response payload into netlink messages. */
+    for (hdr = reinterpret_cast<struct nlmsghdr *>(resp); NLMSG_OK(hdr, len);
+      hdr = NLMSG_NEXT(hdr, len)) {
+      if (hdr->nlmsg_type == NLMSG_DONE) {
+        goto out;
+      }
+      if (hdr->nlmsg_type == NLMSG_ERROR) {
+        /* Even if we found the interface index, something is broken with the
+         * netlink socket, so return an error.
+         */
+        ifidx = -1;
+        goto out;
+      }
+
+      /* If we found the correct answer, skip parsing the attributes. */
+      if (ifidx != -1) {
+        continue;
+      }
+
+      /* Find the message(s) concerting the main routing table, each message
+       * corresponds to a single routing table entry.
+       */
+      rt = static_cast<struct rtmsg *>(NLMSG_DATA(hdr));
+      if (rt->rtm_table != RT_TABLE_MAIN) {
+        continue;
+      }
+
+      /* Parse all the attributes for a single routing table entry. */
+      struct rtattr *attr = RTM_RTA(rt);
+      uint64_t attrlen = RTM_PAYLOAD(hdr);
+      bool has_gateway = false;
+      bool has_destination = false;
+      int temp_idx = -1;
+      for (; RTA_OK(attr, attrlen); attr = RTA_NEXT(attr, attrlen)) {
+        /* Determine if this routing table entry corresponds to the default
+         * route by seeing if it has a gateway, and if a destination addr is
+         * set, that it is all 0s.
+         */
+        switch (attr->rta_type) {
+          case RTA_GATEWAY:
+            /* The gateway of the route.
+             *
+             * If someone every needs to figure out the gateway address as well,
+             * it's here as the attribute payload.
+             */
+            has_gateway = true;
+            break;
+          case RTA_DST: {
+            /* The destination address.
+             * Should be either missing, or maybe all 0s.  Accept both.
+             */
+            const uint32_t nr_zeroes = (family_ == AF_INET) ? 4 : 16;
+            unsigned char c = 0;
+            size_t dstlen = RTA_PAYLOAD(attr);
+            if (dstlen != nr_zeroes) {
+              break;
+            }
+            for (uint32_t i = 0; i < dstlen; i += 1) {
+              c |= *((unsigned char *)RTA_DATA(attr) + i);
+            }
+            has_destination = (c == 0);
+            break;
+          }
+          case RTA_OIF:
+            /* The output interface index. */
+            temp_idx = *static_cast<int*>(RTA_DATA(attr));
+            break;
+          default:
+            break;
+        }
+      }
+      /* If this is the default route, and we know the interface index,
+       * we can stop parsing this message.
+       */
+      if (has_gateway && !has_destination && temp_idx != -1) {
+        ifidx = temp_idx;
+        break;
+      }
+    }
+  } while (true);
+
+out:
+  return ifidx;
+}
+
+void waybar::modules::Network::getInterfaceAddress() {
+  unsigned int cidrRaw;
+  struct ifaddrs *ifaddr, *ifa;
+  int success = getifaddrs(&ifaddr);
+  if (success == 0) {
+    ifa = ifaddr;
+    while (ifa != nullptr && ipaddr_.empty() && netmask_.empty()) {
+      if (ifa->ifa_addr != nullptr && ifa->ifa_addr->sa_family == family_) {
+        if (strcmp(ifa->ifa_name, ifname_.c_str()) == 0) {
+          ipaddr_ = inet_ntoa(((struct sockaddr_in*)ifa->ifa_addr)->sin_addr);
+          netmask_ = inet_ntoa(((struct sockaddr_in*)ifa->ifa_netmask)->sin_addr);
+          cidrRaw = ((struct sockaddr_in *)(ifa->ifa_netmask))->sin_addr.s_addr;
+          unsigned int cidr = 0;
+          while (cidrRaw) {
+              cidr += cidrRaw & 1;
+              cidrRaw >>= 1;
+          }
+          cidr_ = cidr;
+        }
+      }
+      ifa = ifa->ifa_next;
+    }
+    freeifaddrs(ifaddr);
+  } else {
+    ipaddr_.clear();
+    netmask_.clear();
+    cidr_ = 0;
+  }
+}
+
+int waybar::modules::Network::netlinkRequest(int fd, void *req,
+  uint32_t reqlen, uint32_t groups)
+{
+  struct sockaddr_nl sa = {};
+  sa.nl_family = AF_NETLINK;
+  sa.nl_groups = groups;
+  struct iovec iov = { req, reqlen };
+  struct msghdr msg = { &sa, sizeof(sa), &iov, 1, nullptr, 0, 0 };
+  return sendmsg(fd, &msg, 0);
+}
+
+int waybar::modules::Network::netlinkResponse(int fd, void *resp,
+  uint32_t resplen, uint32_t groups)
+{
+  int ret;
+  struct sockaddr_nl sa = {};
+  sa.nl_family = AF_NETLINK;
+  sa.nl_groups = groups;
+  struct iovec iov = { resp, resplen };
+  struct msghdr msg = { &sa, sizeof(sa), &iov, 1, nullptr, 0, 0 };
+  ret = recvmsg(fd, &msg, 0);
+  if (msg.msg_flags & MSG_TRUNC) {
+    return -1;
+  }
+  return ret;
+}
+
+int waybar::modules::Network::scanCb(struct nl_msg *msg, void *data) {
+  auto net = static_cast<waybar::modules::Network *>(data);
+  auto gnlh = static_cast<genlmsghdr *>(nlmsg_data(nlmsg_hdr(msg)));
+  struct nlattr* tb[NL80211_ATTR_MAX + 1];
+  struct nlattr* bss[NL80211_BSS_MAX + 1];
+  struct nla_policy bss_policy[NL80211_BSS_MAX + 1]{};
+  bss_policy[NL80211_BSS_TSF].type = NLA_U64;
+  bss_policy[NL80211_BSS_FREQUENCY].type = NLA_U32;
+  bss_policy[NL80211_BSS_BSSID].type = NLA_UNSPEC;
+  bss_policy[NL80211_BSS_BEACON_INTERVAL].type = NLA_U16;
+  bss_policy[NL80211_BSS_CAPABILITY].type = NLA_U16;
+  bss_policy[NL80211_BSS_INFORMATION_ELEMENTS].type = NLA_UNSPEC;
+  bss_policy[NL80211_BSS_SIGNAL_MBM].type = NLA_U32;
+  bss_policy[NL80211_BSS_SIGNAL_UNSPEC].type = NLA_U8;
+  bss_policy[NL80211_BSS_STATUS].type = NLA_U32;
+
+  if (nla_parse(tb, NL80211_ATTR_MAX, genlmsg_attrdata(gnlh, 0), genlmsg_attrlen(gnlh, 0), nullptr) < 0) {
+    return NL_SKIP;
+  }
+  if (tb[NL80211_ATTR_BSS] == nullptr) {
+    return NL_SKIP;
+  }
+  if (nla_parse_nested(bss, NL80211_BSS_MAX, tb[NL80211_ATTR_BSS], bss_policy) != 0) {
+    return NL_SKIP;
+  }
+  if (!net->associatedOrJoined(bss)) {
+    return NL_SKIP;
+  }
+  net->parseEssid(bss);
+  net->parseSignal(bss);
+  // TODO(someone): parse quality
+  return NL_SKIP;
+}
+
+void waybar::modules::Network::parseEssid(struct nlattr **bss)
+{
+  essid_.clear();
   if (bss[NL80211_BSS_INFORMATION_ELEMENTS] != nullptr) {
     auto ies =
       static_cast<char*>(nla_data(bss[NL80211_BSS_INFORMATION_ELEMENTS]));
@@ -66,73 +382,53 @@ void waybar::modules::Network::_parseEssid(struct nlattr **bss)
     if (ies_len > hdr_len && ies_len > ies[1] + hdr_len) {
       auto essid_begin = ies + hdr_len;
       auto essid_end = essid_begin + ies[1];
-      std::copy(essid_begin, essid_end, std::back_inserter(_essid));
+      std::copy(essid_begin, essid_end, std::back_inserter(essid_));
     }
   }
 }
 
-void waybar::modules::Network::_parseSignal(struct nlattr **bss) {
-    if (bss[NL80211_BSS_SIGNAL_MBM] != nullptr) {
-      // signalstrength in dBm
-      _signalStrengthdBm =
-        static_cast<int>(nla_get_u32(bss[NL80211_BSS_SIGNAL_MBM])) / 100;
+void waybar::modules::Network::parseSignal(struct nlattr **bss) {
+  if (bss[NL80211_BSS_SIGNAL_MBM] != nullptr) {
+    // signalstrength in dBm
+    signal_strength_dbm_ =
+      static_cast<int>(nla_get_u32(bss[NL80211_BSS_SIGNAL_MBM])) / 100;
 
-      // WiFi-hardware usually operates in the range -90 to -20dBm.
-      const int hardwareMax = -20;
-      const int hardwareMin = -90;
-      _signalStrength = ((double)(_signalStrengthdBm - hardwareMin)
-        / (double)(hardwareMax - hardwareMin)) * 100;
-    }
+    // WiFi-hardware usually operates in the range -90 to -20dBm.
+    const int hardwareMax = -20;
+    const int hardwareMin = -90;
+    signal_strength_ = ((signal_strength_dbm_ - hardwareMin)
+      / double{hardwareMax - hardwareMin}) * 100;
   }
+}
 
-bool waybar::modules::Network::_associatedOrJoined(struct nlattr** bss)
+bool waybar::modules::Network::associatedOrJoined(struct nlattr** bss)
 {
-    if (!bss[NL80211_BSS_STATUS])
+  if (bss[NL80211_BSS_STATUS] == nullptr) {
+    return false;
+  }
+  auto status = nla_get_u32(bss[NL80211_BSS_STATUS]);
+  switch (status) {
+    case NL80211_BSS_STATUS_ASSOCIATED:
+    case NL80211_BSS_STATUS_IBSS_JOINED:
+    case NL80211_BSS_STATUS_AUTHENTICATED:
+      return true;
+    default:
       return false;
-    auto status = nla_get_u32(bss[NL80211_BSS_STATUS]);
-    switch (status) {
-      case NL80211_BSS_STATUS_ASSOCIATED:
-      case NL80211_BSS_STATUS_IBSS_JOINED:
-      case NL80211_BSS_STATUS_AUTHENTICATED:
-        return true;
-      default:
-        return false;
-    }
   }
+}
 
-auto waybar::modules::Network::_getInfo() -> void
+auto waybar::modules::Network::getInfo() -> void
 {
-  if (_ifid == 0)
-    return;
-	struct nl_sock *sk = nl_socket_alloc();
-	if (genl_connect(sk) != 0) {
-    nl_socket_free(sk);
+  struct nl_msg* nl_msg = nlmsg_alloc();
+  if (nl_msg == nullptr) {
+    nl_socket_free(sk_);
     return;
   }
-  if (nl_socket_modify_cb(sk, NL_CB_VALID, NL_CB_CUSTOM, _scanCb, this) < 0) {
-    nl_socket_free(sk);
+  if (genlmsg_put(nl_msg, NL_AUTO_PORT, NL_AUTO_SEQ, nl80211_id_, 0, NLM_F_DUMP,
+    NL80211_CMD_GET_SCAN, 0) == nullptr
+    || nla_put_u32(nl_msg, NL80211_ATTR_IFINDEX, ifid_) < 0) {
+    nlmsg_free(nl_msg);
     return;
   }
-  const int nl80211_id = genl_ctrl_resolve(sk, "nl80211");
-  if (nl80211_id < 0) {
-    nl_socket_free(sk);
-    return;
-  }
-  struct nl_msg *msg = nlmsg_alloc();
-  if (!msg) {
-    nl_socket_free(sk);
-    return;
-  }
-  if (!genlmsg_put(msg, NL_AUTO_PORT, NL_AUTO_SEQ, nl80211_id, 0,
-    NLM_F_DUMP, NL80211_CMD_GET_SCAN, 0) ||
-    nla_put_u32(msg, NL80211_ATTR_IFINDEX, _ifid) < 0) {
-    nlmsg_free(msg);
-    return;
-  }
-  nl_send_sync(sk, msg);
-  nl_socket_free(sk);
-}
-
-waybar::modules::Network::operator Gtk::Widget &() {
-  return _label;
+  nl_send_sync(sk_, nl_msg);
 }

src/modules/pulseaudio.cpp

@@ -1,41 +1,64 @@
 #include "modules/pulseaudio.hpp"
 
-waybar::modules::Pulseaudio::Pulseaudio(Json::Value config)
-  : _config(config), _mainloop(nullptr), _mainloop_api(nullptr),
-    _context(nullptr), _sinkIdx(0), _volume(0), _muted(false)
-{
-  _label.get_style_context()->add_class("pulseaudio");
-  _mainloop = pa_threaded_mainloop_new();
-  if (!_mainloop)
+waybar::modules::Pulseaudio::Pulseaudio(const Json::Value &config)
+    : ALabel(config, "{volume}%"),
+      mainloop_(nullptr),
+      mainloop_api_(nullptr),
+      context_(nullptr),
+      sink_idx_(0),
+      volume_(0),
+      muted_(false),
+      scrolling_(false) {
+  label_.set_name("pulseaudio");
+  mainloop_ = pa_threaded_mainloop_new();
+  if (mainloop_ == nullptr) {
     throw std::runtime_error("pa_mainloop_new() failed.");
-  pa_threaded_mainloop_lock(_mainloop);
-  _mainloop_api = pa_threaded_mainloop_get_api(_mainloop);
-  if (pa_signal_init(_mainloop_api) != 0)
-    throw std::runtime_error("pa_signal_init() failed.");
-  _context = pa_context_new(_mainloop_api, "waybar");
-  if (!_context)
+  }
+  pa_threaded_mainloop_lock(mainloop_);
+  mainloop_api_ = pa_threaded_mainloop_get_api(mainloop_);
+  context_ = pa_context_new(mainloop_api_, "waybar");
+  if (context_ == nullptr) {
     throw std::runtime_error("pa_context_new() failed.");
-  if (pa_context_connect(_context, NULL, PA_CONTEXT_NOAUTOSPAWN, NULL) < 0)
-    throw std::runtime_error(fmt::format("pa_context_connect() failed: {}",
-      pa_strerror(pa_context_errno(_context))));
-  pa_context_set_state_callback(_context, _contextStateCb, this);
-  if (pa_threaded_mainloop_start(_mainloop) < 0)
+  }
+  if (pa_context_connect(context_, nullptr, PA_CONTEXT_NOAUTOSPAWN,
+    nullptr) < 0) {
+    auto err = fmt::format("pa_context_connect() failed: {}",
+      pa_strerror(pa_context_errno(context_)));
+    throw std::runtime_error(err);
+  }
+  pa_context_set_state_callback(context_, contextStateCb, this);
+  if (pa_threaded_mainloop_start(mainloop_) < 0) {
     throw std::runtime_error("pa_mainloop_run() failed.");
-  pa_threaded_mainloop_unlock(_mainloop);
-};
+  }
+  pa_threaded_mainloop_unlock(mainloop_);
 
-void waybar::modules::Pulseaudio::_contextStateCb(pa_context *c, void *data)
+  // define the pulse scroll events only when no user provided
+  // events are configured
+  if (!config["on-scroll-up"].isString() &&
+      !config["on-scroll-down"].isString()) {
+    event_box_.add_events(Gdk::SCROLL_MASK);
+    event_box_.signal_scroll_event().connect(
+        sigc::mem_fun(*this, &Pulseaudio::handleScroll));
+  }
+}
+
+waybar::modules::Pulseaudio::~Pulseaudio() {
+  mainloop_api_->quit(mainloop_api_, 0);
+  pa_threaded_mainloop_stop(mainloop_);
+  pa_threaded_mainloop_free(mainloop_);
+}
+
+void waybar::modules::Pulseaudio::contextStateCb(pa_context *c, void *data)
 {
   auto pa = static_cast<waybar::modules::Pulseaudio *>(data);
   switch (pa_context_get_state(c)) {
     case PA_CONTEXT_TERMINATED:
-      pa->_mainloop_api->quit(pa->_mainloop_api, 0);
+      pa->mainloop_api_->quit(pa->mainloop_api_, 0);
       break;
     case PA_CONTEXT_READY:
-      pa_context_get_server_info(c, _serverInfoCb, data);
-      pa_context_set_subscribe_callback(c, _subscribeCb, data);
-      pa_context_subscribe(c, PA_SUBSCRIPTION_MASK_SINK, nullptr,
-        nullptr);
+      pa_context_get_server_info(c, serverInfoCb, data);
+      pa_context_set_subscribe_callback(c, subscribeCb, data);
+      pa_context_subscribe(c, PA_SUBSCRIPTION_MASK_SINK, nullptr, nullptr);
       break;
     case PA_CONTEXT_CONNECTING:
     case PA_CONTEXT_AUTHORIZING:
@@ -43,46 +66,97 @@ void waybar::modules::Pulseaudio::_contextStateCb(pa_context *c, void *data)
       break;
     case PA_CONTEXT_FAILED:
     default:
-      pa->_mainloop_api->quit(pa->_mainloop_api, 1);
+      pa->mainloop_api_->quit(pa->mainloop_api_, 1);
       break;
   }
 }
 
+bool waybar::modules::Pulseaudio::handleScroll(GdkEventScroll *e) {
+  // Avoid concurrent scroll event
+  bool direction_up = false;
+  uint16_t change = config_["scroll-step"].isUInt() ? config_["scroll-step"].asUInt() * 100 : 100;
+  pa_cvolume pa_volume = pa_volume_;
+
+  if (scrolling_) {
+    return false;
+  }
+  scrolling_ = true;
+  if (e->direction == GDK_SCROLL_UP) {
+    direction_up = true;
+  }
+  if (e->direction == GDK_SCROLL_DOWN) {
+    direction_up = false;
+  }
+
+  if (e->direction == GDK_SCROLL_SMOOTH) {
+    gdouble delta_x, delta_y;
+    gdk_event_get_scroll_deltas(reinterpret_cast<const GdkEvent *>(e), &delta_x,
+                                &delta_y);
+    if (delta_y < 0) {
+      direction_up = true;
+    } else if (delta_y > 0) {
+      direction_up = false;
+    }
+  }
+
+  if (direction_up) {
+    if (volume_ + 1 < 100) pa_cvolume_inc(&pa_volume, change);
+  } else {
+    if (volume_ - 1 > 0) pa_cvolume_dec(&pa_volume, change);
+  }
+
+  pa_context_set_sink_volume_by_index(context_, sink_idx_, &pa_volume,
+                                      volumeModifyCb, this);
+
+  return true;
+}
+
 /*
  * Called when an event we subscribed to occurs.
  */
-void waybar::modules::Pulseaudio::_subscribeCb(pa_context *context,
-  pa_subscription_event_type_t type, uint32_t idx, void *data)
+void waybar::modules::Pulseaudio::subscribeCb(pa_context* context,
+  pa_subscription_event_type_t type, uint32_t idx, void* data)
 {
   unsigned facility = type & PA_SUBSCRIPTION_EVENT_FACILITY_MASK;
-  pa_operation *op = nullptr;
 
   switch (facility) {
-      case PA_SUBSCRIPTION_EVENT_SINK:
-          pa_context_get_sink_info_by_index(context, idx, _sinkInfoCb, data);
-          break;
-      default:
-          assert(0);
-          break;
+    case PA_SUBSCRIPTION_EVENT_SINK:
+        pa_context_get_sink_info_by_index(context, idx, sinkInfoCb, data);
+        break;
+    default:
+        break;
+  }
+}
+
+/*
+ * Called in response to a volume change request
+ */
+void waybar::modules::Pulseaudio::volumeModifyCb(pa_context *c, int success,
+                                                 void *data) {
+  auto pa = static_cast<waybar::modules::Pulseaudio *>(data);
+  if (success) {
+    pa_context_get_sink_info_by_index(pa->context_, pa->sink_idx_, sinkInfoCb,
+                                      data);
   }
-  if (op)
-    pa_operation_unref(op);
 }
 
 /*
  * Called when the requested sink information is ready.
  */
-void waybar::modules::Pulseaudio::_sinkInfoCb(pa_context *context,
-  const pa_sink_info *i, int eol, void *data)
-{
-  if (i) {
+void waybar::modules::Pulseaudio::sinkInfoCb(pa_context * /*context*/,
+                                             const pa_sink_info *i, int /*eol*/,
+                                             void *data) {
+  if (i != nullptr) {
     auto pa = static_cast<waybar::modules::Pulseaudio *>(data);
-    float volume = (float)pa_cvolume_avg(&(i->volume)) / (float)PA_VOLUME_NORM;
-    pa->_sinkIdx = i->index;
-    pa->_volume = volume * 100.0f;
-    pa->_muted = i->mute;
-    pa->_desc = i->description;
-    pa->update();
+    pa->pa_volume_ = i->volume;
+    float volume = static_cast<float>(pa_cvolume_avg(&(pa->pa_volume_))) /
+                   float{PA_VOLUME_NORM};
+    pa->sink_idx_ = i->index;
+    pa->volume_ = std::round(volume * 100.0f);
+    pa->muted_ = i->mute != 0;
+    pa->desc_ = i->description;
+    pa->port_name_ = i->active_port ? i->active_port->name : "Unknown";
+    pa->dp.emit();
   }
 }
 
@@ -90,27 +164,54 @@ void waybar::modules::Pulseaudio::_sinkInfoCb(pa_context *context,
  * Called when the requested information on the server is ready. This is
  * used to find the default PulseAudio sink.
  */
-void waybar::modules::Pulseaudio::_serverInfoCb(pa_context *context,
+void waybar::modules::Pulseaudio::serverInfoCb(pa_context *context,
   const pa_server_info *i, void *data)
 {
-    pa_context_get_sink_info_by_name(context, i->default_sink_name, _sinkInfoCb,
-      data);
+  pa_context_get_sink_info_by_name(context, i->default_sink_name,
+    sinkInfoCb, data);
+}
+
+const std::string waybar::modules::Pulseaudio::getPortIcon() const
+{
+  std::vector<std::string> ports = {
+    "headphones",
+    "speaker",
+    "hdmi",
+    "headset",
+    "handsfree",
+    "portable",
+    "car",
+    "hifi",
+    "phone",
+  };
+  for (auto const& port : ports) {
+    if (port_name_.find(port) != std::string::npos) {
+      return port;
+    }
+  }
+  return "";
 }
 
 auto waybar::modules::Pulseaudio::update() -> void
 {
-	  auto format = _config["format"] ? _config["format"].asString() : "{}%";
-    if (_muted) {
-      format =
-        _config["format-muted"] ? _config["format-muted"].asString() : format;
-      if (!_label.get_style_context()->has_class("muted"))
-        _label.get_style_context()->add_class("muted");
-    } else if (_label.get_style_context()->has_class("muted"))
-      _label.get_style_context()->remove_class("muted");
-    _label.set_label(fmt::format(format, _volume));
-    _label.set_tooltip_text(_desc);
-}
-
-waybar::modules::Pulseaudio::operator Gtk::Widget &() {
-  return _label;
+  auto format = format_;
+  if (muted_) {
+    format =
+      config_["format-muted"].isString() ? config_["format-muted"].asString() : format;
+    label_.get_style_context()->add_class("muted");
+  } else if (port_name_.find("a2dp_sink") != std::string::npos) {
+    format = config_["format-bluetooth"].isString()
+      ? config_["format-bluetooth"].asString() : format;
+    label_.get_style_context()->add_class("bluetooth");
+  } else {
+    label_.get_style_context()->remove_class("muted");
+    label_.get_style_context()->add_class("bluetooth");
+  }
+  label_.set_label(
+      fmt::format(format, fmt::arg("volume", volume_),
+                  fmt::arg("icon", getIcon(volume_, getPortIcon()))));
+  label_.set_tooltip_text(desc_);
+  if (scrolling_) {
+    scrolling_ = false;
+  }
 }

src/modules/sni/snh.cpp

@@ -0,0 +1,151 @@
+#include "modules/sni/snh.hpp"
+
+#include <iostream>
+
+using namespace waybar::modules::SNI;
+
+Host::Host(Glib::Dispatcher* dp, const Json::Value &config)
+: dp_(dp), config_(config)
+{
+  GBusNameOwnerFlags flags = static_cast<GBusNameOwnerFlags>(
+    G_BUS_NAME_OWNER_FLAGS_NONE);
+  bus_name_ = "org.kde.StatusNotifierHost-" + std::to_string(getpid());
+  object_path_ = "/StatusNotifierHost";
+  bus_name_id_ = g_bus_own_name(G_BUS_TYPE_SESSION,
+    bus_name_.c_str(), flags,
+    &Host::busAcquired, nullptr, nullptr, this, nullptr);
+}
+
+void Host::busAcquired(GDBusConnection* connection,
+  const gchar* name, gpointer data)
+{
+  auto host = static_cast<SNI::Host *>(data);
+  host->watcher_id_ = g_bus_watch_name(
+    G_BUS_TYPE_SESSION,
+    "org.kde.StatusNotifierWatcher",
+    G_BUS_NAME_WATCHER_FLAGS_NONE,
+    &Host::nameAppeared, &Host::nameVanished, data, nullptr);
+}
+
+void Host::nameAppeared(GDBusConnection* connection,
+  const gchar* name, const gchar* name_owner, gpointer data)
+{
+  auto host = static_cast<SNI::Host *>(data);
+  if (host->cancellable_ != nullptr) {
+    // TODO
+    return;
+  }
+  host->cancellable_ = g_cancellable_new();
+  sn_watcher_proxy_new(
+    connection,
+    G_DBUS_PROXY_FLAGS_NONE,
+    "org.kde.StatusNotifierWatcher",
+    "/StatusNotifierWatcher",
+    host->cancellable_, &Host::proxyReady, data);
+}
+
+void Host::nameVanished(GDBusConnection* connection,
+  const gchar* name, gpointer data)
+{
+  auto host = static_cast<SNI::Host *>(data);
+  g_cancellable_cancel(host->cancellable_);
+  g_clear_object(&host->cancellable_);
+  g_clear_object(&host->watcher_);
+  host->items.clear();
+}
+
+void Host::proxyReady(GObject* src, GAsyncResult* res,
+  gpointer data)
+{
+  GError* error = nullptr;
+  SnWatcher* watcher = sn_watcher_proxy_new_finish(res, &error);
+  if (g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED)) {
+    std::cerr << error->message << std::endl;
+    g_error_free(error);
+    return;
+  }
+  auto host = static_cast<SNI::Host *>(data);
+  host->watcher_ = watcher;
+  if (error != nullptr) {
+    std::cerr << error->message << std::endl;
+    g_error_free(error);
+    return;
+  }
+  sn_watcher_call_register_host(
+    host->watcher_, host->object_path_.c_str(), host->cancellable_,
+    &Host::registerHost, data);
+}
+
+void Host::registerHost(GObject* src, GAsyncResult* res,
+  gpointer data)
+{
+  GError* error = nullptr;
+  sn_watcher_call_register_host_finish(SN_WATCHER(src), res, &error);
+  if (g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED)) {
+    std::cerr << error->message << std::endl;
+    g_error_free(error);
+    return;
+  }
+  auto host = static_cast<SNI::Host *>(data);
+  if (error != nullptr) {
+    std::cerr << error->message << std::endl;
+    g_error_free(error);
+    return; 
+  }
+  g_signal_connect(host->watcher_, "item-registered",
+    G_CALLBACK(&Host::itemRegistered), data);
+  g_signal_connect(host->watcher_, "item-unregistered",
+    G_CALLBACK(&Host::itemUnregistered), data);
+  auto items = sn_watcher_dup_registered_items(host->watcher_);
+  if (items) {
+    for (uint32_t i = 0; items[i] != nullptr; i += 1) {
+      host->addRegisteredItem(items[i]);
+    }
+  }
+  g_strfreev(items);
+}
+
+void Host::itemRegistered(
+  SnWatcher* watcher, const gchar* service, gpointer data)
+{
+  auto host = static_cast<SNI::Host *>(data);
+  host->addRegisteredItem(service);
+}
+
+void Host::itemUnregistered(
+  SnWatcher* watcher, const gchar* service, gpointer data)
+{
+  auto host = static_cast<SNI::Host *>(data);
+  auto [bus_name, object_path] = host->getBusNameAndObjectPath(service);
+  for (auto it = host->items.begin(); it != host->items.end(); ++it) {
+    if (it->bus_name == bus_name && it->object_path == object_path) {
+      host->items.erase(it);
+      break;
+    }
+  }
+  host->dp_->emit();
+}
+
+std::tuple<std::string, std::string> Host::getBusNameAndObjectPath(
+  const gchar* service)
+{
+  std::string bus_name;
+  std::string object_path;
+  gchar* tmp = g_strstr_len(service, -1, "/");
+  if (tmp != nullptr) {
+    gchar** str = g_strsplit(service, "/", 2);
+    bus_name = str[0];
+    object_path = tmp;
+    g_strfreev(str);
+  } else {
+    bus_name = service;
+    object_path = "/StatusNotifierItem";
+  }
+  return { bus_name, object_path };
+}
+
+void Host::addRegisteredItem(const gchar* service)
+{
+  auto [bus_name, object_path] = getBusNameAndObjectPath(service);
+  items.emplace_back(bus_name, object_path, dp_, config_);
+}

src/modules/sni/sni.cpp

@@ -0,0 +1,278 @@
+#include "modules/sni/sni.hpp"
+
+#include <iostream>
+#include <libdbusmenu-gtk/dbusmenu-gtk.h>
+
+waybar::modules::SNI::Item::Item(std::string bn, std::string op,
+  Glib::Dispatcher *dp, Json::Value config)
+    : bus_name(bn), object_path(op), event_box(), icon_size(16),
+      effective_icon_size(0), image(Gtk::manage(new Gtk::Image())),
+      dp_(dp), config_(config)
+{
+  if (config_["icon-size"].isUInt()) {
+    icon_size = config_["icon-size"].asUInt();
+  }
+  event_box.add(*image);
+  event_box.add_events(Gdk::BUTTON_PRESS_MASK);
+  event_box.signal_button_press_event().connect(
+      sigc::mem_fun(*this, &Item::handleClick));
+  cancellable_ = g_cancellable_new();
+  sn_item_proxy_new_for_bus(
+      G_BUS_TYPE_SESSION, G_DBUS_PROXY_FLAGS_NONE, bus_name.c_str(),
+      object_path.c_str(), cancellable_, &Item::proxyReady, this);
+}
+
+void waybar::modules::SNI::Item::proxyReady(GObject *obj, GAsyncResult *res,
+                                            gpointer data) {
+  GError *error = nullptr;
+  SnItem *proxy = sn_item_proxy_new_for_bus_finish(res, &error);
+  if (g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED)) {
+    g_error_free(error);
+    return;
+  }
+  auto item = static_cast<SNI::Item *>(data);
+  item->proxy_ = proxy;
+  if (error) {
+    std::cerr << error->message << std::endl;
+    g_error_free(error);
+    return;
+  }
+  auto conn = g_dbus_proxy_get_connection(G_DBUS_PROXY(proxy));
+
+  g_dbus_connection_call(conn, item->bus_name.c_str(),
+                         item->object_path.c_str(),
+                         "org.freedesktop.DBus.Properties", "GetAll",
+                         g_variant_new("(s)", "org.kde.StatusNotifierItem"),
+                         G_VARIANT_TYPE("(a{sv})"), G_DBUS_CALL_FLAGS_NONE, -1,
+                         item->cancellable_, &Item::getAll, data);
+}
+
+void waybar::modules::SNI::Item::getAll(GObject *obj, GAsyncResult *res,
+                                        gpointer data) {
+  GError *error = nullptr;
+  auto conn = G_DBUS_CONNECTION(obj);
+  GVariant *properties = g_dbus_connection_call_finish(conn, res, &error);
+  if (g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED)) {
+    g_error_free(error);
+    return;
+  }
+  auto item = static_cast<SNI::Item *>(data);
+  if (error) {
+    std::cerr << error->message << std::endl;
+    g_error_free(error);
+    return;
+  }
+  GVariantIter *it = nullptr;
+  g_variant_get(properties, "(a{sv})", &it);
+  gchar *key;
+  GVariant *value;
+  while (g_variant_iter_next(it, "{sv}", &key, &value)) {
+    if (g_strcmp0(key, "Category") == 0) {
+      item->category = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "Id") == 0) {
+      item->id = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "Title") == 0) {
+      item->title = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "Status") == 0) {
+      item->status = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "WindowId") == 0) {
+      item->window_id = g_variant_get_int32(value);
+    } else if (g_strcmp0(key, "IconName") == 0) {
+      item->icon_name = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "IconPixmap") == 0) {
+      item->icon_pixmap = item->extractPixBuf(value);
+    } else if (g_strcmp0(key, "OverlayIconName") == 0) {
+      item->overlay_icon_name = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "OverlayIconPixmap") == 0) {
+      // TODO: overlay_icon_pixmap
+    } else if (g_strcmp0(key, "AttentionIconName") == 0) {
+      item->attention_icon_name = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "AttentionIconPixmap") == 0) {
+      // TODO: attention_icon_pixmap
+    } else if (g_strcmp0(key, "AttentionMovieName") == 0) {
+      item->attention_movie_name = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "ToolTip") == 0) {
+      // TODO: tooltip
+    } else if (g_strcmp0(key, "IconThemePath") == 0) {
+      item->icon_theme_path = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "Menu") == 0) {
+      item->menu = g_variant_dup_string(value, nullptr);
+    } else if (g_strcmp0(key, "ItemIsMenu") == 0) {
+      item->item_is_menu = g_variant_get_boolean(value);
+    }
+    g_variant_unref(value);
+    g_free(key);
+  }
+  g_variant_iter_free(it);
+  g_variant_unref(properties);
+  if (item->id.empty() || item->category.empty() || item->status.empty()) {
+    std::cerr << "Invalid Status Notifier Item: " + item->bus_name + "," +
+                     item->object_path
+              << std::endl;
+    return;
+  }
+  if (!item->icon_theme_path.empty()) {
+    GtkIconTheme *icon_theme = gtk_icon_theme_get_default();
+    gtk_icon_theme_append_search_path(icon_theme,
+                                      item->icon_theme_path.c_str());
+  }
+  item->updateImage();
+  item->updateMenu();
+  item->dp_->emit();
+  // TODO: handle change
+}
+
+Glib::RefPtr<Gdk::Pixbuf>
+waybar::modules::SNI::Item::extractPixBuf(GVariant *variant) {
+  GVariantIter *it;
+  g_variant_get(variant, "a(iiay)", &it);
+  if (it == nullptr) {
+    return Glib::RefPtr<Gdk::Pixbuf>{};
+  }
+  GVariant *val;
+  gint lwidth = 0;
+  gint lheight = 0;
+  gint width;
+  gint height;
+  guchar *array = nullptr;
+  while (g_variant_iter_loop(it, "(ii@ay)", &width, &height, &val)) {
+    if (width > 0 && height > 0 && val != nullptr &&
+        width * height > lwidth * lheight) {
+      auto size = g_variant_get_size(val);
+      /* Sanity check */
+      if (size == 4U * width * height) {
+        /* Find the largest image */
+        gconstpointer data = g_variant_get_data(val);
+        if (data != nullptr) {
+          if (array != nullptr) {
+            g_free(array);
+          }
+          array = static_cast<guchar *>(g_memdup(data, size));
+          lwidth = width;
+          lheight = height;
+        }
+      }
+    }
+  }
+  g_variant_iter_free(it);
+  if (array != nullptr) {
+    /* argb to rgba */
+    for (uint32_t i = 0; i < 4U * lwidth * lheight; i += 4) {
+      guchar alpha = array[i];
+      array[i] = array[i + 1];
+      array[i + 1] = array[i + 2];
+      array[i + 2] = array[i + 3];
+      array[i + 3] = alpha;
+    }
+    return Gdk::Pixbuf::create_from_data(array, Gdk::Colorspace::COLORSPACE_RGB,
+                                         true, 8, lwidth, lheight, 4 * lwidth);
+  }
+  return Glib::RefPtr<Gdk::Pixbuf>{};
+}
+
+void waybar::modules::SNI::Item::updateMenu()
+{
+  event_box.set_tooltip_text(title);
+  if (gtk_menu == nullptr && !menu.empty()) {
+    auto dbmenu = dbusmenu_gtkmenu_new(bus_name.data(), menu.data());
+    if (dbmenu != nullptr) {
+      g_object_ref_sink(dbmenu);
+      gtk_menu = Glib::wrap(GTK_MENU(dbmenu), false);
+      gtk_menu->attach_to_widget(event_box);
+    }
+  }
+}
+
+void waybar::modules::SNI::Item::updateImage()
+{
+  image->set_from_icon_name("image-missing", Gtk::ICON_SIZE_MENU);
+  image->set_pixel_size(icon_size);
+  if (!icon_name.empty()) {
+    try {
+      // Try to find icons specified by path and filename
+#ifdef FILESYSTEM_EXPERIMENTAL
+      if (std::experimental::filesystem::exists(icon_name)) {
+#else
+      if (std::filesystem::exists(icon_name)) {
+#endif
+        auto pixbuf = Gdk::Pixbuf::create_from_file(icon_name);
+        if (pixbuf->gobj() != nullptr) {
+          // An icon specified by path and filename may be the wrong size for
+          // the tray
+          pixbuf = pixbuf->scale_simple(icon_size, icon_size,
+            Gdk::InterpType::INTERP_BILINEAR);
+          image->set(pixbuf);
+        }
+      } else {
+        image->set(getIconByName(icon_name, icon_size));
+      }
+    } catch (Glib::Error &e) {
+      std::cerr << "Exception: " << e.what() << std::endl;
+    }
+  } else if (icon_pixmap) {
+    // An icon extracted may be the wrong size for the tray
+    icon_pixmap = icon_pixmap->scale_simple(icon_size, icon_size,
+      Gdk::InterpType::INTERP_BILINEAR);
+    image->set(icon_pixmap);
+  }
+}
+
+Glib::RefPtr<Gdk::Pixbuf>
+waybar::modules::SNI::Item::getIconByName(std::string name, int request_size) {
+  int tmp_size = 0;
+  Glib::RefPtr<Gtk::IconTheme> icon_theme = Gtk::IconTheme::get_default();
+  icon_theme->rescan_if_needed();
+  auto sizes = icon_theme->get_icon_sizes(name.c_str());
+
+  for (auto const &size : sizes) {
+    // -1 == scalable
+    if (size == request_size || size == -1) {
+      tmp_size = request_size;
+      break;
+    } else if (size < request_size || size > tmp_size) {
+      tmp_size = size;
+    }
+  }
+  if (tmp_size == 0) {
+    tmp_size = request_size;
+  }
+  return icon_theme->load_icon(name.c_str(), tmp_size,
+                               Gtk::IconLookupFlags::ICON_LOOKUP_FORCE_SIZE);
+}
+
+void waybar::modules::SNI::Item::handleActivate(GObject *src, GAsyncResult *res,
+                                                gpointer data) {
+  auto item = static_cast<SNI::Item *>(data);
+  sn_item_call_activate_finish(item->proxy_, res, nullptr);
+}
+
+void waybar::modules::SNI::Item::handleSecondaryActivate(GObject *src,
+                                                         GAsyncResult *res,
+                                                         gpointer data) {
+  auto item = static_cast<SNI::Item *>(data);
+  sn_item_call_secondary_activate_finish(item->proxy_, res, nullptr);
+}
+
+bool waybar::modules::SNI::Item::handleClick(GdkEventButton *const &ev) {
+  if (ev->type == GDK_BUTTON_PRESS) {
+    if (gtk_menu && gtk_menu->get_children().size() > 0) {
+      #if GTK_CHECK_VERSION(3, 22, 0)
+      gtk_menu->popup_at_widget(reinterpret_cast<Gtk::Widget*>(&event_box),
+        Gdk::GRAVITY_NORTH_WEST, Gdk::GRAVITY_NORTH_WEST,
+        reinterpret_cast<GdkEvent*>(ev));
+      #else
+      gtk_menu->popup(ev->button, ev->time);
+      #endif
+      gtk_menu->set_state_flags(Gtk::STATE_FLAG_ACTIVE, false);
+    } else {
+      sn_item_call_activate(
+          proxy_, ev->x, ev->y, nullptr, &Item::handleActivate, this);
+    }
+  } else if (ev->type == GDK_2BUTTON_PRESS) {
+    sn_item_call_secondary_activate(
+        proxy_, ev->x, ev->y, nullptr, &Item::handleSecondaryActivate, this);
+  } else {
+    return false;
+  }
+  return true;
+}

src/modules/sni/snw.cpp

@@ -0,0 +1,167 @@
+#include "modules/sni/snw.hpp"
+
+#include <iostream>
+
+using namespace waybar::modules::SNI;
+
+Watcher::Watcher()
+{
+  GBusNameOwnerFlags flags = static_cast<GBusNameOwnerFlags>(
+    G_BUS_NAME_OWNER_FLAGS_ALLOW_REPLACEMENT
+    | G_BUS_NAME_OWNER_FLAGS_REPLACE);
+  bus_name_id_ = g_bus_own_name(G_BUS_TYPE_SESSION,
+    "org.kde.StatusNotifierWatcher", flags,
+    &Watcher::busAcquired, nullptr, nullptr, this, nullptr);
+  watcher_ = sn_watcher_skeleton_new();
+  sn_watcher_set_protocol_version(watcher_, 1);
+}
+
+Watcher::~Watcher()
+{
+}
+
+void Watcher::busAcquired(GDBusConnection* connection, const gchar* name,
+  gpointer data)
+{
+  GError* error = nullptr;
+  auto host = static_cast<SNI::Watcher*>(data);
+  g_dbus_interface_skeleton_export(G_DBUS_INTERFACE_SKELETON(host->watcher_),
+    connection, "/StatusNotifierWatcher", &error);
+  if (error != nullptr) {
+    std::cerr << error->message << std::endl;
+    g_error_free(error);
+    return;
+  }
+  g_signal_connect_swapped(host->watcher_, "handle-register-item",
+    G_CALLBACK(&Watcher::handleRegisterItem), data);
+  g_signal_connect_swapped(host->watcher_, "handle-register-host",
+    G_CALLBACK(&Watcher::handleRegisterHost), data);
+}
+
+gboolean Watcher::handleRegisterHost(Watcher* obj,
+  GDBusMethodInvocation* invocation, const gchar* service)
+{
+  const gchar* bus_name = service;
+  const gchar* object_path = "/StatusNotifierHost";
+
+  if (*service == '/') {
+    bus_name = g_dbus_method_invocation_get_sender(invocation);
+    object_path = service;
+  }
+  if (g_dbus_is_name(bus_name) == FALSE) {
+    g_dbus_method_invocation_return_error(invocation, G_DBUS_ERROR,
+      G_DBUS_ERROR_INVALID_ARGS, "D-Bus bus name '%s' is not valid", bus_name);
+    return TRUE;
+  }
+  auto watch = gfWatchFind(obj->hosts_, bus_name, object_path);
+  if (watch != nullptr) {
+    g_dbus_method_invocation_return_error(invocation, G_DBUS_ERROR,
+      G_DBUS_ERROR_INVALID_ARGS, "Status Notifier Host with bus name '%s' and object path '%s' is already registered",
+      bus_name, object_path);
+    return TRUE;
+  }
+  watch = gfWatchNew(GF_WATCH_TYPE_HOST, service, bus_name, object_path, obj);
+  obj->hosts_ = g_slist_prepend(obj->hosts_, watch);
+  if (!sn_watcher_get_is_host_registered(obj->watcher_)) {
+    sn_watcher_set_is_host_registered(obj->watcher_, TRUE);
+    sn_watcher_emit_host_registered(obj->watcher_);
+  }
+  sn_watcher_complete_register_host(obj->watcher_, invocation);
+  return TRUE;
+}
+
+gboolean Watcher::handleRegisterItem(Watcher* obj,
+  GDBusMethodInvocation* invocation, const gchar* service)
+{
+  const gchar* bus_name = service;
+  const gchar* object_path = "/StatusNotifierItem";
+
+  if (*service == '/') {
+    bus_name = g_dbus_method_invocation_get_sender(invocation);
+    object_path = service;
+  }
+  if (g_dbus_is_name(bus_name) == FALSE) {
+    g_dbus_method_invocation_return_error(invocation, G_DBUS_ERROR,
+      G_DBUS_ERROR_INVALID_ARGS, "D-Bus bus name '%s' is not valid", bus_name);
+    return TRUE;
+  }
+  auto watch = gfWatchFind(obj->items_, bus_name, object_path);
+  if (watch != nullptr) {
+    g_warning("Status Notifier Item with bus name '%s' and object path '%s' is already registered",
+      bus_name, object_path);
+    sn_watcher_complete_register_item(obj->watcher_, invocation);
+    return TRUE;
+  }
+  watch = gfWatchNew(GF_WATCH_TYPE_ITEM, service, bus_name, object_path, obj);
+  obj->items_ = g_slist_prepend(obj->items_, watch);
+  obj->updateRegisteredItems(obj->watcher_);
+  gchar* tmp = g_strdup_printf("%s%s", bus_name, object_path);
+  sn_watcher_emit_item_registered(obj->watcher_, tmp);
+  g_free(tmp);
+  sn_watcher_complete_register_item(obj->watcher_, invocation);
+  return TRUE;
+}
+
+Watcher::GfWatch* Watcher::gfWatchFind(GSList* list, const gchar* bus_name,
+  const gchar* object_path)
+{
+  for (GSList* l = list; l != nullptr; l = g_slist_next (l)) {
+    GfWatch* watch = static_cast<GfWatch*>(l->data);
+    if (g_strcmp0 (watch->bus_name, bus_name) == 0
+      && g_strcmp0 (watch->object_path, object_path) == 0) {
+      return watch;
+    }
+  }
+  return nullptr;
+}
+
+Watcher::GfWatch* Watcher::gfWatchNew(GfWatchType type, const gchar* service,
+  const gchar* bus_name, const gchar* object_path, Watcher* watcher)
+{
+  GfWatch* watch = g_new0(GfWatch, 1);
+  watch->type = type;
+  watch->watcher = watcher;
+  watch->service = g_strdup(service);
+  watch->bus_name = g_strdup(bus_name);
+  watch->object_path = g_strdup(object_path);
+  watch->watch_id = g_bus_watch_name(G_BUS_TYPE_SESSION, bus_name,
+    G_BUS_NAME_WATCHER_FLAGS_NONE, nullptr, &Watcher::nameVanished, watch,
+    nullptr);
+  return watch;
+}
+
+void Watcher::nameVanished(GDBusConnection* connection, const char* name,
+  gpointer data)
+{
+  auto watch = static_cast<GfWatch *>(data);
+  if (watch->type == GF_WATCH_TYPE_HOST) {
+    watch->watcher->hosts_ = g_slist_remove(watch->watcher->hosts_, watch);
+    if (watch->watcher->hosts_ == nullptr)  {
+      sn_watcher_set_is_host_registered(watch->watcher->watcher_, FALSE);
+      sn_watcher_emit_host_registered(watch->watcher->watcher_);
+    }
+  } else if (watch->type == GF_WATCH_TYPE_ITEM) {
+    watch->watcher->items_ = g_slist_remove(watch->watcher->items_, watch);
+    watch->watcher->updateRegisteredItems(watch->watcher->watcher_);
+    gchar* tmp = g_strdup_printf("%s%s", watch->bus_name, watch->object_path);
+    sn_watcher_emit_item_unregistered(watch->watcher->watcher_, tmp);
+    g_free(tmp);
+  }
+}
+
+void Watcher::updateRegisteredItems(SnWatcher* obj)
+{
+  GVariantBuilder builder;
+  g_variant_builder_init(&builder, G_VARIANT_TYPE("as")); 
+  for (GSList* l = items_; l != nullptr; l = g_slist_next(l)) {
+    GfWatch* watch = static_cast<GfWatch*>(l->data);
+    gchar* item = g_strdup_printf("%s%s", watch->bus_name, watch->object_path);
+    g_variant_builder_add(&builder, "s", item);
+    g_free(item);
+  }
+  GVariant* variant = g_variant_builder_end(&builder);
+  const gchar** items = g_variant_get_strv(variant, nullptr);
+  sn_watcher_set_registered_items(obj, items);
+  g_variant_unref(variant);
+  g_free(items);
+}

src/modules/sni/tray.cpp

@@ -0,0 +1,30 @@
+#include "modules/sni/tray.hpp"
+
+#include <iostream>
+
+waybar::modules::SNI::Tray::Tray(const Json::Value &config)
+    : config_(config), watcher_(), host_(&dp, config)
+{
+  std::cout << "Tray is in beta, so there may be bugs or even be unusable." << std::endl;
+  if (config_["spacing"].isUInt()) {
+    box_.set_spacing(config_["spacing"].asUInt());
+  }
+}
+
+auto waybar::modules::SNI::Tray::update() -> void {
+  auto childrens = box_.get_children();
+  childrens.erase(childrens.begin(), childrens.end());
+  for (auto &item : host_.items) {
+    box_.pack_start(item.event_box);
+  }
+  if (box_.get_children().size() > 0) {
+    box_.set_name("tray");
+    box_.show_all();
+  } else {
+    box_.set_name("");
+  }
+}
+
+waybar::modules::SNI::Tray::operator Gtk::Widget &() {
+  return box_;
+}

src/modules/sway/ipc/client.cpp

@@ -0,0 +1,134 @@
+#include "modules/sway/ipc/client.hpp"
+
+waybar::modules::sway::Ipc::Ipc()
+  : fd_(-1), fd_event_(-1)
+{}
+
+waybar::modules::sway::Ipc::~Ipc()
+{
+  close(fd_);
+  close(fd_event_);
+}
+
+const std::string waybar::modules::sway::Ipc::getSocketPath() const
+{
+  const char *env = getenv("SWAYSOCK");
+  if (env != nullptr) {
+    return std::string(env);
+  }
+  std::string str;
+  {
+    std::string str_buf;
+    FILE*  in;
+    char  buf[512] = { 0 };
+    if ((in = popen("sway --get-socketpath 2>/dev/null", "r")) == nullptr) {
+      throw std::runtime_error("Failed to get socket path");
+    }
+    while (fgets(buf, sizeof(buf), in) != nullptr) {
+      str_buf.append(buf, sizeof(buf));
+    }
+    pclose(in);
+    str = str_buf;
+  }
+  if (str.back() == '\n') {
+    str.pop_back();
+  }
+  return str;
+}
+
+int waybar::modules::sway::Ipc::open(const std::string& socketPath) const
+{
+  struct sockaddr_un addr = {0};
+  int fd = -1;
+  if ((fd = socket(AF_UNIX, SOCK_STREAM, 0)) == -1) {
+    throw std::runtime_error("Unable to open Unix socket");
+  }
+  addr.sun_family = AF_UNIX;
+  strncpy(addr.sun_path, socketPath.c_str(), sizeof(addr.sun_path) - 1);
+  addr.sun_path[sizeof(addr.sun_path) - 1] = 0;
+  int l = sizeof(struct sockaddr_un);
+  if (::connect(fd, reinterpret_cast<struct sockaddr *>(&addr), l) == -1) {
+    throw std::runtime_error("Unable to connect to Sway");
+  }
+  return fd;
+}
+
+void waybar::modules::sway::Ipc::connect()
+{
+  const std::string& socketPath = getSocketPath();
+  fd_ = open(socketPath);
+  fd_event_ = open(socketPath);
+}
+
+struct waybar::modules::sway::Ipc::ipc_response
+  waybar::modules::sway::Ipc::recv(int fd) const
+{
+  std::string header;
+  header.reserve(ipc_header_size_);
+  auto data32 = reinterpret_cast<uint32_t *>(header.data() + ipc_magic_.size());
+  size_t total = 0;
+
+  while (total < ipc_header_size_) {
+    ssize_t res =
+      ::recv(fd, header.data() + total, ipc_header_size_ - total, 0);
+    if (res <= 0) {
+      throw std::runtime_error("Unable to receive IPC response");
+    }
+    total += res;
+  }
+
+  total = 0;
+  std::string payload;
+  payload.reserve(data32[0] + 1);
+  while (total < data32[0]) {
+    ssize_t res =
+      ::recv(fd, payload.data() + total, data32[0] - total, 0);
+    if (res < 0) {
+      throw std::runtime_error("Unable to receive IPC response");
+    }
+    total += res;
+  }
+  payload[data32[0]] = 0;
+  return { data32[0], data32[1], &payload.front() };
+}
+
+struct waybar::modules::sway::Ipc::ipc_response
+  waybar::modules::sway::Ipc::send(int fd, uint32_t type,
+  const std::string& payload) const
+{
+  std::string header;
+  header.reserve(ipc_header_size_);
+  auto data32 = reinterpret_cast<uint32_t *>(header.data() + ipc_magic_.size());
+  memcpy(header.data(), ipc_magic_.c_str(), ipc_magic_.size());
+  data32[0] = payload.size();
+  data32[1] = type;
+
+  if (::send(fd, header.data(), ipc_header_size_, 0) == -1) {
+    throw std::runtime_error("Unable to send IPC header");
+  }
+  if (::send(fd, payload.c_str(), payload.size(), 0) == -1) {
+    throw std::runtime_error("Unable to send IPC payload");
+  }
+  return recv(fd);
+}
+
+struct waybar::modules::sway::Ipc::ipc_response
+  waybar::modules::sway::Ipc::sendCmd(uint32_t type,
+  const std::string& payload) const
+{
+  return send(fd_, type, payload);
+}
+
+void waybar::modules::sway::Ipc::subscribe(const std::string& payload) const
+{
+  auto res = send(fd_event_, IPC_SUBSCRIBE, payload);
+  if (res.payload != "{\"success\": true}") {
+    throw std::runtime_error("Unable to subscribe ipc event");
+  }
+}
+
+struct waybar::modules::sway::Ipc::ipc_response
+  waybar::modules::sway::Ipc::handleEvent() const
+{
+  return recv(fd_event_);
+}

src/modules/sway/mode.cpp

@@ -0,0 +1,43 @@
+#include "modules/sway/mode.hpp"
+
+waybar::modules::sway::Mode::Mode(Bar& bar, const Json::Value& config)
+  : ALabel(config, "{}"), bar_(bar)
+{
+  ipc_.connect();
+  ipc_.subscribe("[ \"mode\" ]");
+  // Launch worker
+  worker();
+}
+
+void waybar::modules::sway::Mode::worker()
+{
+  thread_ = [this] {
+    try {
+      auto res = ipc_.handleEvent();
+      auto parsed = parser_.parse(res.payload);
+      if ((parsed["change"]) != "default" ) {
+        mode_ = parsed["change"].asString();
+        dp.emit();
+      }
+      else if ((parsed["change"]) == "default" ) {
+        mode_.clear();
+        dp.emit();
+      }
+    } catch (const std::exception& e) {
+      std::cerr << e.what() << std::endl;
+    }
+  };
+}
+
+auto waybar::modules::sway::Mode::update() -> void
+{
+  if (mode_.empty()) {
+    label_.set_name("");
+    label_.hide();
+  } else {
+    label_.set_name("mode");
+    label_.set_text(fmt::format(format_, mode_));
+    label_.set_tooltip_text(mode_);
+    label_.show();
+  }
+}

src/modules/sway/window.cpp

@@ -0,0 +1,73 @@
+#include "modules/sway/window.hpp"
+
+waybar::modules::sway::Window::Window(Bar &bar, const Json::Value& config)
+  : ALabel(config, "{}"), bar_(bar), windowId_(-1)
+{
+  label_.set_name("window");
+  ipc_.connect();
+  ipc_.subscribe("[\"window\",\"workspace\"]");
+  getFocusedWindow();
+  // Launch worker
+  worker();
+}
+
+void waybar::modules::sway::Window::worker()
+{
+  thread_ = [this] {
+    try {
+      auto res = ipc_.handleEvent();
+      auto parsed = parser_.parse(res.payload);
+      if ((parsed["change"] == "focus" || parsed["change"] == "title")
+        && parsed["container"]["focused"].asBool()) {
+        window_ = parsed["container"]["name"].asString();
+        windowId_ = parsed["container"]["id"].asInt();
+        dp.emit();
+      } else if ((parsed["change"] == "close"
+        && parsed["container"]["focused"].asBool()
+        && windowId_ == parsed["container"]["id"].asInt())
+        || (parsed["change"] == "focus" && parsed["current"]["focus"].isArray()
+        && parsed["current"]["focus"].empty())) {
+        window_.clear();
+        windowId_ = -1;
+        dp.emit();
+      }
+    } catch (const std::exception& e) {
+      std::cerr << e.what() << std::endl;
+    }
+  };
+}
+
+auto waybar::modules::sway::Window::update() -> void
+{
+  label_.set_text(fmt::format(format_, window_));
+  label_.set_tooltip_text(window_);
+}
+
+std::tuple<int, std::string> waybar::modules::sway::Window::getFocusedNode(
+  Json::Value nodes)
+{
+  for (auto const& node : nodes) {
+    if (node["focused"].asBool() && node["type"] == "con") {
+      return { node["id"].asInt(), node["name"].asString() };
+    }
+    auto [id, name] = getFocusedNode(node["nodes"]);
+    if (id > -1 && !name.empty()) {
+      return { id, name };
+    }
+  }
+  return { -1, std::string() };
+}
+
+void waybar::modules::sway::Window::getFocusedWindow()
+{
+  try {
+    auto res = ipc_.sendCmd(IPC_GET_TREE);
+    auto parsed = parser_.parse(res.payload);
+    auto [id, name] = getFocusedNode(parsed["nodes"]);
+    windowId_ = id;
+    window_ = name;
+    Glib::signal_idle().connect_once(sigc::mem_fun(*this, &Window::update));
+  } catch (const std::exception &e) {
+    std::cerr << e.what() << std::endl;
+  }
+}

src/modules/sway/workspaces.cpp

@@ -0,0 +1,234 @@
+#include "modules/sway/workspaces.hpp"
+
+waybar::modules::sway::Workspaces::Workspaces(Bar& bar,
+  const Json::Value& config)
+  : bar_(bar), config_(config), scrolling_(false)
+{
+  box_.set_name("workspaces");
+  ipc_.connect();
+  ipc_.subscribe("[ \"workspace\" ]");
+  // Launch worker
+  worker();
+}
+
+void waybar::modules::sway::Workspaces::worker()
+{
+  thread_ = [this] {
+    try {
+      // Wait for the name of the output
+      if (!config_["all-outputs"].asBool() && bar_.output_name.empty()) {
+        while (bar_.output_name.empty()) {
+          thread_.sleep_for(chrono::milliseconds(150));
+        }
+      } else if (!workspaces_.empty()) {
+        ipc_.handleEvent();
+      }
+      {
+        std::lock_guard<std::mutex> lock(mutex_);
+        auto res = ipc_.sendCmd(IPC_GET_WORKSPACES);
+        workspaces_ = parser_.parse(res.payload);
+      }
+      dp.emit();
+    } catch (const std::exception& e) {
+      std::cerr << e.what() << std::endl;
+    }
+  };
+}
+
+auto waybar::modules::sway::Workspaces::update() -> void
+{
+  bool needReorder = false;
+  std::lock_guard<std::mutex> lock(mutex_);
+  for (auto it = buttons_.begin(); it != buttons_.end();) {
+    auto ws = std::find_if(workspaces_.begin(), workspaces_.end(),
+      [it](auto node) -> bool { return node["name"].asString() == it->first; });
+    if (ws == workspaces_.end() ||
+            (!config_["all-outputs"].asBool() &&
+             (*ws)["output"].asString() != bar_.output_name)) {
+      it = buttons_.erase(it);
+      needReorder = true;
+    } else {
+      ++it;
+    }
+  }
+  for (auto const& node : workspaces_) {
+    if (!config_["all-outputs"].asBool()
+      && bar_.output_name != node["output"].asString()) {
+      continue;
+    }
+    auto it = buttons_.find(node["name"].asString());
+    if (it == buttons_.end()) {
+      addWorkspace(node);
+      needReorder = true;
+    } else {
+      auto &button = it->second;
+      if (node["focused"].asBool()) {
+        button.get_style_context()->add_class("focused");
+      } else {
+        button.get_style_context()->remove_class("focused");
+      }
+      if (node["visible"].asBool()) {
+        button.get_style_context()->add_class("visible");
+      } else {
+        button.get_style_context()->remove_class("visible");
+      }
+      if (node["urgent"].asBool()) {
+        button.get_style_context()->add_class("urgent");
+      } else {
+        button.get_style_context()->remove_class("urgent");
+      }
+      if (needReorder) {
+        box_.reorder_child(button, node["num"].asInt());
+      }
+      auto icon = getIcon(node["name"].asString(), node);
+      if (config_["format"].isString()) {
+        auto format = config_["format"].asString();
+        button.set_label(fmt::format(format, fmt::arg("icon", icon),
+          fmt::arg("name", node["name"].asString()),
+          fmt::arg("index", node["num"].asString())));
+      } else {
+        button.set_label(icon);
+      }
+      button.show();
+    }
+  }
+  if (scrolling_) {
+    scrolling_ = false;
+  }
+}
+
+void waybar::modules::sway::Workspaces::addWorkspace(Json::Value node)
+{
+  auto icon = getIcon(node["name"].asString(), node);
+  auto format = config_["format"].isString()
+    ? fmt::format(config_["format"].asString(), fmt::arg("icon", icon),
+      fmt::arg("name", node["name"].asString()),
+      fmt::arg("index", node["num"].asString()))
+    : icon;
+  auto pair = buttons_.emplace(node["name"].asString(), format);
+  auto &button = pair.first->second;
+  box_.pack_start(button, false, false, 0);
+  button.set_relief(Gtk::RELIEF_NONE);
+  button.signal_clicked().connect([this, pair] {
+    try {
+      std::lock_guard<std::mutex> lock(mutex_);
+      auto cmd = fmt::format("workspace \"{}\"", pair.first->first);
+      ipc_.sendCmd(IPC_COMMAND, cmd);
+    } catch (const std::exception& e) {
+      std::cerr << e.what() << std::endl;
+    }
+  });
+  button.add_events(Gdk::SCROLL_MASK | Gdk::SMOOTH_SCROLL_MASK);
+  if (!config_["disable-scroll"].asBool()) {
+    button.signal_scroll_event()
+      .connect(sigc::mem_fun(*this, &Workspaces::handleScroll));
+  }
+  box_.reorder_child(button, node["num"].asInt());
+  if (node["focused"].asBool()) {
+    button.get_style_context()->add_class("focused");
+  }
+  if (node["visible"].asBool()) {
+    button.get_style_context()->add_class("visible");
+  }
+  if (node["urgent"].asBool()) {
+    button.get_style_context()->add_class("urgent");
+  }
+  button.show();
+}
+
+std::string waybar::modules::sway::Workspaces::getIcon(std::string name,
+  Json::Value node)
+{
+  std::vector<std::string> keys = {
+    name, "urgent", "focused", "visible", "default"};
+  for (auto const& key : keys) {
+    if (key == "focused" || key == "visible" || key == "urgent") {
+      if (config_["format-icons"][key].isString() && node[key].asBool()) {
+        return config_["format-icons"][key].asString();
+      }
+    } else if (config_["format-icons"][key].isString()) {
+      return config_["format-icons"][key].asString();
+    }
+  }
+  return name;
+}
+
+bool waybar::modules::sway::Workspaces::handleScroll(GdkEventScroll *e)
+{
+  // Avoid concurrent scroll event
+  if (scrolling_) {
+    return false;
+  }
+  scrolling_ = true;
+  std::string name;
+  uint16_t idx = 0;
+  {
+    std::lock_guard<std::mutex> lock(mutex_);
+    for (; idx < workspaces_.size(); idx += 1) {
+      if (workspaces_[idx]["focused"].asBool()) {
+        name = workspaces_[idx]["name"].asString();
+        break;
+      }
+    }
+  }
+  if (name.empty()) {
+    scrolling_ = false;
+    return false;
+  }
+  if (e->direction == GDK_SCROLL_UP) {
+      name = getNextWorkspace();
+  }
+  if (e->direction == GDK_SCROLL_DOWN) {
+      name = getPrevWorkspace();
+  }
+  if (e->direction == GDK_SCROLL_SMOOTH) {
+    gdouble delta_x, delta_y;
+    gdk_event_get_scroll_deltas(reinterpret_cast<const GdkEvent *>(e),
+      &delta_x, &delta_y);
+    if (delta_y < 0) {
+      name = getNextWorkspace();
+    } else if (delta_y > 0) {
+      name = getPrevWorkspace();
+    }
+  }
+  if (!name.empty()) {
+    std::lock_guard<std::mutex> lock(mutex_);
+    if (name == workspaces_[idx]["name"].asString()) {
+      scrolling_ = false;
+      return false;
+    }
+    ipc_.sendCmd(IPC_COMMAND, fmt::format("workspace \"{}\"", name));
+    std::this_thread::sleep_for(std::chrono::milliseconds(150));
+  }
+  return true;
+}
+
+std::string waybar::modules::sway::Workspaces::getPrevWorkspace()
+{
+  for (uint16_t i = 0; i != workspaces_.size(); i += 1) {
+    if (workspaces_[i]["focused"].asBool()) {
+      if (i > 0) {
+        return workspaces_[i - 1]["name"].asString();
+      }
+      return workspaces_[workspaces_.size() - 1]["name"].asString();
+    }
+  }
+  return "";
+}
+
+std::string waybar::modules::sway::Workspaces::getNextWorkspace()
+{
+  for (uint16_t i = 0; i != workspaces_.size(); i += 1) {
+    if (workspaces_[i]["focused"].asBool()) {
+      if (i + 1U < workspaces_.size()) {
+        return workspaces_[i + 1]["name"].asString();
+      }
+      return workspaces_[0]["String"].asString();
+    }
+  }
+  return "";
+}
+
+waybar::modules::sway::Workspaces::operator Gtk::Widget &() {
+  return box_;
+}

src/modules/workspaces.cpp

@@ -1,121 +0,0 @@
-#include "modules/workspaces.hpp"
-#include "ipc/client.hpp"
-
-waybar::modules::Workspaces::Workspaces(Bar &bar)
-  : _bar(bar), _thread(nullptr), _box(Gtk::manage(new Gtk::Box))
-{
-  _box->get_style_context()->add_class("workspaces");
-  std::string socketPath = get_socketpath();
-  _ipcSocketfd = ipc_open_socket(socketPath);
-  _ipcEventSocketfd = ipc_open_socket(socketPath);
-  const char *subscribe = "[ \"workspace\", \"mode\" ]";
-  uint32_t len = strlen(subscribe);
-  ipc_single_command(_ipcEventSocketfd, IPC_SUBSCRIBE, subscribe, &len);
-  _idle_timer =
-		org_kde_kwin_idle_get_idle_timeout(_bar.client.idle_manager,
-      _bar.client.seat, 10000); // 10 seconds
-  static const struct org_kde_kwin_idle_timeout_listener idle_timer_listener = {
-    .idle = _handle_idle,
-    .resumed = _handle_resume,
-  };
-  org_kde_kwin_idle_timeout_add_listener(_idle_timer,
-    &idle_timer_listener, this);
-  _updateThread();
-}
-
-auto waybar::modules::Workspaces::update() -> void
-{
-  Json::Value workspaces = _getWorkspaces();
-  bool hided = false;
-  for (auto it = _buttons.begin(); it != _buttons.end(); ++it) {
-    auto ws = std::find_if(workspaces.begin(), workspaces.end(),
-      [it](auto node) -> bool { return node["num"].asInt() == it->first; });
-    if (ws == workspaces.end()) {
-      it->second.hide();
-      hided = true;
-    }
-  }
-  for (auto node : workspaces) {
-    auto it = _buttons.find(node["num"].asInt());
-    if (it == _buttons.end()) {
-      _addWorkspace(node);
-    } else {
-      auto styleContext = it->second.get_style_context();
-      bool isCurrent = node["focused"].asBool();
-      if (styleContext->has_class("current") && !isCurrent) {
-        styleContext->remove_class("current");
-      } else if (!styleContext->has_class("current") && isCurrent) {
-        styleContext->add_class("current");
-      }
-      if (hided) {
-        _box->reorder_child(it->second, node["num"].asInt() - 1);
-      }
-      it->second.show();
-    }
-  }
-}
-
-void waybar::modules::Workspaces::_updateThread()
-{
-  _thread = new waybar::util::SleeperThread([this] {
-    update();
-    _thread->sleep_for(waybar::chrono::milliseconds(150));
-  });
-}
-
-void waybar::modules::Workspaces::_handle_idle(void *data,
-  struct org_kde_kwin_idle_timeout *timer) {
-	auto o = reinterpret_cast<waybar::modules::Workspaces *>(data);
-  if (o->_thread) {
-	  delete o->_thread;
-    o->_thread = nullptr;
-  }
-}
-
-void waybar::modules::Workspaces::_handle_resume(void *data,
-  struct org_kde_kwin_idle_timeout *timer) {
-	auto o = reinterpret_cast<waybar::modules::Workspaces *>(data);
-  if (!o->_thread) {
-	  o->_updateThread();
-  }
-}
-
-void waybar::modules::Workspaces::_addWorkspace(Json::Value node)
-{
-  auto pair = _buttons.emplace(node["num"].asInt(), node["name"].asString());
-  auto &button = pair.first->second;
-  button.set_relief(Gtk::RELIEF_NONE);
-  button.signal_clicked().connect([this, pair] {
-    auto value = fmt::format("workspace \"{}\"", pair.first->first);
-    uint32_t size = value.size();
-    ipc_single_command(_ipcSocketfd, IPC_COMMAND, value.c_str(), &size);
-  });
-  _box->pack_start(button, false, false, 0);
-  _box->reorder_child(button, node["num"].asInt() - 1);
-  if (node["focused"].asBool()) {
-    button.get_style_context()->add_class("current");
-  }
-  button.show();
-}
-
-Json::Value waybar::modules::Workspaces::_getWorkspaces()
-{
-  uint32_t len = 0;
-  Json::Value root;
-  Json::CharReaderBuilder builder;
-  Json::CharReader* reader = builder.newCharReader();
-  std::string err;
-  std::string str = ipc_single_command(_ipcSocketfd, IPC_GET_WORKSPACES,
-    nullptr, &len);
-  bool res = reader->parse(str.c_str(), str.c_str() + str.size(), &root, &err);
-  delete reader;
-  if (!res) {
-    std::cerr << err << std::endl;
-    return nullptr;
-  }
-  return root;
-}
-
-waybar::modules::Workspaces::operator Gtk::Widget &() {
-  return *_box;
-}

subprojects/fmt.wrap

@@ -0,0 +1,10 @@
+[wrap-file]
+directory = fmt-5.2.1
+
+source_url = https://github.com/fmtlib/fmt/archive/5.2.1.tar.gz
+source_filename = fmt-5.2.1.tar.gz
+source_hash = 3c812a18e9f72a88631ab4732a97ce9ef5bcbefb3235e9fd465f059ba204359b
+
+patch_url = https://wrapdb.mesonbuild.com/v1/projects/fmt/5.2.1/1/get_zip
+patch_filename = fmt-5.2.1-1-wrap.zip
+patch_hash = 7add08bb4e168c0809e88c6aa64ed5c3494b74deb6be12a93e1e4dc5bb3a1fc1

subprojects/fmtlib.wrap

@@ -1,10 +0,0 @@
-[wrap-file]
-directory = fmt-4.1.0
-
-source_url = https://github.com/fmtlib/fmt/archive/4.1.0.tar.gz
-source_filename = fmt-4.1.0.tar.gz
-source_hash = 46628a2f068d0e33c716be0ed9dcae4370242df135aed663a180b9fd8e36733d
-
-patch_url = https://wrapdb.mesonbuild.com/v1/projects/fmt/4.1.0/1/get_zip
-patch_filename = fmt-4.1.0-1-wrap.zip
-patch_hash = 741931f01e558491724fc1c67bff996d1df79c0277626fc463de138052c9ecc0