From 897035cb0b62a1452487eeebc532e689eff0977f Mon Sep 17 00:00:00 2001 From: Wolfgang Sourdeau Date: Mon, 5 May 2008 22:30:06 +0000 Subject: [PATCH] Monotone-Parent: a807a87708b05f4866dff41ca4ac7c75e9a98991 Monotone-Revision: 6d796bebf66ba67880bcd99d4b4acc55bac2e33f Monotone-Author: wsourdeau@inverse.ca Monotone-Date: 2008-05-05T22:30:06 Monotone-Branch: ca.inverse.sogo --- UI/WebServerResources/HTMLElement.js | 10 +++++--- UI/WebServerResources/UIxMailEditor.js | 35 +++++++++++++------------- 2 files changed, 24 insertions(+), 21 deletions(-) diff --git a/UI/WebServerResources/HTMLElement.js b/UI/WebServerResources/HTMLElement.js index e30fcba41..e225fc3c1 100644 --- a/UI/WebServerResources/HTMLElement.js +++ b/UI/WebServerResources/HTMLElement.js @@ -10,14 +10,16 @@ Element.addMethods({ childNodesWithTag: function(element, tagName) { element = $(element); + var matchingNodes = new Array(); var tagName = tagName.toUpperCase(); for (var i = 0; i < element.childNodes.length; i++) { - if (typeof(element.childNodes[i]) == "object" - && element.childNodes[i].tagName - && element.childNodes[i].tagName.toUpperCase() == tagName) - matchingNodes.push(element.childNodes[i]); + var childNode = $(element.childNodes[i]); + if (Object.isElement(childNode) + && childNode.tagName + && childNode.tagName.toUpperCase() == tagName) + matchingNodes.push(childNode); } return matchingNodes; diff --git a/UI/WebServerResources/UIxMailEditor.js b/UI/WebServerResources/UIxMailEditor.js index ee6a56764..45a2cdf22 100644 --- a/UI/WebServerResources/UIxMailEditor.js +++ b/UI/WebServerResources/UIxMailEditor.js @@ -97,10 +97,11 @@ function insertContact(inputNode, contactName, contactEmail) { function toggleAttachments() { var div = $("attachmentsArea"); - if (div.style.display) - div.style.display = ""; + var style = "" + div.getStyle("display"); + if (style.length) + div.setStyle({ display: "" }); else - div.style.display = "block;"; + div.setStyle({ display: "block" }); return false; } @@ -111,21 +112,22 @@ function updateInlineAttachmentList(sender, attachments) { var div = $("attachmentsArea"); if (attachments) count = attachments.length; - if (count) - { - var text = ""; - for (var i = 0; i < count; i++) { - text = text + attachments[i]; - text = text + '
'; - } - var e = $('compose_attachments_list'); - e.innerHTML = text; - if (!div.style.display) - div.style.display = "block;"; + if (count) { + var text = ""; + for (var i = 0; i < count; i++) { + text = text + attachments[i]; + text = text + '
'; } + + var e = $('compose_attachments_list'); + e.innerHTML = text; + var style = "" + div.getStyle("display"); + if (!style.length) + div.setStyle({display: "block"}); + } else - div.style.display = ""; + div.setStyle({display: ""}); } /* mail editor */ @@ -272,7 +274,6 @@ function onTextFirstFocus() { } function onTextContextMenu(event) { - log("coucou"); event.returnValue = true; event.cancelBubble = true; } @@ -287,7 +288,7 @@ function onTextMouseDown(event) { function initMailEditor() { var list = $("attachments"); $(list).attachMenu("attachmentsMenu"); - var elements = list.childNodesWithTag("li"); + var elements = $(list).childNodesWithTag("li"); for (var i = 0; i < elements.length; i++) elements[i].observe("click", onRowClick);