Skip to content

feat(media-volume): add media and volume keybinds with configurable volume steps - #23

Open
GhostEagle68 wants to merge 3 commits into
noctalia-dev:mainfrom
GhostEagle68:feat/media-volume-keybinds
Open

feat(media-volume): add media and volume keybinds with configurable volume steps#23
GhostEagle68 wants to merge 3 commits into
noctalia-dev:mainfrom
GhostEagle68:feat/media-volume-keybinds

Conversation

@GhostEagle68

@GhostEagle68 GhostEagle68 commented Aug 24, 2026

Copy link
Copy Markdown

Summary

Add commented-out media and volume keybinds to examples/config.toml so users can see how to wire up volume control with wpctl and media playback with playerctl. The volume step is a single number users change to their preference.

Update docs/user/keybinds.md to use wpctl directly instead of noctalia msg for the volume example, with a comment clarifying the step size pattern. Add a note that XF86 keys can be remapped to any chord.

Motivation

The default config had no media or volume keybinds, and the docs example depended on Noctalia for volume. Users without Noctalia or without dedicated media keys had no guidance on how to set up audio controls.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Build / packaging
  • Documentation

Testing

  • umbriel validate -c examples/config.toml — passes
  • just format — no C++ changes
  • Manual: keybinds verified working with wpctl and playerctl in a running session

Manual Coverage

  • Tested in a nested Umbriel session
  • Tested in a native Umbriel session
  • Tested with multiple monitors
  • Tested with a scaled output
  • Tested with native Wayland applications
  • Tested with X11 applications through xwayland-satellite
  • Tested with the scrolling layout
  • Tested with the dwindle layout

Checklist

  • This PR is ready for review, or it is marked as Draft.
  • I read and followed the relevant guidance in CONTRIBUTING.md.
  • I initialized and updated the SceneFX submodule where required.
  • I ran just format, or this PR has no C++ changes.
  • I ran the relevant build, test, lint, or verification commands, or explained why they were not run.
  • I functionally verified compositor behavior where automated checks are insufficient.
  • I self-reviewed the changes.
  • I checked for new warnings or errors.
  • I updated docs/ and examples/config.toml, or this PR does not change user-facing configuration or behavior.
  • I used canonical names for config keys, IPC actions, paths, and identifiers.

GhostEagle68 and others added 3 commits August 24, 2026 12:06
… step

Add commented-out media and volume keybinds to examples/config.toml so
users can see how to wire up volume control with wpctl and media
playback with playerctl. The volume step (5%) is a single number users
change to their preference.

Update docs/user/keybinds.md to use wpctl directly instead of noctalia
for the volume example, with a comment clarifying the step size pattern.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
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.

1 participant