Salut, d'après ce que j'ai pu voir, tu as fait un mixe entre ta solution de départ (dans laquelle tu passe à ta fonction l'id de l'élément pour touver le contrôleur) et la solution de Eldebaran (qui consiste à passer l'objet html directement)
voilà une version qui marche chez moi
<html>
<head>
<script type="text/javascript">
function verifchamps()
{
if(document.getElementById('dest').value == "")
{
alert('PROUT -> dest');
return false;
}
if(document.getElementById('objet').value == "")
{
alert('PROUT -> objet');
return false;
}
if(document.getElementById('style').value == "")
{
alert('PROUT -> style');
return false;
}
if (document.getElementById('contenu').value == "")
{
alert('PROUT -> contenu');
return false;
}
if (document.getElementById('radio_1').checked && document.getElementById('after').value == "")
{
alert('PROUT -> after');
return false;
}
return true;
}
function Grise(Controleur, Controle, Masquer)
{
var objControleur = document.getElementById(Controleur);
var objControle = document.getElementById(Controle);
if(Masquer=='true')
objControle.style.visibility=(objControleur.checked)?'visible':'hidden';
else
objControle.disabled=(!objControleur.checked)?false:true;
return true;
}
</script>
</head>
<body onload="Grise('radio_2', 'input_1', false);">
<p align="center"><br /><big><b>MAIL</b></big><br /><br /><small><i>FILLFIELDS</i></small></p>
<form id="form1" name="form1" method="post" action="index.php?file=Mailing&op=do_add" onSubmit="return verifchamps();">
<table width="100%" border="0" cellspacing="3" cellpadding="7">
<tr>
<td width="40%"><div align="right">DEST</div></td>
<td width="3%"><img src="modules/Mailing/images/help.gif" border="0" alt="INFOR" /></td>
<td width="3%">:</td>
<td width="57%"><input type="text" name="dest" /></td>
</tr>
<tr>
<td><div align="right">OBJET</div></td>
<td><img src="modules/Mailing/images/help.gif" border="0" alt="INFOR" /></td>
<td>:</td>
<td><input type="text" name="objet" /></td>
</tr>
<tr>
<td><div align="right">TYPETEXT</div></td>
<td><img src="modules/Mailing/images/help.gif" border="0" alt="INFOR" /></td>
<td>:</td>
<td>
<select name="style">
<option value="1" selected="selected">FORMATTEXT</option>
<option value="2">HTML</option>
</select>
</td>
</tr>
<tr>
<td><div align="right">CONTENU</div></td>
<td><img src="modules/Mailing/images/help.gif" border="0" alt="INFOR" /></td>
<td>:</td>
<td>
<textarea name="contenu" cols="60" rows="8"></textarea>
</td>
</tr>
<tr>
<td><div align="right">TYPESEND</div></td>
<td><img src="modules/Mailing/images/help.gif" border="0" alt="INFOR" /></td>
<td>:</td>
<td>
<input type="radio" id="radio_1" name="radios" onClick="Grise('radio_1', 'select_1', false);Grise('radio_1', 'select_2', false);Grise('radio_1', 'select_3', false);Grise('radio_1', 'select_4', false);Grise('radio_1', 'div_1', false);Grise('radio_2', 'input_1', false);Grise('radio_2', 'div_3', false);"> <label for="radio_1">SELFIXE</label>
<input type="radio" id="radio_2" name="radios" onClick="Grise('radio_1', 'select_1', false);Grise('radio_1', 'select_2', false);Grise('radio_1', 'select_3', false);Grise('radio_1', 'select_4', false);Grise('radio_1', 'div_1', false);Grise('radio_2', 'input_1', false);Grise('radio_2', 'div_3', false);" checked="checked"> <label for="radio_2">SELINSCR</label>
</td>
</tr>
<tr>
<td>
<div align="right">Choix de la date</div>
</td>
<td><img src="modules/Mailing/images/help.gif" border="0" alt="" /></td>
<td>:</td>
<td>
<div id="div_1">
Jour :
<select name="jour" id="select_1">
<option value="1" selected="selected">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
<option value="5">5</option>
<option value="6">6</option>
<option value="7">7</option>
<option value="8">8</option>
<option value="9">9</option>
<option value="10">10</option>
<option value="11">11</option>
<option value="12">12</option>
<option value="13">13</option>
<option value="14">14</option>
<option value="15">15</option>
<option value="16">16</option>
<option value="17">17</option>
<option value="18">18</option>
<option value="19">19</option>
<option value="20">20</option>
<option value="21">21</option>
<option value="22">22</option>
<option value="23">23</option>
<option value="24">24</option>
<option value="25">25</option>
<option value="26">26</option>
<option value="27">27</option>
<option value="28">28</option>
<option value="29">29</option>
<option value="30">30</option>
<option value="31">31</option>
</select>
Mois :
<select name="mois" id="select_2">
<option value="1">JAN</option>
<option value="2">FEB</option>
<option value="3">MAR</option>
<option value="4">APR</option>
<option value="5">MAY</option>
<option value="6">JUN</option>
<option value="7">JUL</option>
<option value="8">AUG</option>
<option value="9">SEP</option>
<option value="10">OCT</option>
<option value="11">NOV</option>
<option value="12">DEC</option>
</select>
Année :
<select name="annee" id="select_3">
<option value="2006" selected="selected">2006</option>
<option value="2007">2007</option>
<option value="2008">2008</option>
<option value="2009">2009</option>
<option value="2010">2010</option>
<option value="2011">2011</option>
<option value="2012">2012</option>
<option value="2013">2013</option>
<option value="2014">2014</option>
<option value="2015">2015</option>
<option value="2016">2016</option>
<option value="2017">2017</option>
<option value="2018">2018</option>
<option value="2019">2019</option>
<option value="2020">2020</option>
</select>
</div>
</td>
</tr>
<tr id="tr_3">
<td><div align="right">ANNUALRENEW</div></td>
<td><img src="modules/Mailing/images/help.gif" border="0" alt="INFOR" /></td>
<td>:</td>
<td>
<select name="renew" id="select_4">
<option value="1">YES</option>
<option value="2">NO</option>
</select>
</td>
</tr>
<tr>
<td><div align="right">SENDMAIL</div></td>
<td><img src="modules/Mailing/images/help.gif" border="0" alt="INFOR" /></td>
<td>:</td>
<td><div id="div_3"><input name="after" type="text" size="2" maxlength="3" id="input_1" /> DAYSAFTER</div></td>
</tr>
<tr>
<td> </td>
<td> </td>
<td> </td>
<td> </td>
</tr>
<tr>
<td colspan="4">
<div align="center"><input type="submit" name="Submit" value="RECORD" /></div>
</td>
</tr>
</table>
</form>
</body>
</html>
Sinon, au niveau de la structure même de ton html, il y pas mal de choses à redire:
1. tu devrais utilisé la balise label pour t'es intitulé example:
<p>
<label for="id_de_l_input">Mon label</label>
<input name="nom_input" id="id_de_l_input" value="valeur_de l_input" type="text">
</p>
2. je pense que tu peux alléger le poid de la page en évitant de systématiquement utilisé un tableau (il y a plein de tutos sur le site pour ça)
3.Il me semble que pour être valide, les attributs et les balises doivent être en MINUSCULE donc éviter "onSubmit" et utilisé "onsubmit" par exemple
4. l'utilisation des espace insécable " " n'est pas approprié pour placer des élément sur la page
5. ...
je vais pas tout dire, c'était juste pour te donner des pistes si tu cherche à t'améliorer et surtout, à rendre ton site accessible.
bon courage
Modifié par Francois44 (20 Dec 2006 - 14:26)
