Opened 12 years ago

Closed 11 years ago

#3722 closed task (fixed)

New Mythweather Tasks

Reported by: stuartm Owned by: stuartm
Priority: major Milestone: 0.21
Component: mythweather Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description (last modified by stuartm)

  • Screen setup could be a lot easier to use and should probably include a Finish button for saving changes instead of using 'M' (see the mythmusic visualizer dialog in trunk for a possible example)
  • The modules for the perl scripts should be installed to the perl directory and not /usr/{local/}share/mythtv/
  • If a script fails we should inform the user and continue display data for the scripts which worked. Current behaviour is just to continue showing the 'retrieving weather data' screen until the user hits escape.

Change History (22)

comment:1 Changed 12 years ago by stuartm

Description: modified (diff)

comment:2 Changed 12 years ago by Stev391

Please remove duplicate images:
text_button_on.png
text_button_off.png
text_button_pushed.png

As these are already included in the default mythtv installation (for svn:13926). and causes packaging issues.

comment:3 Changed 12 years ago by anonymous

Screens can not be added in setup. Each script fails with the "Add Screen Error" except the NWS alerts.

Search box for NWS alerts mangles numbers.

comment:4 Changed 12 years ago by stuartm

Owner: changed from ldunning to stuartm
Status: newassigned

comment:5 Changed 12 years ago by stuartm

I've just finished writing a BBC based grabber which provides data for the UK and international locations (major towns/cities), it should make a reasonable default for most users. I'll attach patches for testing soon, but I want to fix a few of the more annoying problems with the revamped mythweather first.

comment:6 Changed 12 years ago by stuartm

(In [14349]) Don't add formatting touches to the translation string in mythweather.

Refs #3722

comment:7 Changed 12 years ago by stuartm

(In [14350]) Makes a start on fixing a number of issues and bugs in the revamped mythweather.

  • Don't use MENU to save settings changes, adds a Finish button instead for better consistency with other areas of MythTV.
  • Don't hard code strings in the theme files but allow them to be translated (screen settings page)
  • In the source setting page, always select the first item in the list on loading.
  • Minor cleanup

Refs #3722

comment:8 Changed 12 years ago by stuartm

(In [14351]) Refs #3722

Update the wide version of weather-ui.xml for the changes made in [14350]

comment:9 Changed 12 years ago by stuartm

(In [14352]) Data unit fixes.

Refs #3722

comment:10 Changed 12 years ago by stuartm

(In [14353]) Don't wrap the wind speed in parentheses.

Refs #3722

comment:11 Changed 12 years ago by stuartm

(In [14354]) Expand the temperature textarea on the current observations page to use the available space and center align it. Fixes the temperature being obscured by other elements.

Refs #3722

comment:12 Changed 12 years ago by stuartm

(In [14434]) Adds the temperature units to the rest of the Mythweather screens.

Patch by maverik044

Closes #3918 and refs #3722

comment:13 Changed 12 years ago by stuartm

(In [14437]) Set the svn:executable property on the two new grabbers.

Refs #3811 and #3722

comment:14 Changed 12 years ago by stuartm

(In [14441]) Adds animated map grabber for the Environment Canada data source.

Closes #3811 and refs #3722

comment:15 Changed 11 years ago by stuartm

Status: acceptedstarted

comment:16 Changed 11 years ago by stuartm

(In [15908]) Move the weather grabbing scripts into subdirectories according to the source they grab from. This is purely to make it easier to manage the scripts down the line but is also necessary for that same reason.

NOTE you will need to rerun mythweather setup twice, the first time so that it deletes the old screens and a second time to add the re-add them. Apologies for this, but better this change is made now than after 0.21 is released where it will cause much more disruption.

Thanks to Joe Ripley for the original patch which made this possible.

Refs #3722

comment:17 Changed 11 years ago by stuartm

(In [16423]) Prompt for location after adding a new screen, avoids the confusion of the user have to bring up the menu first. Refs #3722

comment:18 Changed 11 years ago by stuartm

(In [16428]) More tweaks to mythweather screen setup. Refs #3722

comment:19 Changed 11 years ago by stuartm

(In [16437]) Reorganise and clean up the mythweather setup screens, no functional change but hopefully a little easier to understand. Refs #3722

comment:20 Changed 11 years ago by stuartm

(In [16441]) Don't add a screen to the inactive list if we don't have the data to actually use it. Refs #3722

comment:21 Changed 11 years ago by stuartm

(In [16451]) Usability improvements to mythweather setup. Inform users when a location search is occuring and list possible sources for each screen. Refs #3722

comment:22 Changed 11 years ago by stuartm

Resolution: fixed
Status: startedclosed

I think I've addressed some of the biggest problems with the current mythweather. Themes could still use some work and setup could be further improved but I'll look at those as separate issues from now on.

Note: See TracTickets for help on using tickets.