Discussion :

[Are-no]

La fonction :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Une idée sur le problème ?

[Auteur]

bonjour,

tu as confondu la syntaxe CSS et la syntaxe javascript :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
div.style.marginLeft = width;

par ailleurs appeler sa variable "div" n'est pas très judicieux. Appelle la "monDiv" par exemple.

[FremyCompany]

bonjour,

tu as confondu la syntaxe CSS et la syntaxe javascript :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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...

[Auteur]

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...

[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]

[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

[Are-no]

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

JS :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

PHP :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.