Closed Bug 600155 Opened 15 years ago Closed 15 years ago

Let the authors override validation message with x-moz-errormessage attribute

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b7

People

(Reporter: mounir, Assigned: mounir)

References

Details

(Keywords: dev-doc-complete, html5)

Attachments

(1 file, 1 obsolete file)

The idea is simple, if an element is invalid and mozErrorMessage is set, the error message will be overwritten by the string in mozErrorMessage. In some way, it lets the authors setting an error message without making the element suffering from a custom validity.
Jonas, should we add an IDL attribute for that? That might be more convenient but that might be annoying for authors if they use it and then we remove it, right?
I would say we should not add an IDL method. Seems better to add minimum amount of code to support an experimental feature.
Status: NEW → ASSIGNED
Summary: Let the authors override validation message with mozErrorMessage attribute → Let the authors override validation message with x-moz-errormessage attribute
Keywords: html5
Attached patch Patch v1 (obsolete) — Splinter Review
Assignee: nobody → mounir.lamouri
Attachment #479948 - Flags: review?(jonas)
Attached patch Patch v1Splinter Review
Attachment #479948 - Attachment is obsolete: true
Attachment #479950 - Flags: review?(jonas)
Attachment #479948 - Flags: review?(jonas)
Why such a very unusual naming for the attribute? Why x- prefix?
(In reply to comment #5) > Why such a very unusual naming for the attribute? Why x- prefix? I agree, that's horrible !
x-vendor is terrible. But I'll argue in the W3C bug.
(In reply to comment #8) > x-vendor is terrible. But I'll argue in the W3C bug. If you do, please, CC me :)
Attachment #479950 - Flags: approval2.0?
Attachment #479950 - Flags: approval2.0? → approval2.0+
Pushed: http://hg.mozilla.org/mozilla-central/rev/188670e40942 But marking Target Milestone: beta7 because I realized this has been pushed accidentally with a big merge after a backout. I've no idea why :-/ I backed it out and pushed it again to have correct log.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b7
Blocks: 344614
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: