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

[keymap] show OSD when pressing SELECT on remotes in Visualisation #6459

Merged
merged 1 commit into from Feb 18, 2015

Conversation

da-anda
Copy link
Member

@da-anda da-anda commented Feb 16, 2015

Currently the behavior when pressing SELECT/OK on remotes in FullscreenVideo and Visualisation window differs. While on FullscreenVideo you get the OSD menu you get some visualisation preset list for music. IMO this should be consistent and always trigger the OSD menu (do we really need a shortcut to visualisation presets on limited remotes?). This is especially useful when using CEC (which is using the remote keymap) with a limited set of buttons forwarded (like on LG TVs).

@mkortstiege
Copy link
Member

+1. Keyboard is already opening the OSD as well.

@da-anda
Copy link
Member Author

da-anda commented Feb 16, 2015

@MartijnKaijser we might want to change it for 14.2 as well?

@da-anda
Copy link
Member Author

da-anda commented Feb 17, 2015

no other comments or is nobody on git using music section ;)

@jjd-uk
Copy link
Member

jjd-uk commented Feb 17, 2015

+1 from me as we should aim for more consistency between the music & videos sections.

@MartijnKaijser
Copy link
Member

sure

@ronie
Copy link
Member

ronie commented Feb 17, 2015

music, always +1

mkortstiege added a commit that referenced this pull request Feb 18, 2015
[keymap] show OSD when pressing SELECT on remotes in Visualisation
@mkortstiege mkortstiege merged commit 894a28f into xbmc:master Feb 18, 2015
@MartijnKaijser MartijnKaijser modified the milestone: I******* 15.0-alpha1 Feb 18, 2015
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.

None yet

5 participants