From dea2b1a8be4034d576f3341a55df014920bc8435 Mon Sep 17 00:00:00 2001 From: Igor Zinovyev Date: Wed, 7 Aug 2013 18:45:40 +0400 Subject: [PATCH 1/2] Fixed the attribute setting to conform to jQuery 1.9 specifications. --- jquery.jeditable.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/jquery.jeditable.js b/jquery.jeditable.js index eb8a911..3c21d83 100644 --- a/jquery.jeditable.js +++ b/jquery.jeditable.js @@ -506,7 +506,7 @@ $('select', this).children().each(function() { if ($(this).val() == json['selected'] || $(this).text() == $.trim(original.revert)) { - $(this).attr('selected', 'selected'); + $(this).prop('selected', true); } }); /* Submit on change if no submit button defined. */ From 362f182c5672617884fd08d9b3bd4bd0039b3908 Mon Sep 17 00:00:00 2001 From: Igor Zinovyev Date: Wed, 28 Aug 2013 19:07:42 +0400 Subject: [PATCH 2/2] Fixed the container height and width bug. --- jquery.jeditable.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/jquery.jeditable.js b/jquery.jeditable.js index 3c21d83..5bfa987 100644 --- a/jquery.jeditable.js +++ b/jquery.jeditable.js @@ -452,8 +452,8 @@ text: { element : function(settings, original) { var input = $(''); - if (settings.width != 'none') { input.attr('width', settings.width); } - if (settings.height != 'none') { input.attr('height', settings.height); } + if (settings.width != 'none') { input.width(settings.width); } + if (settings.height != 'none') { input.height(settings.height); } /* https://bugzilla.mozilla.org/show_bug.cgi?id=236791 */ //input[0].setAttribute('autocomplete','off'); input.attr('autocomplete','off');