Erreur CSS dans Javascript

Version imprimable

La fonction :

Code:

1
2
3
4
5
6
7
function popwin(html){
	div=document.getElementById("popwin");
	div.innerHTML = html ;
	width = math.ceil(div.style.width / 2);
	div.style.margin-left = width;
	div.style.visibility = 'visible';
}

l'erreur donné :

Code:

1
2
3
Erreur : invalid assignment left-hand side
Code source :
	div.style.margin-left = width;

Une idée sur le problème ?

15/08/2006, 11h51
Auteur
bonjour,

tu as confondu la syntaxe CSS et la syntaxe javascript :
Code:

1 2 div.style.marginLeft = width;
par ailleurs appeler sa variable "div" n'est pas très judicieux. Appelle la "monDiv" par exemple.
15/08/2006, 20h24
FremyCompany
Citation:
Envoyé par Auteur

bonjour,

tu as confondu la syntaxe CSS et la syntaxe javascript :

Code:

1 2 div.style.marginLeft = width;

par ailleurs appeler sa variable "div" n'est pas très judicieux. Appelle la "monDiv" par exemple.
lol, div c'est très bien comme nom de variable ;) Notez que moi j'utilise divContent dans ce genre de cas (lol, je sais tlm s'en fout) ;)

Sinon tu as raison sur la manière de corriger le problème...
16/08/2006, 11h50
Auteur

Citation:

Envoyé par FremyCompany

lol, div c'est très bien comme nom de variable ;) Notez que moi j'utilise divContent dans ce genre de cas (lol, je sais tlm s'en fout) ;)

Sinon tu as raison sur la manière de corriger le problème...

C'est que je voulais dire : utiliser un nom de variable en lien avec le rôle ou le contenu du div.
divTitre, divSousTitre, divContenu, divMenu, divConteneur, divBlabla etc...:D

[edit]
Et non (presque) personne sans fout : je fais pareil ;). De toute façon, il faut organiser son code d'une manière ou d'une autre pour s'y retrouver.
[/edit]
16/08/2006, 14h37
Ntilou

salut,

déjà, je pense que ca serait mieux de mettre "var = ..." devant la déclaration de tes variables.

ensuite, tu n'as pas le droit de donner comme nom "width" a une variable, puisqu'il est déjà utilisé par javascript.

Tout ce qui est "height, width, div... et aussi test" comme noms de variables sont la cause d'erreurs dans javascript.

voila j'espère que c'est bon

Et bien comme vous êtes bien parti, je lache le code complet.

JS :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
function popwin(html){
	var popdiv = document.getElementById("popwin");
	var popdiv2 = document.getElementById("popwin2");
 
	var popdivh = 250;
	var popdivw = 350;
 
	popdiv.style.width = popdivw + "px";
	popdiv.style.height = popdivh + "px";
 
	popdiv.innerHTML = '<div style=\'text-align:right;border-bottom:1px solid #999999;margin-bottom:3px\'><span class=\'text\' style=\'cursor:pointer\'onclick="close_popwin()">Fermer [X]</span></div><div><span class=\'text\'>' + html + '<span></div>';
 
if (window.innerWidth)
	var popdiv2w = window.innerWidth;
else
	var popdiv2w = document.body.clientWidth;
 
if (window.innerHeight)
	var popdiv2h = window.innerHeight;
else
	var popdiv2h = screen.height;
 
	popdiv.style.top = "150px";
	popdiv.style.left = (popdiv2w - popdivw)/2 + "px";
 
	popdiv2.style.height = popdiv2h + "px";
	popdiv2.style.width = popdiv2w + "px";
 
	popdiv.style.visibility = 'visible';
	popdiv2.style.visibility = 'visible';
}
 
function close_popwin(){
	popdiv.style.visibility = 'hidden';
	popdiv2.style.visibility = 'hidden';
}

CSS :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#popwin {
	position : absolute;
    visibility : hidden;
	padding:3px;
    background-color : #FFFFFF;
    border : 1px solid #000000;
 
    color : #000000;
	font-family: Verdana, Arial, Helvetica, sans-serif;	
	font-weight:300;
	font-size:11px;
    }
 
#popwin2 {
    position : absolute;
    visibility : hidden;
	top:0;
	left:0;
    background-color : #000000;
	filter : Alpha(opacity=60);
	opacity : 0.60;
    }

HTML :

Code:

<div id='popwin2'></div><div id='popwin'></div>

PHP :

Code:

1
2
3
4
5
6
7
8
function popwin($html)
{
$traduction = array( "'" => "\\'", "\"" => "\\&quot;", "<" => "&lt;", ">" => "&gt;" );
$html = strtr($html , $traduction); 
 
$popwin = "onclick=\"popwin('" . $html . "')\"";
return $popwin;
}

PS : je suis preuneur en matière de critiques.