Posts: 2,087
Joined: Jun 2007
Reputation:
92
djh_
Aeon Project Founder
Posts: 2,087
Would it be possible to get an infolabel with just the numbers for weather temperatures, ditching the bit with degrees celsius etc? Alternatively, maybe create a separate label for the units you can stick on the end. I ask because having the label so wide makes things a bit cumbersome for skinning - in my experience, at least.
Posts: 26,215
Joined: Oct 2003
Reputation:
187
Looks doable.
The question is w(ea|he)ther or not I should change weather.temperature and add weather.temperatureandunits and weather.units.
I can easily update all the skins in the skinning SVN, along with PM3 and PM3.HD ofcourse.
That would leave Aeon, MiniMeedia and Focus as being out of date. Hardly a deal breaker - it just won't show the units.
Thoughts?
Posts: 2,087
Joined: Jun 2007
Reputation:
92
djh_
Aeon Project Founder
Posts: 2,087
Sounds like a good solution to me. Aeon not being updated won't be a problem in a few weeks anyway.
Posts: 3,909
Joined: Dec 2004
Reputation:
20
Nuka1195
Skilled Python Coder
Posts: 3,909
jmarshall, remember there are now window.property()'s for weather that are available throughout the skin.
would it be better to just use those? it would tie in better if you accept the weather plugin patch (or a variant).
Posts: 26,215
Joined: Oct 2003
Reputation:
187
Good idea Nuka1195.
I've added System.TemperatureUnits and changed all the weather Window.Property()'s to no longer have the units. r18045.
Duncan: Check out PM3.HD for how it's done. All of the ones in MyWeather.xml are available everywhere in the skin btw.
Cheers,
Jonathan
Posts: 3,909
Joined: Dec 2004
Reputation:
20
Nuka1195
Skilled Python Coder
Posts: 3,909
2009-02-24, 23:03
(This post was last modified: 2009-02-24, 23:13 by Nuka1195.)
one slight issue, when the temp does not show, like High dissappears after a certain time of day, the temp units still display.
would nesting the $INFO[property,,$INFO[tmpunit]] fix that?
edit: yes this works.
<label>$INFO[Window.Property(Day0.HighTemp),,$INFO[System.TemperatureUnits]]</label>
Posts: 3,909
Joined: Dec 2004
Reputation:
20
Nuka1195
Skilled Python Coder
Posts: 3,909
<label>$INFO[Window.Property(Day0.HighTemp),,$INFO[System.TemperatureUnits]]</label>
while that works if day0.hightemp is empty if it's not the second $INFO[] does not resolve.
should it?
Posts: 26,215
Joined: Oct 2003
Reputation:
187
No, it shouldn't. Recursive info labels aren't an option.
Posts: 2,087
Joined: Jun 2007
Reputation:
92
djh_
Aeon Project Founder
Posts: 2,087
Are we any closer to getting the Refresh / Location functionality on the Home page? Not that I have a clue how such a thing would be done.