Bug Tracker

Ticket #1370 (closed enhancement: wontfix)

Opened 7 years ago

Last modified 6 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 6 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.

Note: See TracTickets for help on using tickets.