This patch improves the UX of selecting a weather station by re-implementing it a column
of checkboxes that are visible by default at the top of the window, so that selecting a
weather station is follows the local top-to-bottom flow of the page.
When no weather stations are selected (as it is by default), everything else in the
window is disabled so it's completely obvious what you need to do to proceed.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
-1 From me, what happens if we have more Weather Providers won't the dialog box be cluttered? If we can sort that out then +1
what happens if we have more Weather Providers won't the dialog box be cluttered?
I wonder how many we actually offer?
Maybe the old way but with a more obvious label could work. Dunno...
Is that in the cards? Right now we have only four. Even if it goes up to eight, I don't think it's really a problem. The window is quite small and there's room to grow if needed, and this patch is solving a usability issue with the current approach.
Hoping to get back into working on the backend as for how many, well depends how many weather providers will provide free API keys to use, I guess this is ok for now then. +1
Can you also make locations show up as you type so that you don't need a search button?