(2014-11-27, 04:53)Ned Scott Wrote: With all that considered, I concluded that it wasn't an issue to advise people to copy the whole keymap file in order to learn and easily make changes.
I've tried to think of some kind of wording that would prevent users from duplicating maps wholesale (or any other setting, for that matter) and I've failed to think of anything that doesn't make the process even more complicated. But even with an appropriate warning, there will always be users that don't read or comprehend the warning and copy entire button maps/settings regardless, rather than cherry-pick.
So I guess we live with the current situation, with some users unintentionally enforcing legacy default behaviour with their custom maps (which in reality might be changing one mapping out of 20+), but this shouldn't prevent default behaviour being changed/improved in future. Anyone applying a custom map is responsible for the mappings from that point onwards - if the default behaviours change but their custom mapping doesn't, IMHO it's their problem as that's surely the accepted risk of making such changes.
I'm also no longer sure the problem reported by Wotok is due to the PlayPause change.
(2014-11-27, 05:18)Ned Scott Wrote: I thought the music behavior was unchanged from that (at least vs v13?), but you got me about the trick-play resume normal speed part. I didn't think of that. Well, okay, then I'm fine with all of them
It's such a long time since I used v13 that I can't be 100% sure but I believe (in OpenELEC, at least) that the VRC-1100 would restart music in a playlist as the remote only outputs the KEY_PLAY code, and with the <play>Play</play> mapping, non-full screen windows (eg. playlist) would interpret the command literally and restart the currently playing item rather than pause it. It was never a problem with full screen playback, where <play>Play</play> would toggle (behaviour removed by PR5245). Certainly this was how it behaved in early v14 until the changes from da-anda.
With PR5245 the Play function only resumes playback - it no longer toggles between play and pause. PR5307 fixes this by changing the default <play> button mapping so that it does toggle.
If the user wants the old behaviour, which is now broken by a custom mapping, they need to change their map so that the <play> button is mapped to the PlayPause function (or better yet, not mapped at all).
I think the new default mapping is saner, but it was always going to break some setups that were enforcing their own behaviours.