It would be great for an option to display both Fahrenheit and Celsius!

CommentFileSizeAuthor
#2 weather.patch1.53 KBjrefano

Comments

jrefano’s picture

jrefano commented

Actually, I just modified the module to do this myself.

I'm not that familiar with submitting a patch but I'll figure it out and come back.

jrefano’s picture

jrefano commented
Version: 6.x-5.3 » 6.x-5.x-dev
Component: Miscellaneous » Code
Assigned: Unassigned » jrefano
Status: Active » Needs review
StatusFileSize
new weather.patch1.53 KB

Here's my patch to allow for Celsius and Fahrenheit in one block. I added two options so you can pick what order you'd like them to appear in.

jrefano’s picture

jrefano commented
Assigned: jrefano » Unassigned
toddy’s picture

toddy commented
Assigned: Unassigned » toddy
Status: Needs review » Fixed

Hi,

thanks for the suggestion and the initial patch. You forgot to also enable this feature for the windchill temperature. However, this is now part of weather 6.x-5.4.

Regards,
Tobias

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.