Explorar el Código

Override `isHidden` in converse-minimize

Otherwise `isHidden` might return true because `minimized` is set to
`true` but `converse-minimize` is not enabled.
JC Brand hace 4 años
padre
commit
9348ac80b5
Se han modificado 2 ficheros con 6 adiciones y 4 borrados
  1. 4 0
      src/converse-minimize.js
  2. 2 4
      src/headless/converse-chat.js

+ 4 - 0
src/converse-minimize.js

@@ -71,6 +71,10 @@ converse.plugins.add('converse-minimize', {
                     return this;
                     return this;
                 }
                 }
                 return this.__super__.maybeShow.apply(this, arguments);
                 return this.__super__.maybeShow.apply(this, arguments);
+            },
+
+            isHidden () {
+                return this.__super__.isHidden.call(this) || this.get('minimized');
             }
             }
         },
         },
 
 

+ 2 - 4
src/headless/converse-chat.js

@@ -1142,10 +1142,8 @@ converse.plugins.add('converse-chat', {
              * @returns {boolean}
              * @returns {boolean}
              */
              */
             isHidden () {
             isHidden () {
-                return this.get('hidden') ||
-                    this.get('minimized') ||
-                    this.isScrolledUp() ||
-                    _converse.windowState === 'hidden';
+                // Note: This methods gets overridden by converse-minimize
+                return this.get('hidden') || this.isScrolledUp() || _converse.windowState === 'hidden';
             },
             },
 
 
             /**
             /**