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] Update AndroidKey.cpp #12000

Merged
merged 1 commit into from
May 10, 2017
Merged

[Android] Update AndroidKey.cpp #12000

merged 1 commit into from
May 10, 2017

Conversation

mihvel
Copy link
Contributor

@mihvel mihvel commented Apr 23, 2017

Extending support for the physical QWERTY keyboard in Android Kodi with processing of AKEYCODE_MOVE_HOME and AKEYCODE_MOVE_END key codes. Currently, these keyboard events are just ignored. In large Kodi list views jumping on FirstPage and LastPage is very convenient. I do not see any reason why it should not be supported in the Android Kodi, as it is on other OS platforms.

Description

Enabling support for HOME/END physical keyboard keys for moving in Kodi lists on Android.

Motivation and Context

There are more and more Android based media players on the market, not just tablets and phones. So, adding full support for the physical QWERTY keyboard in Android Kodi would be nice. Currently, I'm missing support and handling of the Home and End keyboard keys in the Android Kodi. Even my Android HTPC box handles Home/End keys properly, the Android Kodi does not process AKEYCODE_MOVE_HOME and AKEYCODE_MOVE_END key codes, so the keyboard events are just ignored. In large Kodi list views jumping on FirstPage and LastPage is very convenient and I used to use it on my previous Windows based HTPC. I do not see any reason why you should not support the keyboard in Kodi Android, as it is on other OS platforms.

http://forum.kodi.tv/showthread.php?tid=258808
http://forum.kodi.tv/showthread.php?tid=259037

How Has This Been Tested?

I did my own build with these changes included, using Kodi Jarvis as starting point. I'm using it daily for almost a year, without any issues.

Screenshots (if appropriate):

Types of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the Code guidelines of this project
  • My change requires a change to the documentation, either Doxygen or wiki
  • I have updated the documentation accordingly
  • I have read the CONTRIBUTING document
  • I have added tests to cover my change
  • All new and existing tests passed

Extending support for the physical QWERTY keyboard in Android Kodi with processing of AKEYCODE_MOVE_HOME and AKEYCODE_MOVE_END key codes. Currently, these keyboard events are just ignored. In large Kodi list views jumping on FirstPage and LastPage is very convenient. I do not see any reason why it should not be supported in the Android Kodi , as it is on other OS platforms.
@Rechi Rechi added Platform: Android Type: Improvement non-breaking change which improves existing functionality Component: Input v18 Leia labels Apr 23, 2017
@mihvel mihvel changed the title Update AndroidKey.cpp [Android] Update AndroidKey.cpp Apr 23, 2017
@razzeee razzeee requested a review from koying May 6, 2017 14:02
@razzeee
Copy link
Member

razzeee commented May 6, 2017

Thank you for your contribution and please excuse the late reply.
I'll ping @koying which should be able to process this PR!

@koying
Copy link
Contributor

koying commented May 6, 2017

+1
Just needs jenkins back to confirm it builds

@MartijnKaijser
Copy link
Member

jenkins build this please

@razzeee razzeee merged commit d49c04a into xbmc:master May 10, 2017
@Rechi Rechi added this to the L 18.0-alpha1 milestone May 10, 2017
@mihvel mihvel deleted the patch-1 branch May 10, 2017 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Input Platform: Android Type: Improvement non-breaking change which improves existing functionality v18 Leia
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants