[mythtv-commits] Ticket #5615: Add keyboard input to MythConfirmationDialog, exec() to new mythui Dialogs

MythTV mythtv at cvs.mythtv.org
Tue Aug 12 13:13:31 UTC 2008


#5615: Add keyboard input to MythConfirmationDialog, exec() to new mythui Dialogs
-------------------------+--------------------------------------------------
 Reporter:  nigel        |        Owner:  ijr 
     Type:  enhancement  |       Status:  new 
 Priority:  minor        |    Milestone:  0.22
Component:  mythtv       |      Version:  head
 Severity:  medium       |   Resolution:      
  Mlocked:  0            |  
-------------------------+--------------------------------------------------

Comment(by paulh):

 Great minds think alike I was just working on exactly the same idea. I've
 attached what I have for reference. It's only a quick proof of concept and
 isn't complete. I've tested it with the GANT theme which has the fading
 logo and it works fine.

 You would use it just like the old dialogs which would make converting
 things over easy:-
 {{{
    MythConfirmationDialog2 *dialog = new MythConfirmationDialog2("Are you
 sure?", true);
    bool res = dialog->Exec();

    if (res)
    {
          // do something
    }
 }}}

 As Stuart says your keyPressHander probably wont work as expected when you
 have both buttons visible? I think you should always give the currently
 focused widget a change to process the keypress first as well. And also
 while testing I found it a bad idea to pass unhandled keypresses on from
 the poupup because it caused unexpected behavior.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/5615#comment:1>
MythTV <http://www.mythtv.org/>
MythTV


More information about the mythtv-commits mailing list