Bladeren bron

I'm not a js expert, but I think it looks better with the function outside the onready

Signed-off-by: emanuele <emanuele45@gmail.com>
emanuele 12 jaren geleden
bovenliggende
commit
f9e107a0ba
1 gewijzigde bestanden met toevoegingen van 18 en 18 verwijderingen
  1. 18 18
      Themes/default/scripts/icondropdown.js

+ 18 - 18
Themes/default/scripts/icondropdown.js

@@ -13,22 +13,22 @@ $(document).ready(function()
 	
 	
 	$('#icon_image_input').change(function ()
 	$('#icon_image_input').change(function ()
 	{
 	{
-		refreshPreview();
+		refreshIconPreview();
 	});
 	});
-	
+
-	function refreshPreview()
+	refreshIconPreview();
-	{
+});
-		// Get the icon count element.
+
-		var icon_count = $('#icon_count_input');
+function refreshIconPreview()
-		var select_box = $('select#icon_image_input').val();
+{
-	
+	// Get the icon count element.
-		// If it's empty, set it to 1.
+	var icon_count = $('#icon_count_input');
-		if (icon_count.val() == 0 && select_box != 'blank.png')
+	var select_box = $('select#icon_image_input').val();
-			icon_count.val(1);
+
-	
+	// If it's empty, set it to 1.
-		// Update the icon preview.
+	if (icon_count.val() == 0 && select_box != 'blank.png')
-		$('#icon_preview').attr('src', smf_default_theme_url + '/images/membericons/' + select_box);
+		icon_count.val(1);
-	}
+
-	
+	// Update the icon preview.
-	refreshPreview();
+	$('#icon_preview').attr('src', smf_default_theme_url + '/images/membericons/' + select_box);
-});
+}