Skip to content

Use packaged Dell XPS touchpad haptics#5534

Open
jacob-vincent-mink wants to merge 4 commits intobasecamp:devfrom
jacob-vincent-mink:dell-haptics
Open

Use packaged Dell XPS touchpad haptics#5534
jacob-vincent-mink wants to merge 4 commits intobasecamp:devfrom
jacob-vincent-mink:dell-haptics

Conversation

@jacob-vincent-mink
Copy link
Copy Markdown

@jacob-vincent-mink jacob-vincent-mink commented May 1, 2026

Summary

  • move Dell XPS Synaptics haptics out of omarchy and into the new dell-xps-touchpad-haptics package in OPR
  • install that package automatically on supported Dell XPS systems
  • replace the old setup flow with Toggle > Hardware > Touchpad Haptics
  • migrate existing installs off the legacy in-repo service and onto the packaged service

Details

  • keep hardware detection in omarchy with omarchy-hw-dell-xps-haptic-touchpad
  • add package install wiring via install/packaging/dell-xps-touchpad-haptics.sh
  • remove the old in-tree daemon/update/setup commands and legacy service bootstrap
  • add omarchy-toggle-touchpad-haptics, which cycles these preset levels:
    • low
    • mid
    • high
  • use normal Omarchy notifications when the level changes
  • expose the control at:
    • Toggle > Hardware > Touchpad Haptics

Migration

  • keep a single migration that:
    • disables and removes the old dell-xps-haptic-touchpad.service
    • removes the old env/rule/override wiring
    • installs dell-xps-touchpad-haptics
  • service enable/restart is owned by the package install hook in omarchy-pkgs

Dependency

  • depends on omacom-io/omarchy-pkgs#86

Testing

  • bash -n bin/omarchy-hw-dell-xps-haptic-touchpad bin/omarchy-toggle-touchpad-haptics bin/omarchy-menu install/packaging/dell-xps-touchpad-haptics.sh migrations/1777653181.sh
  • git diff --check

Hardware test

  • install the package from omarchy-pkgs
  • run:
    • PATH="$HOME/src/github.com/jacob-vincent-mink/omarchy/bin:$PATH" omarchy-toggle-touchpad-haptics
  • verify the menu entry appears on supported Dell hardware
  • verify the toggle cycles through the three presets and the packaged daemon logs the changes

Copilot AI review requested due to automatic review settings May 1, 2026 17:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR makes the Dell XPS Synaptics haptic touchpad daemon configurable via a user config file, adds hardware detection + setup/update commands for intensity, and exposes the control via the Omarchy Setup menu (with a migration for existing installs).

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

Changes:

  • Add Dell XPS haptic touchpad detection plus setup/update commands to manage intensity.
  • Update the haptic daemon + systemd service wiring to read the desktop user’s config path via an EnvironmentFile.
  • Extend Setup > Input to include a Dell haptics entry (and migrate existing installs to the new service wiring).

Reviewed changes

Copilot reviewed 2 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
migrations/1777653181.sh Migrates existing Dell XPS installs to refresh the service and apply the configurable path.
install/config/hardware/dell/fix-xps-haptic-touchpad.sh Updates the systemd unit to optionally load /etc/omarchy-dell-haptic-touchpad.env and uses the new hardware detector.
bin/omarchy-update-dell-haptic New command to write intensity config + env file and restart/reset trackpad/service.
bin/omarchy-setup-dell-haptic-touchpad New interactive/setup wrapper (gum input) around the update command.
bin/omarchy-menu Adds an Input submenu and exposes “Touchpad Haptics” on supported Dell systems.
bin/omarchy-hw-dell-xps-haptic-touchpad New hardware detection helper for Dell XPS Synaptics haptic touchpads.
bin/omarchy-haptic-touchpad Reads intensity from user config and supports runtime refresh via mtime/SIGHUP.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dhh
Copy link
Copy Markdown
Member

dhh commented May 2, 2026

This is great! But I'd like to do the following:

  • Extract omarchy-haptic-touchpad into a new package called dell-xps-touchpad-haptics that we put in OPR.
  • This new package should expose dell-xps-touchpad-haptics cli which offer something like get, set which we can then call from Omarchy
  • We install this new package if the synaptic haptic engine is detected on new install and via migration
  • We add Toggle > Hardware > Touchpad Haptics which will cycle through 3 pretermined levels: low, mid, high with a notification for feedback.

@jacob-vincent-mink
Copy link
Copy Markdown
Author

Thanks for the feedback @dhh! 30/60/90 works for low/medium/high? Or - any other opinion on which levels made the touchpad feel better?

I’ll get on this after I finish travels in a day.

@dhh
Copy link
Copy Markdown
Member

dhh commented May 2, 2026

I haven't actually tried, but let's just try with that? I'll tweak them after trying it on the real hardware.

@jacob-vincent-mink
Copy link
Copy Markdown
Author

@dhh I've updated omacom-io/omarchy-pkgs#86 towards your feedback and have updated this PR to use it.

After messing around with the numbers on my system, I found 10/50/100 to be more "obvious" when toggling, but you might find different.

Copilot AI review requested due to automatic review settings May 4, 2026 00:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 6 out of 10 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread migrations/1777851157.sh Outdated
Comment thread migrations/1777851157.sh Outdated
Comment thread install/packaging/dell-xps-touchpad-haptics.sh
Comment thread migrations/1777653181.sh
Comment thread migrations/1777653181.sh Outdated
@jacob-vincent-mink jacob-vincent-mink changed the title Configure Dell XPS haptics Use packaged Dell XPS touchpad haptics May 4, 2026
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.

3 participants