Opened 11 years ago

Closed 11 years ago

#5653 closed patch (fixed)

Need MythPopupBox::showOkPopup() replacement

Reported by: Nigel Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description

There are over 80 popup invocations, mostly informing the user of errors. They should be replaced by something that uses libmythui. This is the obvious replacement:

    MythConfirmationDialog *pop;
    MythScreenStack *stk = GetMythMainWindow()->GetStack("popup stack");
    pop = new MythConfirmationDialog(stk, message, false)
    if (pop->Create())
        stk->AddScreen(pop);
    else
        delete pop;

but 80 copies of that is wasteful. Patch contains a simple global function proposal (with some extra error checking)

Attachments (1)

showok.patch (1.8 KB) - added by Nigel 11 years ago.

Download all attachments as: .zip

Change History (2)

Changed 11 years ago by Nigel

Attachment: showok.patch added

comment:1 Changed 11 years ago by Nigel

Resolution: fixed
Status: newclosed

(In [18251]) Replace MythPopupBox::showOkPopup() with non-blocking MythUI replacement, ShowOkPopup?(). This will currently only display correctly where the underlying widgets have been converted from Qt exec() style (like in the main menu where EJECT is called). Closes #5653

Note: See TracTickets for help on using tickets.