Fixes #1356

CHANGES.md

@@ -8,12 +8,14 @@
 - Add a new `converse-pubsub` plugin, for generic PubSub operations
 - #1180 It's now possible to use OMEMO in a MUC (if it's members-only and non-anonymous)
 - #1353 Message Delivery Receipts not working because of the message "type" attribute
+- #1356 Make triangle icon usable 
 - #1374 Can't load embedded chat when changing `view_mode` between page reloads
 - #1376 Fixed some alignment issues in the sidebar
 - #1378 Message Delivery Receipts were being sent for carbons and own messages
 - #1379 MUC unread messages indicator is failing
 - #1382 Message Delivery Receipts: Set store hint and type='chat'
 
+
 ## 4.0.6 (2018-12-07)
 
 - Updated translations: ar, cs, de, es, eu, fr, gl, hu, id, it, ja, nb, pt_BR

dist/converse.js

@@ -48985,7 +48985,7 @@ _converse_headless_converse_core__WEBPACK_IMPORTED_MODULE_0__["default"].plugins
           ev.preventDefault();
         }
 
-        const icon_el = ev.target.querySelector('.fa');
+        const icon_el = ev.target.matches('.fa') ? ev.target : ev.target.querySelector('.fa');
 
         if (u.hasClass('fa-caret-down', icon_el)) {
           u.slideIn(this.el.querySelector('.bookmarks'));
@@ -58913,7 +58913,7 @@ _converse_headless_converse_core__WEBPACK_IMPORTED_MODULE_0__["default"].plugins
           ev.preventDefault();
         }
 
-        const icon_el = ev.target.querySelector('.fa');
+        const icon_el = ev.target.matches('.fa') ? ev.target : ev.target.querySelector('.fa');
 
         if (icon_el.classList.contains("fa-caret-down")) {
           u.slideIn(this.el.querySelector('.open-rooms-list')).then(() => {
@@ -59748,7 +59748,7 @@ _converse_headless_converse_core__WEBPACK_IMPORTED_MODULE_5__["default"].plugins
           ev.preventDefault();
         }
 
-        const icon_el = ev.target.querySelector('.fa');
+        const icon_el = ev.target.matches('.fa') ? ev.target : ev.target.querySelector('.fa');
 
         if (_.includes(icon_el.classList, "fa-caret-down")) {
           this.model.save({

src/converse-bookmarks.js

@@ -507,7 +507,7 @@ converse.plugins.add('converse-bookmarks', {
 
             toggleBookmarksList (ev) {
                 if (ev && ev.preventDefault) { ev.preventDefault(); }
-                const icon_el = ev.target.querySelector('.fa');
+                const icon_el = ev.target.matches('.fa') ? ev.target : ev.target.querySelector('.fa');
                 if (u.hasClass('fa-caret-down', icon_el)) {
                     u.slideIn(this.el.querySelector('.bookmarks'));
                     this.list_model.save({'toggle-state': _converse.CLOSED});

src/converse-roomslist.js

@@ -245,7 +245,7 @@ converse.plugins.add('converse-roomslist', {
 
             toggleRoomsList (ev) {
                 if (ev && ev.preventDefault) { ev.preventDefault(); }
-                const icon_el = ev.target.querySelector('.fa');
+                const icon_el = ev.target.matches('.fa') ? ev.target : ev.target.querySelector('.fa');
                 if (icon_el.classList.contains("fa-caret-down")) {
                     u.slideIn(this.el.querySelector('.open-rooms-list')).then(() => {
                         this.list_model.save({'toggle-state': _converse.CLOSED});

src/converse-rosterview.js

@@ -688,7 +688,7 @@ converse.plugins.add('converse-rosterview', {
 
             async toggle (ev) {
                 if (ev && ev.preventDefault) { ev.preventDefault(); }
-                const icon_el = ev.target.querySelector('.fa');
+                const icon_el = ev.target.matches('.fa') ? ev.target : ev.target.querySelector('.fa');
                 if (_.includes(icon_el.classList, "fa-caret-down")) {
                     this.model.save({state: _converse.CLOSED});
                     await this.collapse();