[dev] weatherdotcom changes

Rick Emery rick at emery.homelinux.net
Mon Mar 14 14:31:19 PST 2005


This message is in MIME format.

--=_53v6lyqjmxz4
Content-Type: text/plain;
	charset=ISO-8859-1;
	format="flowed"
Content-Disposition: inline
Content-Transfer-Encoding: 7bit

I've made some changes to the weatherdotcom block; it turns out that 
there is a lot of information provided by the Services_Weather module 
for forecasts that isn't being displayed (humidity, change of 
precipitation, and winds). So I added a preference for the user to 
display a brief forecast or a detailed forecast.

I could use some advice on handling the layout of each; a lot of the 
rendering code is duplicated, so if I can find a way to have a single 
display that would efficiently handle both layouts, I could eliminate a 
lot of it.

Right now (see attached screenshots), in the brief forecast, each 
day/night forecast takes one column, and all are displayed in one row 
(which gets really wide if the maximum 10 day forecast is selected). 
For the detailed forecast, there's too much information for that, so I 
have each day/night forecast in its own row. Any suggestions for a 
single layout that will handle both efficiently would be *greatly* 
appreciated.

In the mean time, this is my real question: should I post the patch 
here, or create an enhancement request on bugs.horde.org and post the 
patch there?

Thanks in advance,
Rick

-- 
Rick Emery

"When once you have tasted flight, you will forever walk the Earth
with your eyes turned skyward, for there you have been, and there
you will always long to return"
                                              -- Leonardo Da Vinci

--=_53v6lyqjmxz4
Content-Disposition: attachment;
	filename="briefwxtrimmed.png"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

A non-text attachment was scrubbed...
Name: briefwxtrimmed.png
Type: image/png
Size: 23995 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20050314/7ad8f4f1/briefwxtrimmed-0001.png

--=_53v6lyqjmxz4
Content-Disposition: attachment;
	filename="detailedwxtrimmed.png"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

A non-text attachment was scrubbed...
Name: detailedwxtrimmed.png
Type: image/png
Size: 32467 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20050314/7ad8f4f1/detailedwxtrimmed-0001.png

--=_53v6lyqjmxz4--



More information about the dev mailing list