Comment

Comment by zichy

Good job so far, it feels very lightweight and simple. It works well with different input devices, but the accessibility for screen readers could be improved:

* The first day immediately gets focused when opening the datepicker. It might be a bit confusing to just hear a number.

* There's enough space to spell the months in full.

* Use semantic abbrevations for weekdays, e.g. `<abbr title="Monday">Mo</abbr>`.

* Use `label`s which refer to the corresponding `select`.

* Use `table` instead of `div`s for the calendar view.

* Use (hidden) text instead of the `title` attribute in buttons.

Furthermore:

* Try using more unique classes for your elements, e.g. '.aeon-datepicker__select' instead of '.select'.

* `cursor: pointer` for buttons would be helpful to see how large the click area is.

parent zichy 6 years ago

Discussion

Live from the official Hacker News API

A modern reader for Hacker News · SvelteKit + Skeleton