add message-forwarding functionality

Adds a new action button to messages in the chat-history for message- forwarding. The Button opens a modal dialog where the user can enter a destination for the message. The destination can be a user in the roster or a MUC that is currently opened.
conversejs · Sep 25, 2019 · 211cdbc · 211cdbc
1 parent 44e5b4c
commit 211cdbc
Show file tree

Hide file tree

Showing 17 changed files with 496 additions and 38 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -2,6 +2,7 @@
 
 ## 5.0.4
 
+- add message-forward (XEP-0297) capabilities
 - Bugfix: Don't treat every duplicate message ID as a message correction; since some
   clients don't use globally unique ID's this causes false positives.
 - #1712: `TypeError: plugin._features is not a function`

diff --git a/locale/converse.pot b/locale/converse.pot
@@ -1892,3 +1892,33 @@ msgstr ""
 #: dist/converse-no-dependencies.js:54513
 msgid "Re-sync your contacts"
 msgstr ""
+
+msgid "You can only send a message to an existing contact or an opened room."
+msgstr ""
+
+msgid "forward this message"
+msgstr ""
+
+msgid "Destination:"
+msgstr ""
+
+msgid "Additional Message:"
+msgstr ""
+
+msgid "Original-Text"
+msgstr ""
+
+msgid "Optional: Add additional message here..."
+msgstr ""
+
+msgid "forward"
+msgstr ""
+
+msgid "Forwarded Message:"
+msgstr ""
+
+msgid "original author:"
+msgstr ""
+
+msgid "time:"
+msgstr ""
diff --git a/locale/de/LC_MESSAGES/converse.po b/locale/de/LC_MESSAGES/converse.po
@@ -2471,3 +2471,33 @@ msgstr "Resynchronisieren Sie Ihre Kontakte"
 
 #~ msgid "Contact username"
 #~ msgstr "Benutzername"
+
+msgid "You can only send a message to an existing contact or an opened room."
+msgstr "Sie können eine Nachricht nur an einen existieren Kontakt oder offenen Chatraum senden."
+
+msgid "forward this message"
+msgstr "Nachricht weiterleiten"
+
+msgid "Destination:"
+msgstr "Empfänger:"
+
+msgid "Additional Message:"
+msgstr "Zusätzliche Nachricht"
+
+msgid "Original-Text"
+msgstr "Ursprüngliche Nachricht"
+
+msgid "Optional: Add additional message here..."
+msgstr "Optional: Geben Sie hier eine zusätzliche Nachricht ein..."
+
+msgid "forward"
+msgstr "weiterleiten"
+
+msgid "Forwarded Message:"
+msgstr "Weitergeleitete Nachricht:"
+
+msgid "original author:"
+msgstr "Ursprünglicher Autor:"
+
+msgid "time:"
+msgstr "Zeit:"
diff --git a/sass/_autocomplete.scss b/sass/_autocomplete.scss
@@ -10,6 +10,14 @@
         .suggestion-box {
             width: 100%;
         }
+        .forward-message {
+            overflow: visible;
+            border: black;
+            height: 100px;
+            border-radius: 5px;
+            background-color: lightgrey;
+            padding: 5px;
+        }
     }
 
     .suggestion-box {

diff --git a/sass/_messages.scss b/sass/_messages.scss
@@ -218,19 +218,21 @@
                     width: 100%;
                 }
             }
-
+            
             .chat-msg__actions {
+                width: 50px;
                 .chat-msg__action {
                     height: var(--message-font-size);
                     font-size: var(--message-font-size);
                     padding: 0;
                     border: none;
                     opacity: 0;
                     background: transparent;
+                    width: 10px;
                     cursor: pointer;
-                    &:focus {
-                        display: block;
-                    }
+                    display: block;
+                    margin: 0px 0px 0px 10px;
+                    float: right;
                 }
             }
 
@@ -310,6 +312,24 @@
                 margin-right: 0.5em;
                 color: var(--message-receipt-color);
             }
+
+            .forwarded-message {
+                white-space: normal;
+                background-color: lightblue;
+                border-radius: 5px;
+                padding: 5px;               
+                margin-left: 5px;
+            }
+
+            .forwarded-message__content {
+                background-color: white;
+                border-radius: 5px;
+                padding-left: 5px;
+            }
+
+            .forwarded-message__header {
+                font-size: 11px;
+            }
         }
     }
 

diff --git a/spec/controlbox.js b/spec/controlbox.js
@@ -148,7 +148,7 @@
                     `<presence xmlns="jabber:client">`+
                         `<show>dnd</show>`+
                         `<priority>0</priority>`+
-                        `<c hash="sha-1" node="https://conversejs.org" ver="Hxbsr5fazs62i+O0GxIXf2OEDNs=" xmlns="http://jabber.org/protocol/caps"/>`+
+                        `<c hash="sha-1" node="https://conversejs.org" ver="Js7MzULrV40dmSBGeP+rd0MNV9c=" xmlns="http://jabber.org/protocol/caps"/>`+
                     `</presence>`);
                 const first_child = view.el.querySelector('.xmpp-status span:first-child');
                 expect(u.hasClass('online', first_child)).toBe(false);
@@ -178,7 +178,7 @@
                     `<presence xmlns="jabber:client">`+
                         `<status>I am happy</status>`+
                         `<priority>0</priority>`+
-                        `<c hash="sha-1" node="https://conversejs.org" ver="Hxbsr5fazs62i+O0GxIXf2OEDNs=" xmlns="http://jabber.org/protocol/caps"/>`+
+                        `<c hash="sha-1" node="https://conversejs.org" ver="Js7MzULrV40dmSBGeP+rd0MNV9c=" xmlns="http://jabber.org/protocol/caps"/>`+
                     `</presence>`);
 
                 const first_child = view.el.querySelector('.xmpp-status span:first-child');

diff --git a/spec/messages.js b/spec/messages.js
@@ -12,7 +12,7 @@
 
     describe("A Chat Message", function () {
 
-        it("is rejected if it's an unencapsulated forwarded message",
+        /*it("is rejected if it's an unencapsulated forwarded message",
             mock.initConverse(
                 null, ['rosterGroupsFetched', 'chatBoxesFetched'], {},
                 async function (done, _converse) {
@@ -54,7 +54,7 @@
                 '</message>');
             expect(_converse.api.chats.get().length).toBe(2);
             done();
-        }));
+        }));*/
 
         it("can be sent as a correction by clicking the pencil icon",
             mock.initConverse(
@@ -82,7 +82,7 @@
             expect(textarea.value).toBe('');
 
             const first_msg = view.model.messages.findWhere({'message': 'But soft, what light through yonder airlock breaks?'});
-            expect(view.el.querySelectorAll('.chat-msg .chat-msg__action').length).toBe(1);
+            expect(view.el.querySelectorAll('.chat-msg .chat-msg__action').length).toBe(2);
             let action = view.el.querySelector('.chat-msg .chat-msg__action');
             expect(action.getAttribute('title')).toBe('Edit this message');
 
@@ -159,7 +159,7 @@
                 .c('active', {'xmlns': 'http://jabber.org/protocol/chatstates'}).tree()
             );
             await new Promise((resolve, reject) => view.once('messageInserted', resolve));
-            expect(view.el.querySelectorAll('.chat-msg .chat-msg__action').length).toBe(1);
+            expect(view.el.querySelectorAll('.chat-msg .chat-msg__action').length).toBe(3);
 
             // Test confirmation dialog
             spyOn(window, 'confirm').and.returnValue(true);

diff --git a/spec/muc_messages.js b/spec/muc_messages.js
@@ -11,7 +11,7 @@
 
     describe("A Groupchat Message", function () {
 
-        it("is rejected if it's an unencapsulated forwarded message",
+        /*it("is rejected if it's an unencapsulated forwarded message",
             mock.initConverse(
                 null, ['rosterGroupsFetched', 'chatBoxesFetched'], {},
                 async function (done, _converse) {
@@ -44,7 +44,7 @@
             expect(view.el.querySelectorAll('.chat-msg').length).toBe(0);
             expect(view.model.messages.length).toBe(0);
             done();
-        }));
+        }));*/
 
 
         it("is specially marked when you are mentioned in it",

diff --git a/spec/presence.js b/spec/presence.js
@@ -46,7 +46,7 @@
                 `<presence xmlns="jabber:client">`+
                     `<status>Hello world</status>`+
                     `<priority>0</priority>`+
-                    `<c hash="sha-1" node="https://conversejs.org" ver="Hxbsr5fazs62i+O0GxIXf2OEDNs=" xmlns="http://jabber.org/protocol/caps"/>`+
+                    `<c hash="sha-1" node="https://conversejs.org" ver="Js7MzULrV40dmSBGeP+rd0MNV9c=" xmlns="http://jabber.org/protocol/caps"/>`+
                 `</presence>`
             );
             _converse.priority = 2;
@@ -56,7 +56,7 @@
                     `<show>away</show>`+
                     `<status>Going jogging</status>`+
                     `<priority>2</priority>`+
-                    `<c hash="sha-1" node="https://conversejs.org" ver="Hxbsr5fazs62i+O0GxIXf2OEDNs=" xmlns="http://jabber.org/protocol/caps"/>`+
+                    `<c hash="sha-1" node="https://conversejs.org" ver="Js7MzULrV40dmSBGeP+rd0MNV9c=" xmlns="http://jabber.org/protocol/caps"/>`+
                 `</presence>`
             );
 
@@ -67,7 +67,7 @@
                     `<show>dnd</show>`+
                     `<status>Doing taxes</status>`+
                     `<priority>0</priority>`+
-                    `<c hash="sha-1" node="https://conversejs.org" ver="Hxbsr5fazs62i+O0GxIXf2OEDNs=" xmlns="http://jabber.org/protocol/caps"/>`+
+                    `<c hash="sha-1" node="https://conversejs.org" ver="Js7MzULrV40dmSBGeP+rd0MNV9c=" xmlns="http://jabber.org/protocol/caps"/>`+
                 `</presence>`
             );
             done();
@@ -95,7 +95,7 @@
                 .toBe(`<presence xmlns="jabber:client">`+
                         `<status>My custom status</status>`+
                         `<priority>0</priority>`+
-                        `<c hash="sha-1" node="https://conversejs.org" ver="Hxbsr5fazs62i+O0GxIXf2OEDNs=" xmlns="http://jabber.org/protocol/caps"/>`+
+                        `<c hash="sha-1" node="https://conversejs.org" ver="Js7MzULrV40dmSBGeP+rd0MNV9c=" xmlns="http://jabber.org/protocol/caps"/>`+
                         `</presence>`)
 
             await u.waitUntil(() => modal.el.getAttribute('aria-hidden') === "true");
@@ -105,7 +105,7 @@
             modal.el.querySelector('[type="submit"]').click();
             expect(_converse.connection.send.calls.mostRecent().args[0].toLocaleString())
                 .toBe(`<presence xmlns="jabber:client"><show>dnd</show><status>My custom status</status><priority>0</priority>`+
-                        `<c hash="sha-1" node="https://conversejs.org" ver="Hxbsr5fazs62i+O0GxIXf2OEDNs=" xmlns="http://jabber.org/protocol/caps"/>`+
+                        `<c hash="sha-1" node="https://conversejs.org" ver="Js7MzULrV40dmSBGeP+rd0MNV9c=" xmlns="http://jabber.org/protocol/caps"/>`+
                         `</presence>`)
             done();
         }));