Comments on: Создаем плагин для jQuery http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html Блог о программировании Mon, 06 Feb 2012 23:19:00 +0000 hourly 1 http://wordpress.org/?v= By: Владимир http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-4043 Владимир Thu, 12 Feb 2009 17:41:57 +0000 http://www.simplecoding.org/?p=663#comment-4043 Честно говоря, я ничего не забывал :) - это перевод статьи с nettuts.com. Большое спасибо за это замечание! Действительно из-за таких "мелочей" потом возникают очень неприятные баги. Последний вариант кода действительно красиво выглядит и хорошо демонстрирует возможности jQuery. Честно говоря, я ничего не забывал :) – это перевод статьи с nettuts.com.
Большое спасибо за это замечание! Действительно из-за таких "мелочей" потом возникают очень неприятные баги.

Последний вариант кода действительно красиво выглядит и хорошо демонстрирует возможности jQuery.

]]> By: Владимир http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-15788 Владимир Thu, 12 Feb 2009 17:41:00 +0000 http://www.simplecoding.org/?p=663#comment-15788 Честно говоря, я ничего не забывал :) - это перевод статьи с nettuts.com. Большое спасибо за это замечание! Действительно из-за таких "мелочей" потом возникают очень неприятные баги. Последний вариант кода действительно красиво выглядит и хорошо демонстрирует возможности jQuery. Честно говоря, я ничего не забывал :) – это перевод статьи с nettuts.com.
Большое спасибо за это замечание! Действительно из-за таких "мелочей" потом возникают очень неприятные баги.

Последний вариант кода действительно красиво выглядит и хорошо демонстрирует возможности jQuery.

]]> By: Евгений http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-4036 Евгений Wed, 11 Feb 2009 20:11:35 +0000 http://www.simplecoding.org/?p=663#comment-4036 хм, думаю, что правильнее все-таки так: <code> (function($){ $.fn.center = function(){ return this.each(function(){ $(this).load(function(){ var self = this; $(window).bind("resize", function(){ $(self).css({ "position" : "absolute", "left" : $(window).width() / 2 - $(self).width() / 2, "top" : $(window).height() /2 - $(self).height() / 2 }); }).triggerHandler('resize'); }); }); }; })(jQuery); </code> хм, думаю, что правильнее все-таки так:

(function($){
$.fn.center = function(){
return this.each(function(){
$(this).load(function(){
var self = this;
$(window).bind("resize", function(){
$(self).css({
"position" : "absolute",
"left" : $(window).width() / 2 - $(self).width() / 2,
"top" : $(window).height() /2 - $(self).height() / 2
});
}).triggerHandler('resize');
});
});
};
})(jQuery);

]]> By: Евгений http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-15787 Евгений Wed, 11 Feb 2009 20:11:00 +0000 http://www.simplecoding.org/?p=663#comment-15787 хм, думаю, что правильнее все-таки так: <code> (function($){ $.fn.center = function(){ return this.each(function(){ $(this).load(function(){ var self = this; $(window).bind("resize", function(){ $(self).css({ "position" : "absolute", "left" : $(window).width() / 2 - $(self).width() / 2, "top" : $(window).height() /2 - $(self).height() / 2 }); }).triggerHandler('resize'); }); }); }; })(jQuery); </code> хм, думаю, что правильнее все-таки так:

(function($){
$.fn.center = function(){
return this.each(function(){
$(this).load(function(){
var self = this;
$(window).bind("resize", function(){
$(self).css({
"position" : "absolute",
"left" : $(window).width() / 2 - $(self).width() / 2,
"top" : $(window).height() /2 - $(self).height() / 2
});
}).triggerHandler('resize');
});
});
};
})(jQuery);

]]> By: Евгений http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-4035 Евгений Wed, 11 Feb 2009 20:05:48 +0000 http://www.simplecoding.org/?p=663#comment-4035 Владимир, моя ошибка, в прошлых версиях jQuery trigger - не приводил к появлению события, а вызывал обработчики, как обычные функции. "Обновил" знания, предоженную вами задачу можно решить следующим способом: <code> (function($){ $.fn.center = function(){ return $(this).load(function(){ var self = this; $(window).bind("resize", function(){ $(self).css({ "position" : "absolute", "left" : $(window).width() / 2 - $(self).width() / 2, "top" : $(window).height() /2 - $(self).height() / 2 }); }).triggerHandler('resize'); }); }; })(jQuery); </code> PS. из функции $.fn.center вы забыли вернуть обернутый элемент, что привело бы к обрыванию "цепочки". Владимир, моя ошибка, в прошлых версиях jQuery trigger – не приводил к появлению события, а вызывал обработчики, как обычные функции. "Обновил" знания, предоженную вами задачу можно решить следующим способом:

(function($){
$.fn.center = function(){
return $(this).load(function(){
var self = this;
$(window).bind("resize", function(){
$(self).css({
"position" : "absolute",
"left" : $(window).width() / 2 - $(self).width() / 2,
"top" : $(window).height() /2 - $(self).height() / 2
});
}).triggerHandler('resize');
});
};
})(jQuery);

PS. из функции $.fn.center вы забыли вернуть обернутый элемент, что привело бы к обрыванию "цепочки".

]]> By: Евгений http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-15786 Евгений Wed, 11 Feb 2009 20:05:00 +0000 http://www.simplecoding.org/?p=663#comment-15786 Владимир, моя ошибка, в прошлых версиях jQuery trigger - не приводил к появлению события, а вызывал обработчики, как обычные функции. "Обновил" знания, предоженную вами задачу можно решить следующим способом: <code> (function($){ $.fn.center = function(){ return $(this).load(function(){ var self = this; $(window).bind("resize", function(){ $(self).css({ "position" : "absolute", "left" : $(window).width() / 2 - $(self).width() / 2, "top" : $(window).height() /2 - $(self).height() / 2 }); }).triggerHandler('resize'); }); }; })(jQuery); </code> PS. из функции $.fn.center вы забыли вернуть обернутый элемент, что привело бы к обрыванию "цепочки". Владимир, моя ошибка, в прошлых версиях jQuery trigger – не приводил к появлению события, а вызывал обработчики, как обычные функции. "Обновил" знания, предоженную вами задачу можно решить следующим способом:

(function($){
$.fn.center = function(){
return $(this).load(function(){
var self = this;
$(window).bind("resize", function(){
$(self).css({
"position" : "absolute",
"left" : $(window).width() / 2 - $(self).width() / 2,
"top" : $(window).height() /2 - $(self).height() / 2
});
}).triggerHandler('resize');
});
};
})(jQuery);

PS. из функции $.fn.center вы забыли вернуть обернутый элемент, что привело бы к обрыванию "цепочки".

]]> By: Владимир http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-4033 Владимир Wed, 11 Feb 2009 19:14:35 +0000 http://www.simplecoding.org/?p=663#comment-4033 Я согласен, что дублировать код - плохо. Но, во-первых, в данном случае речь идет о двух вызовах одной и той же функции. А они [функции] как раз и предназначены для того, чтобы можно было вызывать один и тотже код несколько раз. Если вызов только один, то в принципе нет смысла создавать функцию (можно сразу разместить нужный код внутри метода bind). Во-вторых, генерировать событие "resize" сразу после загрузки страницы только ради того, чтобы вызвать функцию, как-то не очень красиво ;) Я согласен, что дублировать код – плохо.
Но, во-первых, в данном случае речь идет о двух вызовах одной и той же функции. А они [функции] как раз и предназначены для того, чтобы можно было вызывать один и тотже код несколько раз.
Если вызов только один, то в принципе нет смысла создавать функцию (можно сразу разместить нужный код внутри метода bind).
Во-вторых, генерировать событие "resize" сразу после загрузки страницы только ради того, чтобы вызвать функцию, как-то не очень красиво ;)

]]> By: Владимир http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-15785 Владимир Wed, 11 Feb 2009 19:14:00 +0000 http://www.simplecoding.org/?p=663#comment-15785 Я согласен, что дублировать код - плохо. Но, во-первых, в данном случае речь идет о двух вызовах одной и той же функции. А они [функции] как раз и предназначены для того, чтобы можно было вызывать один и тотже код несколько раз. Если вызов только один, то в принципе нет смысла создавать функцию (можно сразу разместить нужный код внутри метода bind). Во-вторых, генерировать событие "resize" сразу после загрузки страницы только ради того, чтобы вызвать функцию, как-то не очень красиво ;) Я согласен, что дублировать код – плохо.
Но, во-первых, в данном случае речь идет о двух вызовах одной и той же функции. А они [функции] как раз и предназначены для того, чтобы можно было вызывать один и тотже код несколько раз.
Если вызов только один, то в принципе нет смысла создавать функцию (можно сразу разместить нужный код внутри метода bind).
Во-вторых, генерировать событие "resize" сразу после загрузки страницы только ради того, чтобы вызвать функцию, как-то не очень красиво ;)

]]> By: Евгений http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-4030 Евгений Wed, 11 Feb 2009 18:40:52 +0000 http://www.simplecoding.org/?p=663#comment-4030 Тогда лучше так: <code> $(this).load( function() { $(window).bind("resize", function(){ changeCss(); }).trigger("resize"); }); </code> дабы не дублировать. Тогда лучше так:


$(this).load( function() {
$(window).bind("resize", function(){
changeCss();
}).trigger("resize");
});

дабы не дублировать.

]]> By: Евгений http://www.simplecoding.org/sozdaem-plagin-dlya-jquery.html#comment-15784 Евгений Wed, 11 Feb 2009 18:40:00 +0000 http://www.simplecoding.org/?p=663#comment-15784 Тогда лучше так: <code> $(this).load( function() { $(window).bind("resize", function(){ changeCss(); }).trigger("resize"); }); </code> дабы не дублировать. Тогда лучше так:


$(this).load( function() {
$(window).bind("resize", function(){
changeCss();
}).trigger("resize");
});

дабы не дублировать.

]]>