[erledigt] Greasemonkey: Links anhängen, wie?

Bazon Bloch

Hallo allerseits!

Habe zur Zeit folgendes Problem:
Ich würde auf einigen Seiten direkt hinter vorhandenen Links Variationen dieser Links (nämlich anhängen von "#end") hinzufügen.

Die Links bekomme ich, aber sie landen nicht an der richtigen Stelle!
(Sondern total konfus an allen möglichen Stellen...)

Wie mache ich es also richtig?

Danke schonmal!

Anhang: So mache ich es bisher falsch:

Code

var links, laenge;


links = document.getElementsByTagName('a');
laenge = links.length;
var endlink = "";
var endlinks = new Array(laenge);    
for (var i = 0; i kleiner laenge ; i++)   // kleiner-Zeichen führt zu falscher Darstellung
{
   endlinks[i] = document.createElement("a");
   endlink = String(links[i]);
   if (endlink.search(/daswonachichsuche/) > 0) 
   {
       endlink = endlink + "#end";
       endlinks[i].innerHTML =  '<a href="' + endlink + '"> end</a>';
       document.links[i].parentNode.appendChild(endlinks[i]);
    }
}

Alles anzeigen

boardraider

Machs dir einfacher:

Code

var links = document.getElementsByTagName('a');
for (var i = 0; i < links.length; i++)
{
   if (links[i].href.search(/daswonachichsuche/) > 0)
   {
       links[i].href = links[i].href + "#end";
       links[i].innerHTML =  'end';
    } 
}

boardraider

Was du oben übrigens falsch gemacht hast, ist die letzte Zeile. Da greifst du auf die parentNode des Links zu und fügst dann dort einen neuen Link als Element an. Das ersetzt natürlich nicht und der Link rutscht unter Umständen an eine andere Stelle.

Hast du folgende Schachtelung:

Code

div
    link
    table
    div

Dann hast du danach:

Code

div
    link
    table
    div
    link

Der Link rutscht also unter table und div, daher wohl die Verschiebungen bei dir. Ersetzen geht übrigens mit replaceChild.

Bazon Bloch

Danke, ABER:

So werden ja alle Links ersetzt!
Ich möchte, dass diese neuen Links direkt neben die alten hinzugefügt werden, soll also praktisch so aussehen:

alt:
irgendein Link

neu:
irgendein Link end

Ist hier zwar sinnlos, aber prizipiell eben...

Bazon Bloch

Nachtrag:
Hatte nur den ersten gelesen, aber wie schon gesagt:
Mir geht es ja gerade darum, neue Links zu erzeugen....

Konkreter:
alte Struktur:

Code

div
    link
    text

erwünschte neue Struktur:

Code

div
    link
    link (der neue!)
    text

boardraider

Oh, ok, das habe ich falsch verstanden, dass das deine Absicht war. Dann nutz das:

http://de.selfhtml.org/javascript/obj…m#insert_before

Finde den nextSibling zu deinem Link und nutze die Funktion.

boardraider

Freihändig ohne Test:

Code

if (links[i].href.search(/daswonachichsuche/) > 0)
{     
       newLink = document.createElement("a");
       newLink.href= "#end";
       newLink.innerHTML = "end";
       links[i].parentNode.insertBefore(newLink, links[i].nextSibling);
}

Eventuell musst du noch prüfen, was passiert, wenn es keinen nextSibling gibt.

Bazon Bloch

Cool, danke, fast!
Der Link war so noch nicht korrekt, stattdessen:

Code

if (links[i].href.search(/daswonachichsuche/) > 0)
{     
       newLink = document.createElement("a");
       newLink.href= String(links[i]) + "#end";
       newLink.innerHTML = "end";
       links[i].parentNode.insertBefore(newLink, links[i].nextSibling);
}

Komisch ist nur, dass das ganze jetzt nur bis zur Hälfte der Links durchgeführt wird, kann das sein, dass die links[i] dynamisch abgepasst wird?

Ich probiers mal gleich mit der doppelten Länge....

Bazon Bloch

jepp, mit 2 * laenge in der for Schleife bekommt jeder Link ein end hintendran...
...komisch.

Aber was solls, funktioniert!

Danke!!!

boardraider

Der Link ist schon korrekt, es reicht die Angabe des Anchors. Der Browser vervollständigt das automatisch. Da ich die Seite nicht kenne, kann ich nicht sagen, ob dort dynamisch neue Links eingebaut werden. Bei einer statischen Anzahl an Links klappt es, ich habe es gerade hier getestet. Der Code komplett wäre:

Code

var links = document.getElementsByTagName('a');
for (var i = 0; i < links.length; i++)
{
    if (links[i].href.search(/xxx/) > 0)
    {     
       newLink = document.createElement("a");
       newLink.href= "#end";
       newLink.innerHTML = "end";
       links[i].parentNode.insertBefore(newLink, links[i].nextSibling);
    } 
}

Alles anzeigen

Bazon Bloch

Zitat von boardraider

Der Link ist schon korrekt, es reicht die Angabe des Anchors. Der Browser vervollständigt das automatisch.

Aber doch nur für die Seite, auf der man sich befindet!
Es geht ja hier um Links zu anderen Seiten....

Noch eine Sache:
Sooo toll funktioniert die #end Anchor Geschichte leider nicht:
Ich lande dann nicht ganz unten, sonder ein bisschen weiter oben.

Ich vermute, dass durch die ganze Seitenmodifikation (noch einiges mehr außer den Endlinks...) Firefox mit seienn vertikalen Positionen durcheinanderkommt...
...da gibt es wahrscheinlich aber keinen Trick um zu bewirken, dass man doch am Ende ankommt, oder?

boardraider

Da zeigt es sich wieder, wie praktisch es ist, wenn man über konkrete Seiten spricht. Ich ging davon aus, dass du einen Endanker auf der selben Seite anspringen willst. Wenn nicht, muss es natürlich so sein wie von dir beschrieben.

Da ich die Seite nicht kenne, auf dem sich der Anker #end befindet, kann ich nichts dazu sagen.

Bazon Bloch

Hehe, da muss ich Dir aber widersprechen!

Um einen Endanker zur aktuellen Siete zu bekommen bräuchte ich doch keine For-Schleife und müsste sowieso nicht groß nach Links suchen.

Ja, ich weiß natürlich, dass sich das im konkreten Falle immer besser nachvollziehen lässt, aber in diesem Fall ist das leider nicht möglich, es handelt sich um eine nicht öffentlich zugängliche Seite.

Aber ist ja egal, abgesehen von der nicht ganz perfekten Findung des Endankers beim Laden von einer anderen Seite aus (wenn man sich bereits auf der Seite befindet wird der Endanker richtig gefunden...!) ist ja alles paletti...

boardraider

Retourkutsche:
Wenn man bspw. über die Links "Nach oben" hier im Forum iteriert, braucht man i.d.R. schon eine Schleife. Daneben könnte man (so in meinem Test geschehen) einen Link "Nach unten" setzen. Um aus der Masse der Links auf der Seite genau diese zu finden müsste man auch ein Kriterium anlegen, wobei ich das in der Regel mit XPath mache.