Modify ↓
Ticket #10794 (closed bug: fixed)
.triggerHandler should not .preventDefault()
| Reported by: | dmethvin | Owned by: | dmethvin |
|---|---|---|---|
| Priority: | high | Milestone: | 1.7.1 |
| Component: | event | Version: | 1.7 |
| Keywords: | Cc: | ||
| Blocking: | #10699 | Blocked by: |
Description
Currently, .triggerHandler() calls event.preventDefault() to avoid the default action running. However, this prevents the caller from using the default action indication that might be returned by one of the handlers. Since we have an onlyHandlers variable we can use that instead and not mess with the event.
Change History
comment:1 Changed 4 days ago by dmethvin
- Owner set to dmethvin
- Priority changed from undecided to high
- Status changed from new to assigned
- Component changed from unfiled to event
- Milestone changed from None to 1.7.1
comment:3 Changed 4 days ago by Dave Methvin
- Status changed from assigned to closed
- Resolution set to fixed
Fix #10794. .triggerHandler() should not .preventDefault().
This also provides a resolution for #10699.
Changeset: 6736dd7168ccd1e6d72aca2e3c89d685c5d9c63a
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.