Hello,

First - this weather module is AWESOME. I was looking for a few days before I ran across it and it's the only one to do forecasts and is pretty easily themeable. Thanks!

I'd be nice if there was an easy way to replace the default icons with custom icons. Even something as simple as exporting a custom_image variable that could be used in the .tpl file would be fine (so no extra config needed).

See the attached patch for a kind of hack that i did, but works for me. Note that I don't care about day/night icons, and it's easiest if the icon names are just the weather condition code as listed in yahoo_weather_forecast.inc (e.g. 0 for tornado), since the icon packs I find all have icons named numerically (e.g. icon sets from liquid weather).

CommentFileSizeAuthor
#5 ywf-623728-5.patch11.57 KBplach
#3 ywf-623728-3.patch11.49 KBplach
custom-images.patch683 bytesshark

Comments

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

I like this idea!

Perhaps a icon-set selector with a preview and icon-set packs posted here or in the module's page somewhere.

timtunbridge’s picture

timtunbridge commented

This gets my vote too.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Active » Needs review
StatusFileSize
new ywf-623728-3.patch11.49 KB

The attached patch lets specify a location outside the YWF module and optionally override the file names and extensions. An UI should be also provided, but let's have feedback on the approach taken, before. The patch should also ease block theming.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Version: 6.x-1.0 » 6.x-1.x-dev
plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
StatusFileSize
new ywf-623728-5.patch11.57 KB

Here is a reroll for D7 that can be applied to #1417284-8: [Yahoo Weather Forecast] D7 version?.