[mythtv-commits] Ticket #13280: Setup - Edit Keys not reading modifiers correctly

MythTV noreply at mythtv.org
Mon May 21 09:32:45 UTC 2018


#13280: Setup - Edit Keys not reading modifiers correctly
---------------------------------------+---------------------------
     Reporter:  warmchocolatecake      |      Owner:  Peter Bennett
         Type:  Bug Report - General   |     Status:  new
     Priority:  minor                  |  Milestone:  needs_triage
    Component:  MythTV - Mythtv-setup  |    Version:  v29.1
     Severity:  low                    |   Keywords:  setup key
Ticket locked:  0                      |
---------------------------------------+---------------------------
 Whilst trying to add a symbol ("@") as a Jump Point, the actual key
 sequence that is stored is "Shift+@" - ie, I would need to press Shift
 twice

 (I have a UK keyboard, where @=Shift+' a US keyboard would have @=Shift+2
 )

 So, in the database, this is stored as:

 {{{
 MariaDB [mythconverg]> select * from jumppoints where keylist like
 'Shift%';
 +-------------+-------------+---------+----------+
 | destination | description | keylist | hostname |
 +-------------+-------------+---------+----------+
 | Play music  |             | Shift+@ | tower    |
 +-------------+-------------+---------+----------+
 }}}


 Manually changing the DB, permits the correct key to be used:

 {{{
 MariaDB [mythconverg]> update jumppoints set keylist='@' where destination
 ='Play music' and hostname ='tower';
 Query OK, 1 row affected (0.00 sec)
 Rows matched: 1  Changed: 1  Warnings: 0

 MariaDB [mythconverg]> select * from jumppoints where keylist like
 'Shift%';
 Empty set (0.00 sec)
 }}}


 This appears to be an issue in keygrabber.cpp at
 KeyGrabPopupBox::keyPressEvent() where key_name is set as a simple append
 of the modifier and the final key.

 So, for my UK keyboard, this means I cannot use !"£$%, etc...

 I'm not a QT developer, but I'd say the easiest solution here would be to
 only store the SHIFT modifier if the key pressed was an alphabetical one -
 this would then still permit ALT+2 (for example)

 Currently, there's no way to use many of the symbol keys (without
 modifying the DB directly), so this limits the user selectable range of
 inputs.

-- 
Ticket URL: <https://code.mythtv.org/trac/ticket/13280>
MythTV <http://www.mythtv.org>
MythTV Media Center


More information about the mythtv-commits mailing list