Closed Bug 90221 Opened 23 years ago Closed 23 years ago

[code] nsIDOMXULElement::Click() doesn't simulate a real click

Categories

(Core :: XUL, defect)

Product:
Core
Component:
XUL
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla0.9.9

People

(Reporter: caustin, Assigned: jag+mozilla)

References

Details

Attachments

(1 file, 4 obsolete files)

updated per hyatt's comments
4.98 KB, patch
jag+mozilla
: review+
jag+mozilla
: superreview+
Details | Diff | Splinter Review
For example, in the checkbox/button case, it merely gives the element focus and doesn't fire oncommand or toggle the checked attribute. I need this for bug 959. On the other hand, if click() isn't supposed to actually simulate a click (what's it for then?), how would I go about simulating a real click?
Blocks: Accesskey-XUL
'suppose this one goes to me
Assignee: trudelle → aegis
Attached patch it's a mouse event! (obsolete) — — Splinter Review
Attached patch take two (obsolete) — — Splinter Review
sr=blake
This has to go in before (or with) bug 959.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.3
pushing to 0.9.4 for aegis. won't make the 0.9.3 train unfortunately...
Target Milestone: mozilla0.9.3 → mozilla0.9.4
I no longer have time for these. Anyone else want to take them?
May God have mercy on us all. The 212 bug spam-o-rama is Now!
QA Contact: aegis → jrgm
-> jag, 0.9.5 (this'll get fixed with bug 959)
Assignee: aegis → jaggernaut
Status: ASSIGNED → NEW
Target Milestone: mozilla0.9.4 → mozilla0.9.5
-> saari One more for your collection.
Assignee: jaggernaut → saari
-> 0.9.6
Status: NEW → ASSIGNED
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Target Milestone: mozilla0.9.6 → mozilla0.9.9
-> me
Assignee: saari → jaggernaut
Status: ASSIGNED → NEW
Attachment #41851 - Attachment is obsolete: true
Attachment #41857 - Attachment is obsolete: true
Comment on attachment 67165 [details] [diff] [review] Make click() act like a real click, including mousedown and mouseup r=jst
Attachment #67165 - Flags: review+
You need to reset the status variable's value in between each HandleDOMEvent. Nit: Also remove the extra braces. Fix that, and sr=hyatt
Attachment #67165 - Attachment is obsolete: true
Attachment #67355 - Attachment is obsolete: true
Comment on attachment 67513 [details] [diff] [review] updated per hyatt's comments Copying over r= and marking sr=hyatt
Attachment #67513 - Flags: superreview+
Attachment #67513 - Flags: review+
Checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: