Fixes #3032: toggler pattern is now <a href="#target" class="elgg-toggler">...</a> Wow, so much nicer.

git-svn-id: http://code.elgg.org/elgg/trunk@8538 36083f99-b078-4883-b0ff-0f9b5a30f544

1 files changed, 3 insertions, 6 deletions

diff --git a/js/lib/ui.js b/js/lib/ui.js
index df3c102ce..3118bb408 100644
--- a/js/lib/ui.js
+++ b/js/lib/ui.js
@@ -25,8 +25,7 @@ elgg.ui.init = function () {
  * Toggles an element based on clicking a separate element
  *
  * Use .elgg-toggler on the toggler element
- * Add another class to the toggler called elgg-toggles-<id>
- * The id of the element being toggled is <id>
+ * Set the href to target the item you want to toggle (<a href="#id-of-target">)
  *
  * @param {Object} event
  * @return void
@@ -34,11 +33,9 @@ elgg.ui.init = function () {
 elgg.ui.toggles = function(event) {
 	event.preventDefault();
 
-	$(this).toggleClass('elgg-state-active');
+	var target = $(this).toggleClass('elgg-state-active').attr('href');
 
-	var togglees = $(this).attr('class').match(/elgg-toggles-[^ ]*/i);
-
-	$('#' + togglees[0].replace('elgg-toggles-', '')).slideToggle('medium');
+	$(target).slideToggle('medium');
 }
 
 /**