search

o355 / PyWeather

Viewing the weather in a CLI has never been this much fun. (ARCHIVED)
https://gitlab.com/o355/PyWeather
GNU General Public License v3.0
28 stars 8 forks source link

Enhancement: Changing radar size on the fly #50

Closed o355 closed 6 years ago

o355 commented 6 years ago

Description: Users can change the radar image size from inside the radar GUI through a settings window.

Checklist:

Estimated completion time: January 29, 2017

o355 commented 6 years ago

Now working on this enhancement. On-the-fly changes will be stored in the config file, as there's no easy way to change the radar GUI size without modifying the config by hand.

Now investigating where to put the button in the GUI & sub windows.

o355 commented 6 years ago

I've decided to make a menu (no buttons) which will have options for emptying the cache & changing radar size.

File will have the empty cache option and exit, separated by a spacebar. Config will have one option, "Radar Window Size", with radial drop downs. Options will list the name and the resolution (e.g. "Normal (600x600)").

Work is now beginning, and this feature should be done relatively soon.

o355 commented 6 years ago

The menus have been implemented, with basic functionality working. Now ultimately working on the handling of a radar size change.

o355 commented 6 years ago

The main code has been completed, and testing is now underway.

o355 commented 6 years ago

While all the code mostly works (switching to all radar sizes & switching without a config file still needs testing), I'm troubleshooting an issue where the viewer isn't properly resizing.

o355 commented 6 years ago

Still working on the proper frame resizing issue, hopefully I'll find a way soon.

o355 commented 6 years ago

Everything works! The extrasmall size does cause a few GUI issues, but that'll be dealt with later.

Now doing a little more testing and the feature is complete.

o355 commented 6 years ago

It works, but with one limitation: When changing the radar size, if you scale up, the GUI also scales up, but if you scale down the GUI doesn't scale down. I've tried everything I can to make it scale down, but nothing at the moment.

I'm going to continue investigating what's going on. appJar does have an option to force the GUI size, and I'll be investigating more.

o355 commented 6 years ago

I investigated more and noted the resolution sizes. I'll be adding it to the code, but my laptop is running at 110% scaling, so I won't call it quits until I hop on my desktop with 100% scaling and make sure things work.

o355 commented 6 years ago

Can't change the size of the GUI. At this point, radar size switching on the fly is complete.

o355 commented 6 years ago

Reopening this as there's still issues with that the radar images don't properly scale up to the image frame.

o355 commented 6 years ago

There was an issue with globalizing the _https variable for the radar URLs.

Now I'm thinking that whenever a radar size switch occurs that we kill then relaunch the radar.

o355 commented 6 years ago

Ok...never mind. It's a bit more complicated.

It's all fixed now.