[fastedit] Javascript-Problem in FF3.6

[wdburgdorf]

Hallo,

ich weiss nicht, ob das FF3.6-spezifisch ist, oder TL2.8.3, oder nur bei mir ... jedenfalls geht unter diesen Umständen fastedit 0.7.1 bei mir nicht richtig. Der Bleistift erscheint nicht bei mouseover.

Nun habe ich herausgefunden, dass es an der Interpretation von firstChild liegt. FF zählt auch whitespace, IE nicht. Also geht es im IE, aber FF macht nicht den Bleistift 'visible', sondern den Zeilenumbruch.
Im Quelltext (FastEdit.php) ist der zugehörige Code in einer Zeile ohne whitespace zwischen den Elementen, ausgegeben wird er aber mit Zeilenumbruch zwischen divs.

Meine Lösung (in FastEdit.php, Zeile 96):
Original

return '<div onmouseover="this.style.background=\'#EBFDD7\'; this.firstChild.nextSibling.style.visibility=\'visible\'"  onmouseout="this.style.background=\'transparent\'; this.firstChild.nextSibling.style.visibility=\'hidden\'"><div style="position: absolute; border: 1px solid #FF0000; background-color: #FFFFFF; z-index:998; padding: 2px; padding-top: 4px; visibility: hidden"><a href="typolight/main.php?do='.$do.'&id='.$id.'" onclick="window.open(this.href, \'backend\').focus(); this.blur(); return false"><img src="system/themes/default/images/edit.gif" alt="" /></a></div>'.$strBuffer.'</div>';

Fälschung

return '<div onmouseover="this.style.background=\'#EBFDD7\'; this.getElementsByTagName(\'div\')[0].style.visibility=\'visible\'"  onmouseout="this.style.background=\'transparent\'; this.getElementsByTagName(\'div\')[0].style.visibility=\'hidden\'"><div style="position: absolute; border: 1px solid #FF0000; background-color: #FFFFFF; z-index:998; padding: 2px; padding-top: 4px; visibility: hidden"><a href="typolight/main.php?do='.$do.'&id='.$id.'" onclick="window.open(this.href, \'backend\').focus(); this.blur(); return false"><img src="system/themes/default/images/edit.gif" alt="" /></a></div>'.$strBuffer.'</div>';

Das funktioniert mit FF und IE (woanders nicht getestet).

Vielleicht könnte das (oder eine bessere Lösung) in die nächste Version eingehen?

Gruss,
Ralf.

[blueamerican]

ja das könnte die lösung sein, aber:

das kann man oder sollte man in einer extension schon eleganter machen, zb. mit mootols oder jquery... ohne js code im html...

vg blueamerican

[andreas.schempp]

Der Javascript-Code wird wohl nicht aus dem HTML entfernt. Dazu lohnt sich einfach der mehraufwand nicht. Aber wie wärs mit getFirst() ?