Weather 3 Wishlist

[whurlston]

Hopefully from my point of view "todo sooner" rather than later as it was taking around 40 seconds to load each time the other day I was playing with the skin which is a tad painful when opening/closing it a lot as you have to when skinning (today it's only taking around 15 seconds which isn't so bad)

Working on this tonight.

Got it now thanks..... any chance of a "0%" zoom as well for us living on smaller islands as what is happening over Qz & up in the Islands is of some importance to whats likely to happen to our weather in the coming days/weeks & currently with 25% zoom I can't actually see the whole picture.

I will add the option for custom zoom levels.

One other wishlist item (I just checked out on the Accuweather site) is if you select degreesC at the top of the site it also gives you degreesC for the maps where as it is showing me degreesF on the maps in Weather3 & as we went metric for this stuff when I was just a little fellow degreesF really don't mean damn all to me now.

That would be an oversight on my part. I intended for maps to use the user's preferred measurement system.

[Jaggy]

Thank you for all that

Sorry to say I have uncovered a bug or two..... I changed from metric to imperial (for skinning size testing) & noticed it had un-selected the maps, on switching back to metric & trying to re-select them it throws this error now (it was fine when I first set it up though without any errors)

Code:

See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.

************** Exception Text **************
System.NullReferenceException: Object reference not set to an instance of an object.
   at NExtPlugins.Weather.Plugin.SettingsForm.chkLBEnabledMaps_ItemCheck(Object sender, ItemCheckEventArgs e) in d:\New folder\NPVR Plugins\Weather3\Plugin\SettingsForm.cs:line 432
   at System.Windows.Forms.CheckedListBox.OnItemCheck(ItemCheckEventArgs ice)
   at System.Windows.Forms.CheckedListBox.LbnSelChange()
   at System.Windows.Forms.CheckedListBox.WmReflectCommand(Message& m)
   at System.Windows.Forms.ListBox.WndProc(Message& m)
   at System.Windows.Forms.CheckedListBox.WndProc(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

The other bug is, at least for the wind images, it isn't loading my modified ones from NPVR\Skin\JustRetro\Weather\Wind as it's still displaying the default ones that came with it.

[whurlston]

Thank you for all that

Sorry to say I have uncovered a bug or two..... I changed from metric to imperial (for skinning size testing) & noticed it had un-selected the maps, on switching back to metric & trying to re-select them it throws this error now (it was fine when I first set it up though without any errors)

That's why it's beta I'll look into this one.

The other bug is, at least for the wind images, it isn't loading my modified ones from NPVR\Skin\JustRetro\Weather\Wind as it's still displaying the default ones that came with it.

Do you have an NPVR\Skin\JustRetro\Weather\skin.xml file in that directory? If not, it falls back to the Default skin and will load the images from there. For temporary purposes, you can just copy the default skin into that directory.

[whurlston]

Jaggy, would you prefer to have a skin element for each section (Current, Hourly, Daily, etc) or would you rather stick with simple ListViewItem, ListSelectionMetadata and CoversViewItem, CoversSelectionMetadata?

[Jaggy]

Do you have an NPVR\Skin\JustRetro\Weather\skin.xml file in that directory? If not, it falls back to the Default skin and will load the images from there. For temporary purposes, you can just copy the default skin into that directory.

Oi you!!! I didn't start doing this skinning stuff yesterday you know Of course I've got a skin.xml in the JustRetro directory that I'm butchering (& I also have the images sub directories) it's just that it's not using the modified images I have in the wind one & must be using the default skin ones

Jaggy, would you prefer to have a skin element for each section (Current, Hourly, Daily, etc) or would you rather stick with simple ListViewItem, ListSelectionMetadata and CoversViewItem, CoversSelectionMetadata?

What is there now (while confusing to start with) works pretty well, I have just added things like

<!-- ~~~~~~~~~~~~ -->
<!-- Current Menu Items -->
<!-- ~~~~~~~~~~~~ -->

<!-- ~~~~~~~~~~~~~~~~ -->
<!-- Location Title for All Views -->
<!-- ~~~~~~~~~~~~~~~~ -->

at the top of each section to make them easy to find later.......

The only ones that may be an issue later on are the map as it would be useful to be able to skin them differently.... but I can also see that as being overly complicated so.....

[whurlston]

Oi you!!! I didn't start doing this skinning stuff yesterday you know Of course I've got a skin.xml in the JustRetro directory that I'm butchering (& I also have the images sub directories) it's just that it's not using the modified images I have in the wind one & must be using the default skin ones

I only asked because it works for me here. I will add some extra logging to the next build to try to figure it out.

What is there now (while confusing to start with) works pretty well, I have just added things like

<!-- ~~~~~~~~~~~~ -->
<!-- Current Menu Items -->
<!-- ~~~~~~~~~~~~ -->

<!-- ~~~~~~~~~~~~~~~~ -->
<!-- Location Title for All Views -->
<!-- ~~~~~~~~~~~~~~~~ -->

at the top of each section to make them easy to find later.......

The only ones that may be an issue later on are the map as it would be useful to be able to skin them differently.... but I can also see that as being overly complicated so.....

K. I can move things around however you like. I will have that guide ready for you tomorrow.

[Jaggy]

I only asked because it works for me here. I will add some extra logging to the next build to try to figure it out.

HoKay this is f'ing weird, after a shut down over night & a clean boot this morning it is now showing the modified wind icons from the correct sub directory..... don't ask me.....

[whurlston]

Maybe because I am loading them from sub's ImageCache routine. I will change that for the next build.

[Jaggy]

On second fire up (on both the PC & NMT both showed the right images first fire) it is now showing the wrong images again..... & while yesterday I said I hadn't checked any of the other images, that was a small lie, I had forgotten I have changed the Icons to the ones from Weather2 & they are working fine.

Another one that has now cropped up twice, is it occasionally doesn't seem to display the Element name="Backdrop" (which I have my masks loading in as well as the backdrop) as it doesn't display the backdrop image or the masks.

[ACTCMS]

• You can also choose the default language and measurement system that you want. This setting is also a global setting and will be applied across all locations.
  

In the UK we are pretty much metricised now - except for distance...

It would be nice if you could add a separate Measurement Unit setting for distance so I could see temps in centigrade and wind speeds/visibility in miles.

Otherwise, it looks great - nice work...

Alex