View Single Post
ישן 22-07-12, 20:34   # 3
IgalSt
מנהל פורום, עסק רשום
 
IgalSt's Avatar
 
מיני פרופיל
תאריך הצטרפות: Oct 2005
מיקום: המרכז
גיל: 38
הודעות: 1,432
Send a message via Skype™ to IgalSt

IgalSt לא מחובר  

Thumbs down

כמה הערות לגבי קוד ה-jQuery עצמו, כדי לשפר וליעל אותו:

1. מומלץ להשתמש ב- on

2. מומלץ בחום לעשות cache לאובייקטים של jQuery ולא לפזר דולרים על ימין ועל שמאל.

3. כדאי לשרשר מספר פעולות ביחד. אם אתה עושה על אותו האלמנט fadeIn

(4. סטיילינג: תחליט אם אתה משתמש בגרש בודד או גרשיים, ותהיה עקבי)

כך שאת הקוד שלך הייתי משנה ככה:
קוד:
$('#Images').on("click", "img", function(){
	var img = $(this);
	var leftSide = $("#Left-Side");
	leftSide.fadeOut("slow", function(){
                 leftSide.fadeIn("slow");
                 $("#largeImage").attr('src', img.attr("src").replace("thumbs", "newsize"));
                 $("#description").html(img.attr("alt"));
	});
});
  Reply With Quote