Bug Tracker

Modify ↓

Ticket #1370 (closed enhancement: wontfix)

Opened 6 years ago

Last modified 5 years ago

prevent default action

Reported by: yangjh02@… Owned by: stefan
Priority: trivial Milestone: 1.1.4
Component: interface Version: 1.1.3
Keywords: Cc:
Blocking: Blocked by:

Description

In lots of plac of interface library, following code is used,

---quote---

if (window.event) {

window.event.cancelBubble = true; window.event.returnValue = false;

} else {

e.preventDefault(); e.stopPropagation();

}

return false;

--endquote----

I find that this is not necessary, because jquery already normalized the event object, just safely use e.preventDefault() and remove the the quoted code, it works in IE and firefox. This make the code cleaner.

Change History

comment:1 Changed 5 years ago by scott.gonzal

  • Status changed from new to closed
  • Resolution set to wontfix

Interface is no longer supported; consider switching to  jQuery UI.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.