Improve form validation messages.
Created attachment 48754 [details] Proposed patch
Created attachment 48756 [details] Proposed patch
Created attachment 50278 [details] Proposed patch (rev.3) Upload the same patch again to kick EWS.
(In reply to comment #3) > Created an attachment (id=50278) [details] > Proposed patch (rev.3) > > Upload the same patch again to kick EWS. Oh, I understand. svn-apply doesn't support Git binary diff.
It does not support delta binary diffs, only literal binary diffs. :)
Comment on attachment 50278 [details] Proposed patch (rev.3) This patch has been up for review for too long. Who's the right person to review this patch? Why has it stalled out? I don't know whether the new messages are better or worse than the old messages. Is there a spec for this stuff? Something has broken down here. LayoutTests/fast/forms/script-tests/validationMessage.js:16 + "'The value does not match to a character restriction for this field.'"); Where do these messages come from? Are they speced somewhere?
I have changed my mind. I won't change the existing form validation messages, and just add some additional parameters to validationMessage*() functions in order that a port can improve the messages.
Created attachment 73583 [details] Patch
Comment on attachment 73583 [details] Patch ok.
Created attachment 73873 [details] Patch 5
(In reply to comment #9) > (From update of attachment 73583 [details]) > ok. Thank you for r+. But I have updated the patch. I added a dedicated function for <input type=email multiple>. It should show a different message than <input type=email>.
Comment on attachment 73873 [details] Patch 5 ok.
Comment on attachment 73873 [details] Patch 5 Clearing flags on attachment: 73873 Committed r71996: <http://trac.webkit.org/changeset/71996>
All reviewed patches have been landed. Closing bug.