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

Enable haptic feedback for controllers #1068

Merged
merged 3 commits into from Oct 30, 2023
Merged

Enable haptic feedback for controllers #1068

merged 3 commits into from Oct 30, 2023

Conversation

HollowMan6
Copy link
Collaborator

Haptic feedback will be triggered when:

  • Android view global focus changes
  • press on the keyboard
  • MotionEvent.ACTION_HOVER_ENTER

Users can disable the haptic feedback in controller settings.

Resolves #1067
image

Copy link
Member

@svillar svillar left a comment

Choose a reason for hiding this comment

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

Looking great. This is so intrusive that we should start with the feature disabled by default.

Apart from that I just have a couple of minor comments.

app/src/main/cpp/BrowserWorld.cpp Outdated Show resolved Hide resolved
@HollowMan6 HollowMan6 force-pushed the haptic-feedback branch 2 times, most recently from 7818a04 to 5882e2e Compare October 27, 2023 17:56
@HollowMan6
Copy link
Collaborator Author

Looking great. This is so intrusive that we should start with the feature disabled by default.

Now we also have this disabled by default

app/src/main/cpp/BrowserWorld.cpp Outdated Show resolved Hide resolved
app/src/main/cpp/BrowserWorld.cpp Outdated Show resolved Hide resolved
Users can also disable the haptic feedback in
controller settings.

Resolves #1067

Signed-off-by: Songlin Jiang <sjiang@igalia.com>
Signed-off-by: Songlin Jiang <sjiang@igalia.com>
Signed-off-by: Songlin Jiang <sjiang@igalia.com>
Copy link
Member

@svillar svillar left a comment

Choose a reason for hiding this comment

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

Neat!

@svillar svillar dismissed javifernandez’s stale review October 30, 2023 10:09

Javi's concerns will be addressed in a follow up as agreed in the review process

@svillar svillar merged commit 92f3016 into main Oct 30, 2023
18 checks passed
@svillar svillar deleted the haptic-feedback branch October 30, 2023 10:09
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.

Enable haptic feedback when controller pointer swipes across widgets/web page
3 participants