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

Android: Somewhat align scancode conversions with Firefox #4023

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

madsmtm
Copy link
Member

@madsmtm madsmtm commented Dec 1, 2024

Similar to #4019, but for Android, and only for the Firefox sources (haven't gone through Chromium's sources for this kind of stuff yet).

Firefox' sources seems to use some other form of scan code that I couldn't figure out? So instead we try to map as many of the codes available on Keycode that they do as possible.

  • Tested on all platforms changed
  • Added an entry to the changelog module if knowledge of this change could be valuable to users
  • Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • Created or updated an example program if it would help users understand this functionality

@madsmtm madsmtm added S - enhancement Wouldn't this be the coolest? DS - android labels Dec 1, 2024
@madsmtm madsmtm requested a review from MarijnS95 as a code owner December 1, 2024 22:02
Firefox' source at:
https://searchfox.org/mozilla-central/rev/c597e9c789ad36af84a0370d395be066b7dc94f4/widget/NativeKeyToDOMCodeName.h

Seems to use some other form of scan code? So instead we try to map as
many of the codes available on `Keycode` that they do as possible.
@madsmtm madsmtm force-pushed the madsmtm/android-scancode-firefox branch from b538cef to 27840eb Compare December 2, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DS - android S - enhancement Wouldn't this be the coolest?
Development

Successfully merging this pull request may close these issues.

1 participant