Fixed bug #10892: Make it possible to style a hover color for pagetree/filetree items
authorSteffen Kamper <info@sk-typo3.de>
Tue, 21 Apr 2009 11:07:13 +0000 (11:07 +0000)
committerSteffen Kamper <info@sk-typo3.de>
Tue, 21 Apr 2009 11:07:13 +0000 (11:07 +0000)
git-svn-id: https://svn.typo3.org/TYPO3v4/Core/trunk@5342 709f56b5-9817-0410-a4d7-c38de5d9e867

ChangeLog
typo3/class.filelistfoldertree.php
typo3/class.webpagetree.php
typo3/js/tree.js

index 4887e90..33244e3 100755 (executable)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2009-04-21  Steffen Kamper  <info@sk-typo3.de>
+
+       * Fixed bug #10892: Make it possible to style a hover color for pagetree/filetree items
+
 2009-04-21  Patrick Broens  <patrick@patrickbroens.nl>
 
        * Fixed bug #10939: Wrong nesting of <ul> <li> tags in expanded mount root in module file > filelist
index 0529716..dcc1ed0 100644 (file)
@@ -174,7 +174,7 @@ class filelistFolderTree extends t3lib_folderTree {
                        // if this item is the start of a new level,
                        // then a new level <ul> is needed, but not in ajax mode
                        if($v['isFirst'] && !($doCollapse) && !($doExpand && $expandedFolderUid == $uid))       {
-                               $itemHTML = "<ul>\n";
+                               $itemHTML = "</div><ul>\n";
                        }
 
                        // add CSS classes to the list item
@@ -182,12 +182,12 @@ class filelistFolderTree extends t3lib_folderTree {
                        if($v['isLast']) { $classAttr = ($classAttr) ? ' last'  : 'last';        }
 
                        $itemHTML .='
-                               <li id="'.$idAttr.'"'.($classAttr ? ' class="'.$classAttr.'"' : '').'>'.
+                               <li id="'.$idAttr.'"'.($classAttr ? ' class="'.$classAttr.'"' : '').'><div class="treeLinkItem">'.
                                        $v['HTML'].
                                        $this->wrapTitle($this->getTitleStr($v['row'],$titleLen),$v['row'],$v['bank']);
 
 
-                       if(!$v['hasSub']) { $itemHTML .= "</li>\n"; }
+                       if(!$v['hasSub']) { $itemHTML .= "</div></li>\n"; }
 
                        // we have to remember if this is the last one
                        // on level X so the last child on level X+1 closes the <ul>-tag
index 27a9b7f..0414246 100644 (file)
@@ -216,7 +216,7 @@ class webPageTree extends t3lib_browseTree {
                        // if this item is the start of a new level,
                        // then a new level <ul> is needed, but not in ajax mode
                        if($v['isFirst'] && !($doCollapse) && !($doExpand && $expandedPageUid == $uid)) {
-                               $itemHTML = '<ul>';
+                               $itemHTML = '</div><ul>';
                        }
 
                        // add CSS classes to the list item
@@ -224,12 +224,12 @@ class webPageTree extends t3lib_browseTree {
                        if($v['isLast']) { $classAttr .= ($classAttr) ? ' last' : 'last';        }
 
                        $itemHTML .='
-                               <li id="'.$idAttr.'"'.($classAttr ? ' class="'.$classAttr.'"' : '').'>'.
+                               <li id="'.$idAttr.'"'.($classAttr ? ' class="'.$classAttr.'"' : '').'><div class="treeLinkItem">'.
                                        $v['HTML'].
                                        $this->wrapTitle($this->getTitleStr($v['row'],$titleLen),$v['row'],$v['bank'])."\n";
 
 
-                       if(!$v['hasSub']) { $itemHTML .= '</li>'; }
+                       if(!$v['hasSub']) { $itemHTML .= '</div></li>'; }
 
                        // we have to remember if this is the last one
                        // on level X so the last child on level X+1 closes the <ul>-tag
index 35d061c..d0d71bd 100644 (file)
@@ -49,8 +49,8 @@ var Tree = {
                // immediately collapse the subtree and change the plus to a minus when collapsing
                // without waiting for the response
                if (!isExpand) {
-                       var ul = obj.parentNode.getElementsByTagName('ul')[0];
-                       obj.parentNode.removeChild(ul); // no remove() directly because of IE 5.5
+                       var ul = obj.parentNode.parentNode.getElementsByTagName('ul')[0];
+                       obj.parentNode.parentNode.removeChild(ul); // no remove() directly because of IE 5.5
                        var pm = Selector.findChildElements(obj.parentNode, ['.pm'])[0]; // Getting pm object by CSS selector (because document.getElementsByClassName() doesn't seem to work on Konqueror)
                        if (pm) {
                                pm.onclick = null;