EZP-27246: Content type Description field appears on content-in-the-f…

…ly, even if the description is empty
ezsystems · Apr 13, 2017 · 6f96360 · 6f96360
1 parent 0628a14
commit 6f96360
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/bundle/Resources/public/js/views/cof-contenttypeselectorview.js b/bundle/Resources/public/js/views/cof-contenttypeselectorview.js
@@ -119,17 +119,23 @@ YUI.add('cof-contenttypeselectorview', function (Y) {
          * @param show {Boolean} should show the tooltip?
          */
         _toggleTooltip: function (show) {
-            var methodName = show ? 'removeClass' : 'addClass',
-                container = this.get('container'),
+            var container = this.get('container'),
                 selectedItem = container.one(SELECTOR_ITEM_SELECTED),
-                tooltip = container.one(SELECTOR_TOOLTIP);
+                tooltip = container.one(SELECTOR_TOOLTIP),
+                methodName,
+                description;
 
-            if (selectedItem) {
-                tooltip.setHTML(selectedItem.getAttribute(ATTR_DESCRIPTION));
-
-                tooltip[methodName](CLASS_HIDDEN);
-                tooltip.removeClass(CLASS_TOOLTIP_ERROR);
+            if (!selectedItem) {
+                return;
             }
+
+            description = selectedItem.getAttribute(ATTR_DESCRIPTION);
+            methodName = show && !!description ? 'removeClass' : 'addClass';
+
+            tooltip.setHTML(description);
+
+            tooltip[methodName](CLASS_HIDDEN);
+            tooltip.removeClass(CLASS_TOOLTIP_ERROR);
         },
 
         /**