onclick su file js esterno

[serialkiller]

rieccomi

sto cercando di eliminare alcuni eventi onclick da una pagina, per farlo, ho implementato una funzione che ho nel mio js esterno, ma mi manca un passagio, la funzione è questa:

function popW3C(){

// controllo su tag a e li

  if(!document.getElementsByTagName) return;
  var l=document.getElementsByTagName("a");
  var m=document.getElementsByTagName("li");

  for(i=0;i<l.length;i++){
	  
    if (l[i].className.match(/ext/) || l[i].className.match(/extNormal/)){
      l[i].target = "_blank";
    }
	
	if (l[i].id.match(/tessere/)){//menù tesseramento
      l[i].target = "_blank";
    }
	
	if (l[i].className.match(/popNew/)){//popup
	  window.open(l[i].href,'popup ' + i,'scrollbars=no,resizable=yes,width=764,height=550,status=no,location=no,toolbar=no'); return false;
    }
	
  }
}

il problema è nella parte in grassetto, giustamente, in questo modo, la popup, si apre all'evento onload, richiamato nella pagina.
Come posso evitare la cosa ?

Siccome sono limitato in javascript, esite la possibilità di assegnare una sorta di onclick ad ogni tag <a> con classe "popNew".

sono dislessico....spero sia comprensibile

[willybit]

Ciao serialkiller,

puoi impostare l'onclick del link in questo modo

l[i].onclick=Function("window.open('"+l[i].href+"','popup'+i,'scrollbars=no,resizable=yes,width=764,height=550,status=no,location=no,toolbar=no');return false");

[serialkiller]

Originariamente inviato da willybit
Ciao serialkiller,

puoi impostare l'onclick del link in questo modo

l[i].onclick=Function("window.open('"+l[i].href+"','popup'+i,'scrollbars=no,resizable=yes,width=764,height=550,status=no,location=no,toolbar=no');return false");

si si si...ci stavo arrivando.....ma

io ho questo, dentro al ciclo

	if (l[i].className.match(/popNew/)){//popup
	  l[i].onclick=function(){window.open(this.href, 'popup ' + i, "width=764, height=550");return(false)};
	}

solo che non capisco il perchè....ma se ho anche solo 2 classi "popNew", lui considera sempre e solo la prima

[willybit]

Originariamente inviato da serialkiller
si si si...ci stavo arrivando.....ma

io ho questo, dentro al ciclo

	if (l[i].className.match(/popNew/)){//popup
	  l[i].onclick=function(){window.open(this.href, 'popup ' + i, "width=764, height=550");return(false)};
	}

solo che non capisco il perchè....ma se ho anche solo 2 classi "popNew", lui considera sempre e solo la prima

come imposti l'onclick tu è sbagliato... il discorso "lui considera sempre e solo la prima" non l'ho capito :master:

[serialkiller]

intanto ciao, che prima nemmeno ti ho salutato

non avevo visto le differenze

visto che ci sei ho ancora 1 cosetta

	for(p=0;p<m.length;p++){
		
    if (m[p].id.match(/profilo/)){
	m[p].onclick=Function("ctrlMenu("+m[p].id+",0); return false");
    }
	
	}

poco più sotto ho questa parte di codice, sempre dentro alla medesima funzione, faccio un ciclo diverso, perchè in questo caso il controllo è sul selettore[*] e non <a>

sulla pagina l'attributo onclick è:

onclick="ctrlMenu(this,0); return false;"

cosa sbaglio qui ?

mi dice che profilo non è definito

[willybit]

cosa deve fare la funzione ctrlMenu? hai più tag[*] con id="profilo"?
non sapendo altro il massimo che ti posso consigliare è

m[p].onclick=Function("ctrlMenu(this,0); return false");

[serialkiller]

Originariamente inviato da willybit
cosa deve fare la funzione ctrlMenu? hai più tag[*] con id="profilo"?
non sapendo altro il massimo che ti posso consigliare è

m[p].onclick=Function("ctrlMenu(this,0); return false");

infatti ho appena risolto così

non ho capito perchè ma funziona

di id profilo c'è nè uno solo ovviamente

senti, ancora una cosa poi giuro che non rompo più

facendo così è vero funziona, ma credo di avere qualcosina da modificare, nei controlli, se non dico una cavolata match, controlla l'esistenza della tal stringa in una seconda stringa di paragone, cosa devo usare in modo che il controllo sia totale....mi spiego, se la classe è "pippo", non mi deve considerare anche "pippoPluto".

if (m[p].id.indexOf("profilo")>=0){...

ma credo sia simile a match

o semplicemente

if (m[p].id == "profilo"){...

[willybit]

il match come parametro vuole una regular expression
se vuoi trovare che sia esattamente "pippo" sarebbe
TuaStringa.match(/^pippo$/)
che equivale a
TuaStringa=='pippo'

[serialkiller]

niente, ho provato anche così

m[p].onclick=Function("ctrlMenu('"+m[p].id+"',0); return false;");

ma non funziona, sono costretto a cambiare i nomi id

perchè se ho un id="pippo" e un id="pippoDue", me li da buoni entrambi

[serialkiller]

Originariamente inviato da willybit
il match come parametro vuole una regular expression
se vuoi trovare che sia esattamente "pippo" sarebbe
TuaStringa.match(/^pippo$/)
che equivale a
TuaStringa=='pippo'

quindi così dovrebbe funzionare

    if (m[p].id == "profilo" || m[p].id == "settore" || m[p].id == "sala_stampa" || m[p].id == "link"){
	m[p].onclick=Function("ctrlMenu('"+m[p].id+"',0); return false;");
    }

ma non funziona