-
Notifications
You must be signed in to change notification settings - Fork 22
Aria attributes #86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Aria attributes #86
Conversation
While i reviewed this an issue came to my mind about the month and year picker. Wouldn't it make more sense if there is just the month or year in the label? Because you would select this specific month in the year without a day if you still go to the day view. This would change if you have the Same goes if you have the With those examples only the last entry would be formatted which means it would look something like that: year -> month year This way the buttons aria labels always reflect what the user selects because on the normal datepicker he won't select a date on the year view only a number which represents the year. Does this make any sense? Kinda confusing what i wrote :D |
Hello @MrWook, you're absolutely right and I agree with you about the month and year labels. What you wrote makes sense 😄 I've gone ahead and changed this to a draft while we discuss this further. In my first pass, I was trying to achieve something similar to what you describe with it's use of Maybe the solution would be to use the format property only for |
I think we should check the minimum-view and format the aria label accordingly
This way the aria labels would match the button values and at the end the input value. |
I've been giving this some thought and wonder how to implement this while respecting the format property. That is, if the value of format is |
Like i said only the last view will have the formatted date inside the aria label. If it is not the last view only the text inside the button will be used for the aria label. So basically:
If the user has the day view as last view but formats the date like |
@MrWook @betweenbrain any updates on this pr getting merged? |
span
tobutton
..cell
elements fromspan
tobutton
.aria-controls
andaria-expanded
to Calendar Button.aria-label
to Calendar Button and interactive.cell
elements.