Opened 12 years ago
Last modified 9 years ago
#7467 open bug
Datepicker icon should have more descriptive alt text
Reported by: | rdworth | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | none |
Component: | ui.datepicker | Version: | 1.8.13 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description (last modified by )
The datepicker alt attribute value is currently '...' This is not sufficiently descriptive as alternative text. From http://forum.jquery.com/topic/alt
Change History (12)
comment:1 Changed 12 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 12 years ago by
I'm inclined to close this as wontfix. There's almost no chance of us landing something reasonable for all of our supported locales and this will almost certainly be handled in a different way with the rewrite.
comment:4 Changed 12 years ago by
I think english is a much better default than no language at all. Even once we switch to another system, a reasonable translation system will not return an empty string if there's no translation found, it will return the untranslated string. So I don't see a reason to not do the same here already.
comment:5 Changed 12 years ago by
maybe we should just default to 'date' as in input type=date. Imagine a non-english speaker with images turned off. Much better for them to see an input followed by the english word 'date' than an input followed by '...'
comment:6 Changed 12 years ago by
What about putting it in the regional files? Or is that not possible?
Another possibility would be to use the filename (without slashes and the file extension). So '/images/date_picker.png' would result in 'date picker', or something similar.
comment:7 Changed 11 years ago by
Milestone: | 1.9.0 → 1.11.0 |
---|
comment:8 Changed 11 years ago by
Status: | new → open |
---|
comment:9 Changed 10 years ago by
My preference is for the button option to just go away, but a final decision will have to wait for the API redesign.
comment:10 Changed 10 years ago by
@scott.gonzalez - in the interim, we could update the examples to use the buttonText option to set the text
comment:11 Changed 10 years ago by
I've created a ticket for this: #9626. Do you want to send a PR? If not, I can land an update during the week.
comment:12 Changed 9 years ago by
Milestone: | 1.11.0 → none |
---|
Submitted pull request, changed default to 'Pick Date'. https://github.com/jquery/jquery-ui/pull/362