prevent page jumping when utility links are clicked

Courseware · Jun 6, 2014 · c9231c7 · c9231c7
1 parent ff73a2c
commit c9231c7
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/static/js/bibliography.js b/static/js/bibliography.js
@@ -9,25 +9,33 @@ jQuery("#courseware-bibs-list").hide();
 jQuery("#courseware-bibs-form .add").hide();
 jQuery("ul.courseware-meta li.add").hide();
 
-jQuery("ul.courseware-meta li.show-bibs a").bind('click', function(){
+jQuery("ul.courseware-meta li.show-bibs a").on( 'click', function() {
     jQuery("ul.courseware-meta li.add").slideToggle();
     jQuery("#courseware-bibs-form .add").hide();
     jQuery("#courseware-bibs-list").slideToggle();
+	event.preventDefault();
+	return false;
 })
 
-jQuery("ul.courseware-meta li.add.bib").bind('click', function(){
+jQuery("ul.courseware-meta li.add.bib").on( 'click', function() {
     jQuery("#courseware-bibs-form .add").hide();
     jQuery("#courseware-bibs-form .existing").slideToggle();
+	event.preventDefault();
+	return false;
 })
 
-jQuery("ul.courseware-meta li.add.book").bind('click', function(){
+jQuery("ul.courseware-meta li.add.book").on( 'click', function() {
     jQuery("#courseware-bibs-form .add").hide();
     jQuery("#courseware-bibs-form .book").slideToggle();
+	event.preventDefault();
+	return false;
 })
 
-jQuery("ul.courseware-meta li.add.www").bind('click', function(){
+jQuery("ul.courseware-meta li.add.www").on( 'click', function() {
     jQuery("#courseware-bibs-form .add").hide();
     jQuery("#courseware-bibs-form .www").slideToggle();
+	event.preventDefault();
+	return false;
 })
 
 // Didn't find another way to 'localize' this

diff --git a/static/js/single-assignment.js b/static/js/single-assignment.js
@@ -3,6 +3,8 @@
  */
 jQuery('#courseware-responses-list').hide();
 
-jQuery('#responses').bind('click', function(){
-    jQuery('#courseware-responses-list').slideToggle();
+jQuery('#responses').on('click', function() {
+	jQuery('#courseware-responses-list').slideToggle();
+	event.preventDefault();
+	return false;
 });