Implement editing in basic view.

horde · Oct 25, 2013 · 9a9448c · 9a9448c
1 parent 5f224b0
commit 9a9448c
Show file tree

Hide file tree

Showing 8 changed files with 24 additions and 15 deletions.
diff --git a/imp/js/compose-base.js b/imp/js/compose-base.js
@@ -66,8 +66,7 @@ var ImpComposeBase = {
         }
 
         if (this.editor_on) {
-            s.removeClassName('fixed')
-                .update(identity.hsig);
+            s.update(identity.hsig);
 
             if (Object.isUndefined(this.rte_loaded)) {
                 CKEDITOR.on('instanceReady', function(evt) {
@@ -92,8 +91,7 @@ var ImpComposeBase = {
                 this.rte.destroy(true);
                 delete this.rte;
             }
-            s.addClassName('fixed')
-                .update(identity.sig);
+            s.update(identity.sig);
         }
     },
 

diff --git a/imp/lib/Basic/Compose.php b/imp/lib/Basic/Compose.php
@@ -525,6 +525,7 @@ protected function _init()
                         $header,
                         $identity,
                         array(
+                            'signature' => $this->vars->signature,
                             'encrypt' => $prefs->isLocked('default_encrypt') ? $prefs->getValue('default_encrypt') : $this->vars->encrypt_options,
                             'html' => $rtemode,
                             'pgp_attach_pubkey' => $this->vars->pgp_attach_pubkey,

diff --git a/imp/lib/Compose.php b/imp/lib/Compose.php
@@ -1340,7 +1340,7 @@ protected function _createMimeMessage(
         if (!empty($options['signature'])) {
             if (is_string($options['signature'])) {
                 if (empty($options['html'])) {
-                    $body .= $options['signature'];
+                    $body .= "\n" . $options['signature'];
                 } else {
                     $html_sig = $options['signature'];
                     $body .= "\n" . $injector->getInstance('Horde_Core_Factory_TextFilter')->filter($html_sig, 'Html2text');

diff --git a/imp/templates/basic/compose/compose.html.php b/imp/templates/basic/compose/compose.html.php
@@ -207,7 +207,9 @@
     <strong><?php echo _("Signature") ?></strong>
    </td>
    <td class="item">
-    <div id="signature" title="<?php echo _("The signature is not editable.") ?>"></div>
+    <div id="signatureBorder"<?php if (!$this->rtemode) echo ' class="signaturePlain"' ?>>
+     <textarea id="signature" name="signature" cols="80" rows="3" class="fixed"></textarea>
+    </div>
    </td>
   </tr>
 <?php endif; ?>

diff --git a/imp/templates/dynamic/compose.html.php b/imp/templates/dynamic/compose.html.php
@@ -209,7 +209,7 @@
     <?php echo _("Signature")?>:
    </div>
    <div id="signatureBorder">
-    <textarea id="signature" name="signature"<?php if (!$this->sigExpanded) echo ' style="display:none"' ?>></textarea>
+    <textarea id="signature" name="signature" class="fixed"<?php if (!$this->sigExpanded) echo ' style="display:none"' ?>></textarea>
    </div>
   </div>
 <?php endif; ?>

diff --git a/imp/themes/default/basic/screen.css b/imp/themes/default/basic/screen.css
@@ -224,7 +224,7 @@ div.msgActions, #fmanager div.folderActions {
 .composebody {
     padding: 2px;
 }
-#text_identity, #signatureBorder {
+#text_identity {
     width: 500px;
 }
 #compose td.light {
@@ -237,6 +237,14 @@ div.msgActions, #fmanager div.folderActions {
     margin-bottom: -8px;
 }
 
+#signature {
+    padding: 2px;
+}
+#signatureBorder.signaturePlain {
+    border: 0;
+    padding: 0;
+}
+
 .seen {
     background: #fff;
 }

diff --git a/imp/themes/default/dynamic/screen.css b/imp/themes/default/dynamic/screen.css
@@ -691,12 +691,19 @@ div.dimpActionsMsg #windowclose {
     padding: 10px;
 }
 #signatureBorder {
+    height: 4.4em;
     margin-top: 4px;
 }
 #signatureParent .label {
     white-space: nowrap;
     font-weight: bold;
 }
+#signature {
+    height: 100%;
+    width: 100%;
+    border: 0;
+    background-color: #fff;
+}
 #signatureToggle {
     cursor: pointer;
     float: left;

diff --git a/imp/themes/default/screen.css b/imp/themes/default/screen.css
@@ -139,17 +139,10 @@ div.partsTreeDiv {
     width: 500px;
 }
 #signatureBorder {
-    height: 4.4em;
     border: 1px solid #d0d0d0;
     background-color: #fff;
     padding: 0.2em 5px;
 }
-#signature {
-    height: 100%;
-    width: 100%;
-    border: 0;
-    background-color: #fff;
-}
 #signatureBorder .cke_skin_kama, #signatureBorder .cke_skin_kama .cke_wrapper {
     border: 0;
     padding: 0;