[BUGFIX] Showing/Hiding toolbars of CE's in PageModule broken
authorSteffen Ritter <info@rs-websystems.de>
Sun, 18 Dec 2011 16:19:31 +0000 (17:19 +0100)
committerGeorg Ringer <mail@ringerge.org>
Mon, 19 Dec 2011 08:34:25 +0000 (09:34 +0100)
The JavaScript uses addClass/removeClass to show hide the tool-
bar for each Content-Element even for browsers which do not
support :hover on div-Containers.

For ExtJS 4 the functions have been renamed to addCls/removeCls
which needs to be done in the pageModule script accordingly.

Change-Id: I3a7f9c78d90a9cd9c09c26be17392b39caef353a
Fixes: #32669
Releases: 4.7
Reviewed-on: http://review.typo3.org/7366
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Ralf Merz
Tested-by: Ralf Merz
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
typo3/sysext/cms/layout/js/typo3pageModule.js

index 00926cf..a0cd2a8 100644 (file)
@@ -35,7 +35,7 @@ TYPO3.Components.PageModule = {
        },
 
        /**
-        * This method is used to bind the higlighting function "setActive"
+        * This method is used to bind the highlighting function "setActive"
         * to the mouseenter event and the "setInactive" to the mouseleave event.
         */
        enableHighlighting: function() {
@@ -45,7 +45,7 @@ TYPO3.Components.PageModule = {
        },
 
        /**
-        * This method is used to unbind the higlighting function "setActive"
+        * This method is used to unbind the highlighting function "setActive"
         * from the mouseenter event and the "setInactive" from the mouseleave event.
         */
        disableHighlighting: function() {
@@ -59,7 +59,7 @@ TYPO3.Components.PageModule = {
         * user hovers the a content element.
         */
        setActive: function(event, target) {
-               Ext.get(target).findParent('div.t3-page-ce', null, true).addClass('active');
+               Ext.get(target).up('div.t3-page-ce').addCls('active');
        },
 
        /**
@@ -68,8 +68,7 @@ TYPO3.Components.PageModule = {
         * content element.
         */
        setInactive: function(event, target) {
-               Ext.get(target).findParent('div.t3-page-ce', null, true).removeClass('active');
-
+               Ext.get(target).up('div.t3-page-ce').removeCls('active');
        }
 }