foo.oninput and foo.oninvalid should be working

RESOLVED FIXED in mozilla2.0b9

Status

()

Core
DOM: Events
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: mounir, Assigned: mounir)

Tracking

({html5})

Trunk
mozilla2.0b9
html5
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
It looks like a lot of properties to define event handlers are not available.

See the exhaustive list here:
http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#elements-in-the-dom

Comment 1

7 years ago
I think this is a dup. Currently we add onfoo listeners in nsDOMClassInfo, but
we should change that.
(Assignee)

Comment 2

7 years ago
(In reply to comment #1)
> I think this is a dup. Currently we add onfoo listeners in nsDOMClassInfo, but
> we should change that.

A dup of which bug?

Actually, this bug could be spitted in two:
1. oninput, oninvalid, onchange (and probably some other) properties doesn't work if set.
2. all on<eventName> are not valid properties of HTMLElement.

Is 2. the dup?
(Assignee)

Updated

6 years ago
Summary: Define oninput, oninvalid, onchange and other properties related to events on HTMLElement → foo.oninput and foo.oninvalid should be working
(Assignee)

Comment 3

6 years ago
Created attachment 499013 [details] [diff] [review]
Patch v1
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #499013 - Flags: review?(Olli.Pettay)
(Assignee)

Updated

6 years ago
Whiteboard: [passed-try][needs-review]

Updated

6 years ago
Attachment #499013 - Flags: review?(Olli.Pettay) → review+
(Assignee)

Updated

6 years ago
Attachment #499013 - Flags: approval2.0?
(Assignee)

Comment 4

6 years ago
Approval request: there is no way to check if oninput or oninvalid is supported by Firefox by an author because |"oninput" in inputElement| will always return false for the moment. I think we should make sure it will work given that we support the events.
Whiteboard: [passed-try][needs-review] → [passed-try][needs-approval]
Attachment #499013 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 5

6 years ago
Pushed:
https://hg.mozilla.org/mozilla-central/rev/59203494cf75
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [passed-try][needs-approval]
Target Milestone: --- → mozilla2.0b9
You need to log in before you can comment on or make changes to this bug.