[mythtv-commits] Ticket #10017: 'Edit Keys' should not allow you to set keybindings that myth will ignore

[MythTV]

#10017: 'Edit Keys' should not allow you to set keybindings that myth will ignore
----------------------------------+----------------------------
 Reporter:  dantheperson@…        |          Owner:  mdean
     Type:  Bug Report - General  |         Status:  accepted
 Priority:  minor                 |      Milestone:  unknown
Component:  MythTV - General      |        Version:  0.24-fixes
 Severity:  medium                |     Resolution:
 Keywords:  keybinding Shift      |  Ticket locked:  0
----------------------------------+----------------------------

Comment (by paulh):

 Replying to [comment:4 Ken Truesdale <kentruesdale@…>]:
 > I thought it was just me until I recently did some deeper testing.  I
 discovered that when I entered a keybinding such as a "+" key, the key
 that showed on the screen was "Shift++" meaning it knew I was holding down
 the shift key and that I generated a "+" and it put them together.  To me
 that was always redundant terminology - on my small keyboard, I can't get
 a "+" without the shift key, so yeah, shift key, whatever.  And I was
 always frustrated that keys like this didn't work.  But I now realize the
 redundancy of the "shift" is the crux of the problem.  Because I modified
 the database directly to change the keybinding from "Shift++" to simply
 "+" and it works fine.  Therefore, I think the issue is in the front end,
 when it captures the keybinding, it seems to be working a little too hard
 to figure out what the user did - it should simply accept the key that it
 got.


 Myth has always ignored the shift key so that 'd' and 'D' would be the
 same key binding. IIRC the thinking was if you allowed key and shift+key
 to allow different bindings users would get annoyed if they didn't realise
 the caps lock was on and accidentally activated the wrong binding.

 It's true Myth should ignore or warn about invalid key bindings like these
 but on the grand scheme of things it's a pretty trivial problem which is
 probably why this ticket hasn't been touched in 3 years :(

 As always developer resources are very scares at the moment so any help
 fixing the problem would be much appreciated :)

--
Ticket URL: <https://code.mythtv.org/trac/ticket/10017#comment:5>
MythTV <http://www.mythtv.org>
MythTV Media Center