search

WhatSock / apex

Apex 4X: The Comprehensive ARIA Development Suite
MIT License
29 stars 9 forks source link

dayToday class moves to selected day #28

Closed adam-delaney closed 2 years ago

adam-delaney commented 2 years ago

Issue

The dayToday class seems to move to the selected day. I would expect that dayToday class would remain on the date for today's Date. I can replicate the issue for this example: http://whatsock.com/Templates/Datepickers/Basic/index.htm.

Steps to reproduce

1) Launch the datePicker and inspect the class for today's date. This should have the dayToday. 2) Select another date other than today 3) Relaunch the datePicker and dayToday class is not on today's date.

accdc commented 2 years ago

Thanks, I just pushed a fix for this issue. The core is the same, but the Datepicker module includes logic to only set the today class and aria-current on the cell matching today's date. To load the new code, you may need to clear your browser file cache to make sure everything loads correctly when testing. I'll close this issue as fixed for now, but please reopen if you notice any other issues relating to this.

adam-delaney commented 2 years ago

I've tested this and it works as expected. Thanks @accdc for the quick fix!