Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#3363 closed patch (fixed)

TranslateKeyPress: prefer local action over jump point's

Reported by: anonymous Owned by: stuartm
Priority: minor Milestone: unknown
Component: mythtv Version: unknown
Severity: medium Keywords: key binding
Cc: Ticket locked: no

Description

The attached patch changes MythMainWindow::TranslateKeyPress? to prefer the local context's action for a keypress over the global "jump points" action.

Attachments (1)

KeyLocalContext.patch (1.0 KB) - added by jppoet@… 13 years ago.
Keybinding: Local context priority over jump points

Download all attachments as: .zip

Change History (6)

Changed 13 years ago by jppoet@…

Attachment: KeyLocalContext.patch added

Keybinding: Local context priority over jump points

comment:1 Changed 13 years ago by stuartm

Owner: changed from Isaac Richards to stuartm

comment:2 Changed 13 years ago by mythtv@…

This will most likely fix the behaviour I saw in #2668 when the 's' key was bound to both the program guide in LiveTV and also the program guide jump point.

As no one ever commented on that ticket I will leave it as is. stuartm at some point became a CC so I will leave it up to more knowledgable people to make the decision as to whether #2668 is pointless and should be closed.

Johan.

comment:3 Changed 13 years ago by stuartm

Resolution: fixed
Status: newclosed

(In [13614]) Swap the order in which jumppoints and keybindings are processed. Local and global context keybindings are now considered first followed by jumpoints.

Note, you should check for duplicate bindings if jumppoints no longer appear to work.

Patch by John Poet.

Closes #3363

comment:4 Changed 13 years ago by stuartm

(In [13615]) Reverts [13614] refs #3363 (Reverse order in which jumppoints and keybindings are handled)

Specific fixes to handle the situations where a jumppoint should be context sensitive. E.g. The Guide jumppoint in livetv would be preferred.

Please keep any discussion in the mailing list and don't re-open #3363

comment:5 Changed 13 years ago by Roo <roo.watt@…>

See #3642 for a different approach.

Note: See TracTickets for help on using tickets.