From 491133a0b24133bdeda1a07bb1e7bd06c7346bea Mon Sep 17 00:00:00 2001
From: alecpl <alec@alec.pl>
Date: Sat, 14 May 2011 03:25:13 -0400
Subject: [PATCH] - Fixed listmenu (added workaround for http://bugs.jquery.com/ticket/9284) - Small code improvements and use preferred .prop() instead of .attr() for some properties
---
program/js/app.js | 26 ++++++++++----------------
1 files changed, 10 insertions(+), 16 deletions(-)
diff --git a/program/js/app.js b/program/js/app.js
index aac432f..b0ccf35 100644
--- a/program/js/app.js
+++ b/program/js/app.js
@@ -394,7 +394,7 @@
// display 'loading' message on form submit, lock submit button
$('form').submit(function () {
- $('input[type=submit]', this).attr('disabled', true);
+ $('input[type=submit]', this).prop('disabled', true);
rcmail.display_message('', 'loading');
});
@@ -4018,7 +4018,7 @@
var elem = $('#ff_'+col);
if (elem.length) {
elem.show().focus();
- $(menu).children('option[value="'+col+'"]').attr('disabled', true);
+ $(menu).children('option[value="'+col+'"]').prop('disabled', true);
}
else {
var lastelem = $('.ff_'+col),
@@ -4042,22 +4042,18 @@
if (colprop.type == 'text' || colprop.type == 'date') {
input = $('<input>')
.addClass('ff_'+col)
- .attr('type', 'text')
- .attr('name', '_'+col+name_suffix)
- .attr('size', colprop.size)
+ .attr({type: 'text', name: '_'+col+name_suffix, size: colprop.size})
.appendTo(cell);
this.init_edit_field(col, input);
}
else if (colprop.type == 'composite') {
var childcol, cp, first;
- for (var childcol in colprop.childs) {
+ for (childcol in colprop.childs) {
cp = colprop.childs[childcol];
input = $('<input>')
.addClass('ff_'+childcol)
- .attr('type', 'text')
- .attr('name', '_'+childcol+name_suffix)
- .attr('size', cp.size)
+ .attr({type: 'text', name: '_'+childcol+name_suffix, size: cp.size})
.appendTo(cell);
cell.append(" ");
this.init_edit_field(childcol, input);
@@ -4080,8 +4076,7 @@
if (input) {
var delbutton = $('<a href="#del"></a>')
.addClass('contactfieldbutton deletebutton')
- .attr('title', this.get_label('delete'))
- .attr('rel', col)
+ .attr({title: this.get_label('delete'), rel: col})
.html(this.env.delbutton)
.click(function(){ ref.delete_edit_field(this); return false })
.appendTo(cell);
@@ -4092,7 +4087,7 @@
// disable option if limit reached
if (!colprop.count) colprop.count = 0;
if (++colprop.count == colprop.limit && colprop.limit)
- $(menu).children('option[value="'+col+'"]').attr('disabled', true);
+ $(menu).children('option[value="'+col+'"]').prop('disabled', true);
}
}
}
@@ -4119,13 +4114,12 @@
if (addmenu.length) {
var option = addmenu.children('option[value="'+col+'"]');
if (option.length)
- option.attr('disabled', false);
+ option.prop('disabled', false);
else
option = $('<option>').attr('value', col).html(colprop.label).appendTo(addmenu);
addmenu.show();
}
};
-
this.upload_contact_photo = function(form)
{
@@ -4392,7 +4386,7 @@
row.cells[1].innerHTML = '*';
// update subscription checkbox
- $('input[name="_subscribed[]"]', row).val(name).attr('checked', true);
+ $('input[name="_subscribed[]"]', row).val(name).prop('checked', true);
}
this.init_subscription_list();
@@ -4501,7 +4495,7 @@
{
var id = this.get_folder_row_id(folder);
if (id)
- $('input[name="_subscribed[]"]', $('#'+id)).attr('disabled', true);
+ $('input[name="_subscribed[]"]', $('#'+id)).prop('disabled', true);
};
this.folder_size = function(folder)
--
Gitblit v1.9.1