Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keyboard state module #1024

Merged
merged 10 commits into from
Jul 23, 2021
Merged

Keyboard state module #1024

merged 10 commits into from
Jul 23, 2021

Conversation

GrantMoyer
Copy link
Contributor

Adds a new module which shows the state of num lock, caps lock, and scroll lock.

screencap

@GrantMoyer GrantMoyer changed the title Keyboard state Keyboard state module Feb 8, 2021
@GrantMoyer GrantMoyer marked this pull request as draft February 8, 2021 01:15
@GrantMoyer GrantMoyer marked this pull request as ready for review February 8, 2021 02:07
@Alexays
Copy link
Owner

Alexays commented Apr 15, 2021

Hi, Can you add this module into man?

@GrantMoyer
Copy link
Contributor Author

It looks like I need to fix the BSD build.

@GrantMoyer GrantMoyer force-pushed the keyboard_state branch 3 times, most recently from 3c40504 to 1f0a859 Compare July 21, 2021 03:02
@GrantMoyer
Copy link
Contributor Author

The build should pass on all test systems now (it passes in my staging PR). There are a few outdated github actions workflows queued up for this PR, and I'm not sure GitHub will cancel them automatically, so you may want to cancel those manually.

src/modules/keyboard_state.cpp Outdated Show resolved Hide resolved
src/factory.cpp Outdated Show resolved Hide resolved
resources/style.css Outdated Show resolved Hide resolved
resources/config Outdated Show resolved Hide resolved
man/waybar-keyboard-state.5.scd Outdated Show resolved Hide resolved
@Alexays Alexays merged commit 1b4ddbc into Alexays:master Jul 23, 2021
@Alexays
Copy link
Owner

Alexays commented Jul 23, 2021

Thanks!

@GrantMoyer GrantMoyer deleted the keyboard_state branch July 26, 2021 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants