Zander671/Waybar
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #5163 from Alexays/fix-2215

Browse Source 
fix(keyboard-state): read lock LEDs from all devices, not just the first
This commit is contained in:
Alexis Rouillard
2026-07-04 12:53:02 +02:00
committed by GitHub
parent 33223d5534 b2d15d1e3a
commit 0895465430
1 changed files with 32 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/modules/keyboard_state.cpp
+16 -7
View File
@@ -297,8 +297,7 @@ auto waybar::modules::KeyboardState::update() -> void {
sleep(0); // Wait for keyboard status change sleep(0); // Wait for keyboard status change
int numl = 0, capsl = 0, scrolll = 0; int numl = 0, capsl = 0, scrolll = 0;
try { std::vector<std::string> dev_paths;
std::string dev_path;
{ {
std::lock_guard<std::mutex> lock(devices_mutex_); std::lock_guard<std::mutex> lock(devices_mutex_);
if (libinput_devices_.empty()) { if (libinput_devices_.empty()) {
@@ -306,16 +305,25 @@ auto waybar::modules::KeyboardState::update() -> void {
} }
if (config_["device-path"].isString() && if (config_["device-path"].isString() &&
libinput_devices_.find(config_["device-path"].asString()) != libinput_devices_.end()) { libinput_devices_.find(config_["device-path"].asString()) != libinput_devices_.end()) {
dev_path = config_["device-path"].asString(); // An explicit device was configured: read lock state from just that device.
dev_paths.push_back(config_["device-path"].asString());
} else { } else {
dev_path = libinput_devices_.begin()->first; // No explicit device: a multi-node keyboard may expose several event
// devices where only one of them actually toggles the lock LEDs. OR the
// LED values across all devices so a lock is reported on if any device
// reports it on.
for (const auto& [dev_path, _] : libinput_devices_) {
dev_paths.push_back(dev_path);
} }
} }
}
for (const auto& dev_path : dev_paths) {
try {
int fd = openFile(dev_path, O_NONBLOCK | O_CLOEXEC | O_RDONLY); int fd = openFile(dev_path, O_NONBLOCK | O_CLOEXEC | O_RDONLY);
auto dev = openDevice(fd); auto dev = openDevice(fd);
numl = libevdev_get_event_value(dev, EV_LED, LED_NUML); numl |= libevdev_get_event_value(dev, EV_LED, LED_NUML);
capsl = libevdev_get_event_value(dev, EV_LED, LED_CAPSL); capsl |= libevdev_get_event_value(dev, EV_LED, LED_CAPSL);
scrolll = libevdev_get_event_value(dev, EV_LED, LED_SCROLLL); scrolll |= libevdev_get_event_value(dev, EV_LED, LED_SCROLLL);
libevdev_free(dev); libevdev_free(dev);
closeFile(fd); closeFile(fd);
} catch (const errno_error& e) { } catch (const errno_error& e) {
@@ -324,6 +332,7 @@ auto waybar::modules::KeyboardState::update() -> void {
spdlog::warn(e.what()); spdlog::warn(e.what());
} }
} }
}
struct { struct {
bool state; bool state;