3 Bugs in messageBox.js.php

[stefan25376]

Hallo Leute,

Ich habe 3 Bugs in der Datei messageBox.js.php gefunden. Aufgetaucht und getestet habe ich diese in Firefox 2.0 auf Windows XP.

1. Wenn Contenido in einem Unterverzeichnis läuft bzw. in tieferen Unterverzeichnis-Strukturen werden die Graphiken in der Messagebox nicht richtig referenziert. Dies liegt daran, das die Messagebox ohne Angabe einer URI geöffnet wird. Daher nimmt der Browser an der Hostname sei die URI und referenziert die relativ referenzierten Graphiken falsch.

2. Dasselbe wie in 1. passiert daraufhin auch in der Funktion performAction(), da diese die Datei main.php auch relativ referenziert.

3. Im Firefox 2.0 funktioneren die 100% Angaben für die Tabelle in der Messagebox nicht. Dadurch, daß man die Messagebox-Größe nicht ändern kann, verschwinden die beiden Knöpfe für Abbrechen und Bestätigen im rechten Rand. Man kann nur noch blind mit Tabs auf die Knöpfe navigieren.

Ich habe bereits ein Bugfix geschrieben, daß hier heruntergeladen werden kann: http://www.handgestrickt.biz/bblog/file ... _patch.zip

Darin sind alle 3 Bugs wie folgt behoben:

1. Ich errechne die tatsächliche URI und benutze das base-Tag im head der messageBox, um Bilder korrekt zu referenzieren.

2. Ich errechne die tatsächliche URI und benutze diese in der Funktion performAction() vor main.php.

3. Für die Tabelle habe ich zwei neue Template-Platzhalter eingeführt, die mit den tatsächlichen Höhen- und Weitenangaben ersetzt werden.

[emergence]

das verhalten kann ich mit dem alten code nicht reproduzieren...

verschoben...

[stefan25376]

Hallo emergence,

Ich kann auch nur anbieten Screenshots zu senden, um das Verhalten zu zeigen. Teilweise wundert man sich schon, was bei manchen Leuten für exotische und unerwartete Bugs auftreten.

Habe das Problem mit den verschobenen Knöpfen heute auch in anderen Messageboxen beobachtet. (Beispielsweise im tinyMCE im Bilderdialog). Werde mal mehr testen und alle Messages durchchecken.

[emergence]

screenshots sind nicht notwendig...

die fixes sind meiner meinung nach unbedenklich... kann nicht schaden sie zu ergänzen...

[stefan25376]

Hallo emergence,

Hat sich erledigt. Habe alle Dateien, die messageBox verwenden durchgeschaut und es klappt überall. Im Bilddialog des tinyMCE hat das andere Ursachen. Das werde ich eventuell anderweitig beheben und ins tinyMCE-Forum posten.

[HerrB]

Hat sich erledigt

Ist das erledigt oder sind die Änderungen noch notwendig?

Gruß
HerrB

[emergence]

das mit WIDTH und HEIGHT halt ich für okay...

[HerrB]

WIDTH und HEIGHT eingebaut, Checkin.

Gruß
HerrB