Reset min date on the slider demos to lowest possible (0?)

Open-Historical-Map-Labs / openhistoricaltiles

First iteration of vector tiles from OHM Planet data

BSD 2-Clause "Simplified" License

3 stars 0 forks source link

Closed danrademacher closed 5 years ago

danrademacher commented 5 years ago

Looks like this is an almost instant change in the demo params, right?

For low end of "When am I?" and low end of time-slider, let's allow this to be "0" or "1" whichever works.

gregallensworth commented 5 years ago

[x] rename datespan to range to be consistent with wording throughout the code and documentation
[x] Demos change range to [0, CurrentYear]
- [x] MBGL demo
- [x] Leaflet demo
[x] fix breakages due to years not being 4 digits; tests so far have always had 4 digits, never gone to <1000
- [x] MBGL control
- [x] Leaflet wrapper

gregallensworth commented 5 years ago

MBGL demo:

Date range changed to [ 0, currentyear ] and will change with each calendar year.

Internal rename of the range parameter, and updated docs.

Reworked date filter logic, to pad out years to 4 digits, e.g. the year 718 would be represented by the bookends "0718-01-01" and "0718-12-31"

gregallensworth commented 5 years ago

Leaflet demo:

Date range is now [ 0, currentyear ] same as MBGL demo.

Changed to datespan parameter to range.

Confirmed that other behaviors such as the casting to 4 digit years, is handled by the underlying MBGL control which has been upgraded.