Merge pull request #1522 from FlexW/bugfix/app-icon

include/modules/sway/window.hpp

@@ -25,6 +25,7 @@ class Window : public AIconLabel, public sigc::trackable {
                                                                         std::string& output);
   void getTree();
   std::string rewriteTitle(const std::string& title);
+  void updateAppIconName();
   void updateAppIcon();
 
   const Bar& bar_;
@@ -33,6 +34,8 @@ class Window : public AIconLabel, public sigc::trackable {
   std::string app_id_;
   std::string old_app_id_;
   std::size_t app_nb_;
+  bool update_app_icon_{true};
+  std::string app_icon_name_;
   util::JsonParser parser_;
   std::mutex mutex_;
   Ipc ipc_;

src/modules/sway/window.cpp

@@ -38,7 +38,7 @@ void Window::onCmd(const struct Ipc::ipc_response& res) {
     auto payload = parser_.parse(res.payload);
     auto output = payload["output"].isString() ? payload["output"].asString() : "";
     std::tie(app_nb_, windowId_, window_, app_id_) = getFocusedNode(payload["nodes"], output);
-    updateAppIcon();
+    updateAppIconName();
     dp.emit();
   } catch (const std::exception& e) {
     spdlog::error("Window: {}", e.what());
@@ -79,17 +79,30 @@ std::optional<Glib::ustring> getIconName(const std::string& app_id) {
   return {};
 }
 
-void Window::updateAppIcon() {
+void Window::updateAppIconName() {
   if (!iconEnabled()) {
     return;
   }
+
   const auto icon_name = getIconName(app_id_);
   if (icon_name.has_value()) {
-    image_.set_from_icon_name(icon_name.value(), Gtk::ICON_SIZE_LARGE_TOOLBAR);
-    image_.set_visible(true);
-    return;
+    app_icon_name_ = icon_name.value();
+  } else {
+    app_icon_name_ = "";
+  }
+  update_app_icon_ = true;
+}
+
+void Window::updateAppIcon() {
+  if (update_app_icon_) {
+    update_app_icon_ = false;
+    if (app_icon_name_.empty()) {
+      image_.set_visible(false);
+    } else {
+      image_.set_from_icon_name(app_icon_name_, Gtk::ICON_SIZE_LARGE_TOOLBAR);
+      image_.set_visible(true);
+    }
   }
-  image_.set_visible(false);
 }
 
 auto Window::update() -> void {
@@ -119,6 +132,9 @@ auto Window::update() -> void {
   if (tooltipEnabled()) {
     label_.set_tooltip_text(window_);
   }
+
+  updateAppIcon();
+
   // Call parent update
   AIconLabel::update();
 }