Skip to content

feat(bongocat): add executable_path setting#6

Open
NousadS wants to merge 1 commit into
noctalia-dev:mainfrom
NousadS:feat/bongocat-executable-path
Open

feat(bongocat): add executable_path setting#6
NousadS wants to merge 1 commit into
noctalia-dev:mainfrom
NousadS:feat/bongocat-executable-path

Conversation

@NousadS

@NousadS NousadS commented Jun 26, 2026

Copy link
Copy Markdown

Summary

Added executable_path setting, replacing the previous hardcoded executable path.

Motivation

See #4.

Type of Change

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

Related Issue

Closes #4.

Testing

Tested the executable_path setting with:

  • Valid paths: evtest, /usr/bin/evtest, /run/current-system/sw/bin/evtest
  • Invalid paths: /path, /path/, /usr/, /

(Note: Directory paths like / and /usr/ should give an error but don't do so, see noctalia#3167)

Manual Coverage

  • Tested on Niri
  • Tested on Hyprland
  • Tested on Sway
  • Tested on another compositor:
  • Tested with different bar positions and density settings
  • Tested at different interface scaling values
  • Tested with multiple monitors

Screenshots / Videos

Checklist

  • This PR is ready for review, or it is marked as Draft.
  • I self-reviewed the changes.
  • I added or updated translations/en.json, or this PR adds no new user-facing strings.
  • I did not edit non-English translation files unless this PR is explicitly for translation tooling, an import/export sync, or a maintainer-requested locale change.

Additional Notes

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.

[FEATURE] Add executable option for evtest in Bongo Cat plugin

1 participant