RasPlex / OpenPHT

OpenPHT is a community driven fork of Plex Home Theater
Other
599 stars 109 forks source link

Feature Request - Go all the way back to the main menu when pressing "root menu" button on remote #208

Open ndbroadbent opened 7 years ago

ndbroadbent commented 7 years ago

I have an AV receiver remote that is working really well via HDMI-CEC. I'm 99% happy with OpenPHT, but there's one little thing that I would like to fix. Sometimes I navigate deep into something like TV Shows => Show => Season => Episode => Play. When the episode finishes, I usually just want to go all the way back to the main menu, so I have to press return 4 times. However, my remote has "Top Menu" and "Menu" buttons, so it would be really nice if I could use one of those to return to the home screen.

I ran cec-client to get information about these buttons:

I think it makes a lot of sense to map the "root menu" button to go to the home screen. I had a look at the Kodi keymaps, but I'm not sure what to do. I have a file at ~/.plexht/userdata/keymaps/remote.xml, but I'm not sure how to configure that.

OpenPHT: 1.7.1.137, Ubuntu 16.04.1 LTS PMS: 1.4.2.3400

ndbroadbent commented 7 years ago

Aha, I see the problem. OpenPHT doesn't differentiate between the menu and rootmenu buttons. However, I do see it in the xbmc remote keymap

I don't really want to override the menu button, since that's really useful for deleting and marking things as watched. So for now I'm just going to use one of the color buttons, by adding this to my ~/.plexht/userdata/keymaps/remote.xml:

<keymap>
  <global>
    <remote>
      <red>XBMC.ActivateWindow(Home)</red>
    </remote>
  </global>
</keymap>

I found this commit in XBMC where the rootmenu and contentsmenu buttons were added: https://github.com/xbmc/xbmc/commit/07fb4f0a82c1688e8229d78635213dd4f2efdb5a

And I see OpenPHT doesn't have that commit, since it only has the one menu button: https://github.com/RasPlex/OpenPHT/blob/openpht-1.7/xbmc/input/ButtonTranslator.cpp#L1217

Would it be possible to merge that commit with the extra CEC buttons?