Repository navigation
Conversation
5580105 to
bcde6f8
Compare
|
@ashuntu @Feichtmeier Could you take a look at this? I've put a lot of work into it and would love to see it merged :D |
|
I'd be interested to see if @paul-geoghegan has input as well from an accessibility POV |
Feichtmeier
left a comment
There was a problem hiding this comment.
for me it looks good and it works!
thank you very much so for far for this huge work!
Let canonical people double check!
|
I have an exam today but will try look at this by the end of the week. |
|
Thanks a lot @paul-geoghegan 🙏 |
There was a problem hiding this comment.
I also think this looks good.
One thing, could we get a focus border on the month/year dropdown input? I suppose also on the actual month/years in the dropdown menu, similar to #1103, but that may be more invasive of a change.
|
@ashuntu Good point 👍 |
|
Thanks for working on this. I tested the calendar with a keyboard and a screen reader and found the following. Screen readers
Keyboard navigation
Today is wrong
Out-of-month days
Focus
Also noticed from the code (not yet verified at runtime)
Not yet checked
Happy to help with fixes. |
|
@paul-geoghegan Thanks for your inputs. I'll fix all your points next week. |
Uh oh!
There was an error while loading. Please reload this page.