Opened 16 years ago
Closed 16 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)
Change History (2)
Changed 16 years ago by
Attachment: | showok.patch added |
---|
comment:1 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
(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