Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#8169 closed patch (fixed)

"Searching for scripts" busy dialog appears behind the current weather screen

Reported by: Jonatan <mythtv@…> Owned by: paulh
Priority: minor Milestone: 0.23
Component: Plugin - MythWeather Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Pressing "m" in MythWeather? to enter the weather screen setup creates a busy dialog that appears behind the current weather screen.

The dialog uses the "popup stack" MythScreenStack? but it should be using "weather stack".

Attachments (1)

weather_busydialog.patch (616 bytes) - added by Jonatan <mythtv@…> 10 years ago.

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by Jonatan <mythtv@…>

Attachment: weather_busydialog.patch added

comment:1 Changed 10 years ago by paulh

Owner: changed from Isaac Richards to paulh
Status: newassigned

This patch doesn't work for me, it just locked up the frontend when I tried it.

comment:2 Changed 10 years ago by paulh

Milestone: unknown0.23

comment:3 Changed 10 years ago by paulh

Resolution: fixed
Status: assignedclosed

(In [23734]) Move the progress popup in MythWeather? to the weather stack so it's actually visible and also when clearing the screens make sure we are popping the right screens from the stack. Fixes #8169.

comment:4 Changed 10 years ago by sphery

(In [23817]) Fixes MythWeather? setup segfault.

The fix for #8169 assumed that the MythWeather? plugin was running when it changed the SourceManager::findScripts() busy popup to appear on the weather stack. When entering MythWeather? setup through Utilities/Setup?|Setup|Info Center Settings|Weather Settings|Screen Settings, this resulted in a NULL MythScreenStack?. Therefore, if the MythScreenStack? for the weather stack is null, just use the popup stack.

Fixes #8223. Refs #8169.

Note: See TracTickets for help on using tickets.