Seite 1 von 1
Spaw: target problem
Verfasst: Di 23. Mär 2004, 14:44
von emergence
ergänzung zu
->
http://www.contenido.de/forum/viewtopic ... ted+target
ich hab mir das jetzt angesehen und bin da auf etwas merkwürdiges gestossen...
der fehler läßt sich nur reproduzieren wenn mittels maus zwei wörter und ein leerzeichen ausgewählt wurden... mittels tastertur funktioniert es jedoch ganz wunderbar...
ähm auch nicht schlecht was der internet explorer da wieder zum besten gibt...
Verfasst: Di 23. Mär 2004, 16:32
von emergence
okay ich hab nen bugfix für den schwachsinn gefunden...
mit bitte um feedback...
in der contenido/external/wysiwyg/spaw/class/script.js.php
bei
function SPAW_hyperlink_click(editor, sender)
Code: Alles auswählen
if(slink != null) {
if (slink) {
this[editor+'_rEdit'].document.execCommand('CreateLink',false,slink.Href); // Insert Url
if (!hyp.target) { // try to get link
var hyp = SPAW_getLink(editor);
}
if (hyp) {
if (slink.Target != "") { // if target available set it
hyp.target = slink.Target;
} else { // remove it
hyp.removeAttribute("target");
}
} else { // link capturing failed
if (slink.Target != "") { // only warn if target was selected
alert("Selected target could not be set. Select link once and try again.");
}
}
}
} else {
this[editor+'_rEdit'].document.execCommand('UnLink',false);
}
ersetzen durch
Code: Alles auswählen
if(slink != null) {
if (slink) {
// v2.0 version modified version by horwath@opensa.org
idstr = "556e697175657e537472696e67"; // new link creation ID, set HREF to this
this[editor+'_rEdit'].document.execCommand('CreateLink',false,idstr);
var oAnchors = this[editor+'_rEdit'].document.all.tags("A");
if (oAnchors != null) {
for (var i = oAnchors.length - 1; i >= 0; i--) {
if (oAnchors[i] == idstr) {
hyp = oAnchors[i];
break;
}
}
}
hyp.href = slink.Href; // set selected hyperlink
if (slink.Target != "") { // if target available set it
hyp.target = slink.Target;
} else { // remove it
hyp.removeAttribute("target");
}
}
} else {
this[editor+'_rEdit'].document.execCommand('UnLink',false);
}
anmerkung:
ein setzen des targets sollte jetzt immer möglich sein...
ich habs bis jetzt noch nicht geschafft die fehlermeldung zu reproduzieren...
Verfasst: Di 23. Mär 2004, 16:57
von emergence
eine kleine änderung hab ich bei dieser version noch ergänzt...
Code: Alles auswählen
if(slink != null) {
if (slink) {
// v2.1 version modified version by horwath@opensa.org
idstr = "556e697175657e537472696e67"; // new link creation ID, set HREF to this
this[editor+'_rEdit'].document.execCommand('CreateLink',false,idstr);
var oAnchors = this[editor+'_rEdit'].document.all.tags("A");
if (oAnchors != null) {
for (var i = oAnchors.length; i >= 0; i--) {
if (oAnchors[i] == idstr) {
hyp = oAnchors[i];
hyp.href = slink.Href; // set selected hyperlink
if (slink.Target != "") { // if target available set it
hyp.target = slink.Target;
} else { // remove it
hyp.removeAttribute("target");
}
}
}
}
}
} else {
this[editor+'_rEdit'].document.execCommand('UnLink',false);
}
sobald man versucht sagen wir mal ein bild und text einen link gleichzeitig zugeben ,würde bei der ersten version nur der erste gefundene string wieder ersetzt...
sollte mit dieser änderung nicht auftreten...
Verfasst: Mi 24. Mär 2004, 15:27
von emergence
hat das schon einer getestet ?
Verfasst: Mi 24. Mär 2004, 15:32
von timo
bisher noch keine Zeit gehabt, mach ich aber die Tage mal
Verfasst: Do 25. Mär 2004, 08:34
von Alex
Hi!
Danke für das Bugfixing. Ich werde nächste Woche Dienstag mich dran setzen und melde mich dann, ob der Fehler noch besteht.
THANKS
Alex
Sorry...
Verfasst: Mo 5. Apr 2004, 10:16
von Alex
Hi!
Sorry, das das so lange gedauert hat.
Aber es funktioniert. Ich habe es heute morgen eingespielt...
Vielen Dank
Alex...
Verfasst: Mo 5. Apr 2004, 13:34
von timo
ja, schaut gut aus. Habe es eingebaut.