diff --git a/src/modules/battery.cpp b/src/modules/battery.cpp index 12396d46..efab6357 100644 --- a/src/modules/battery.cpp +++ b/src/modules/battery.cpp @@ -37,7 +37,7 @@ auto waybar::modules::Battery::update() -> void charging = true; } } - if (charging == true) { + if (charging) { _label.get_style_context()->add_class("charging"); } else { _label.get_style_context()->remove_class("charging"); diff --git a/src/modules/custom.cpp b/src/modules/custom.cpp index 693e2200..8f5ab562 100644 --- a/src/modules/custom.cpp +++ b/src/modules/custom.cpp @@ -4,7 +4,6 @@ waybar::modules::Custom::Custom(std::string name, Json::Value config) : _name(name), _config(config) { - _label.get_style_context()->add_class("custom-" + name); if (!_config["exec"]) { std::cerr << name + " has no exec path." << std::endl; return; @@ -36,9 +35,11 @@ auto waybar::modules::Custom::update() -> void } // Hide label if output is empty - if (output.empty()) + if (output.empty()) { + _label.get_style_context()->remove_class("custom-" + _name); _label.hide(); - else { + } else { + _label.get_style_context()->add_class("custom-" + _name); auto format = _config["format"] ? _config["format"].asString() : "{}"; _label.set_text(fmt::format(format, output)); _label.show(); diff --git a/src/modules/pulseaudio.cpp b/src/modules/pulseaudio.cpp index 1b196cc5..7e4b99b8 100644 --- a/src/modules/pulseaudio.cpp +++ b/src/modules/pulseaudio.cpp @@ -103,9 +103,8 @@ auto waybar::modules::Pulseaudio::update() -> void 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()->add_class("muted"); + } else _label.get_style_context()->remove_class("muted"); _label.set_label(fmt::format(format, _volume)); _label.set_tooltip_text(_desc); diff --git a/src/modules/workspaces.cpp b/src/modules/workspaces.cpp index 33e76604..80c29bea 100644 --- a/src/modules/workspaces.cpp +++ b/src/modules/workspaces.cpp @@ -42,9 +42,9 @@ auto waybar::modules::Workspaces::update() -> void } else { auto styleContext = it->second.get_style_context(); bool isCurrent = node["focused"].asBool(); - if (styleContext->has_class("current") && !isCurrent) { + if (!isCurrent) { styleContext->remove_class("current"); - } else if (!styleContext->has_class("current") && isCurrent) { + } else if (isCurrent) { styleContext->add_class("current"); } if (hided) {