search

Baremetrics / calendar

Date range picker for Baremetrics
MIT License
679 stars 78 forks source link

Shouldn't split on – #27

Closed SleeplessByte closed 8 years ago

SleeplessByte commented 8 years ago

It's not available in all fonts.

Rather use an – in the first place and annotate the two dates in separate elements. Also removes the need to translate the dash using css.

kalepail commented 8 years ago

https://github.com/Baremetrics/calendar/commit/3c4b5bc16c8ee67e95ab6fb5ccc502400a34fda5

SleeplessByte commented 8 years ago

Well, you're still splitting on that character, so didn't this break the calendar?

kalepail commented 8 years ago

What do you mean "splitting"? Not quite sure what you're referring to.

SleeplessByte commented 8 years ago

Nvm, you didn't change the presets, which wouldn't work because of Calendar.js#L43. :)