Opened 14 years ago
Closed 14 years ago
Last modified 14 years ago
#8169 closed patch (fixed)
"Searching for scripts" busy dialog appears behind the current weather screen
Reported by: | 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)
Change History (5)
Changed 14 years ago by
Attachment: | weather_busydialog.patch added |
---|
comment:1 Changed 14 years ago by
Owner: | changed from Isaac Richards to paulh |
---|---|
Status: | new → assigned |
comment:2 Changed 14 years ago by
Milestone: | unknown → 0.23 |
---|
comment:3 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
(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 14 years ago by
(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.
This patch doesn't work for me, it just locked up the frontend when I tried it.