Opened 8 years ago

Last modified 5 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 rdworth)

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 8 years ago by rdworth

Description: modified (diff)

comment:2 Changed 8 years ago by Assimilater

Submitted pull request, changed default to 'Pick Date'. https://github.com/jquery/jquery-ui/pull/362

Last edited 8 years ago by Corey Frang (previous) (diff)

comment:3 Changed 8 years ago by Scott González

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.

Last edited 8 years ago by Scott González (previous) (diff)

comment:4 Changed 8 years ago by rdworth

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 8 years ago by rdworth

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 7 years ago by vanderhoorn

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 6 years ago by Scott González

Milestone: 1.9.01.11.0

comment:8 Changed 6 years ago by mikesherov

Status: newopen

comment:9 Changed 6 years ago by Scott González

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 5 years ago by dylanb

@scott.gonzalez - in the interim, we could update the examples to use the buttonText option to set the text

comment:11 Changed 5 years ago by Scott González

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 5 years ago by Scott González

Milestone: 1.11.0none
Note: See TracTickets for help on using tickets.