Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Remove inline onclick handlers #417

Closed
wants to merge 1 commit into
from
Jump to file or symbol
Failed to load files and symbols.
+3 −3
Split
View
@@ -1389,7 +1389,7 @@
var container = $("<div></div>", {
"class": "select2-container"
}).html([
- " <a href='#' onclick='return false;' class='select2-choice'>",
+ " <a href='#' class='select2-choice'>",
" <span></span><abbr class='select2-search-choice-close' style='display:none;'></abbr>",
" <div><b></b></div>" ,
"</a>",
@@ -1488,7 +1488,7 @@
}
}
}));
-
+ this.selection.bind("click", killEvent)
this.search.bind("focus", this.bind(function() {
this.selection.attr("tabIndex", "-1");
}));
@@ -2052,7 +2052,7 @@
var choice=$(
"<li class='select2-search-choice'>" +
" <div></div>" +
- " <a href='#' onclick='return false;' class='select2-search-choice-close' tabindex='-1'></a>" +
+ " <a href='#' class='select2-search-choice-close' tabindex='-1'></a>" +
"</li>"),
id = this.id(data),
val = this.getVal(),