From 0d66652e6e401991864fd180bc18d07ecb066656 Mon Sep 17 00:00:00 2001
From: Pascal Corpet <pascal.corpet@m4x.org>
Date: Sat, 29 May 2010 00:34:01 +0200
Subject: [PATCH] More HTML fixes

---
 plugins/function.display_education.php |  4 +++-
 templates/carnet/mescontacts.tpl       | 20 ++++++++------------
 templates/emails/alias.tpl             |  2 +-
 templates/emails/index.tpl             |  4 ++--
 templates/emails/redirect.tpl          |  4 ++--
 templates/platal/motdepasse.tpl        |  8 +++++---
 6 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/plugins/function.display_education.php b/plugins/function.display_education.php
index da0ea96..bbf4208 100644
--- a/plugins/function.display_education.php
+++ b/plugins/function.display_education.php
@@ -62,7 +62,9 @@ function display_education($name, $url, $degree, $grad_year, $field, $program, $
             $txt .= $name;
         }
     }
-    $txt .= "</span>";
+    if ($grad_year || $field || $program) {
+        $txt .= "</span>";
+    }
 
     return $txt;
 }
diff --git a/templates/carnet/mescontacts.tpl b/templates/carnet/mescontacts.tpl
index 64a068c..167bdcc 100644
--- a/templates/carnet/mescontacts.tpl
+++ b/templates/carnet/mescontacts.tpl
@@ -26,25 +26,23 @@
   Ma liste personnelle de contacts
 </h1>
 
-<p>
+<form id="add_user" action="carnet/contacts" method="post">
   <div style="float: right">
-  <form id="add_user" action="carnet/contacts" method="post">
     {xsrf_token_field}
-    <div>
     <input type="hidden" name="action" value="ajouter" />
     <input type="text" size="30" name="user" class="quick_search"
            value="ajouter prenom.nom"
            onfocus="if (this.value == 'ajouter prenom.nom') this.value=''; $('#add_button').show()"
            onblur="if (this.value == '') this.value='ajouter prenom.nom'"
-           size="20" maxlength="70"/>
-    <a id="add_button" href="" style="display: none" onclick="document.getElementById('add_user').submit(); return false;">
+           maxlength="70"/>
+    <a id="add_button" href="carnet/contacts/add" style="display: none" onclick="document.getElementById('add_user').submit(); return false;">
       {icon name=add title="Ajouter la personne"}
     </a>
-    </div>
-  </form>
   </div>
+<p>
   Ajouter à tes contacts&nbsp;:
 </p>
+</form>
 <p style="clear: both">
   Sur la page de résultats d'une recherche, tu peux ajouter un contact en cliquant sur {icon name=add}.
 </p>
@@ -75,18 +73,16 @@
   </li>
 </ul>
 
-<p>
+<form action="carnet/contacts/search#plset_content" method="get">
   <div style="float: right">
-    <form action="carnet/contacts/search#plset_content" method="get">
-      <div>
       <input type="text" size="30" name="quick" class="quick_search"
              value="{$smarty.request.quick|default:'recherche dans tes contacts'}"
              onfocus="if (this.value == 'recherche dans tes contacts') this.value=''"
              onblur="if (this.value == '') this.value='recherche dans tes contacts'"/>
       <a href="carnet/contacts">{icon name=cross title='Annuler la recherche'}</a>
-      </div>
-    </form>
   </div>
+</form>
+<p>
   Rechercher dans tes contacts&nbsp;:
 </p>
 
diff --git a/templates/emails/alias.tpl b/templates/emails/alias.tpl
index a6253ee..14e6cf6 100644
--- a/templates/emails/alias.tpl
+++ b/templates/emails/alias.tpl
@@ -103,7 +103,7 @@
                 <input type="checkbox" name="public" {if $mail_public}checked="checked"{/if} id="pub_add"/>
               </td>
               <td class="texte">
-                <label for="pub_add">adresse publique (apparaît sur ta fiche).
+                <label for="pub_add">adresse publique (apparaît sur ta fiche).</label>
               </td>
              </tr>
           </table>
diff --git a/templates/emails/index.tpl b/templates/emails/index.tpl
index a0d4555..0aad81e 100644
--- a/templates/emails/index.tpl
+++ b/templates/emails/index.tpl
@@ -55,7 +55,7 @@
     <br />
     <div>
     Tu dispose également de l'alias&nbsp;: <strong>{$melix}</strong>
-    (<a href="email/alias">changer ou supprimer mon alias melix</a>)
+    (<a href="emails/alias">changer ou supprimer mon alias melix</a>)
     </div>
     {/if}
   </div>
@@ -66,7 +66,7 @@
     {if !$melix}
     Tu peux ouvrir en supplément une adresse synonyme de ton adresse @{#globals.mail.domain#},
     sur les domaines @{#globals.mail.alias_dom#} et @{#globals.mail.alias_dom2#} (melix = Mél X).<br />
-    <div class="center"><a href="email/alias">Créer un alias melix</a></div>
+    <div class="center"><a href="emails/alias">Créer un alias melix</a></div>
     {/if}
   </div>
 </fieldset>
diff --git a/templates/emails/redirect.tpl b/templates/emails/redirect.tpl
index 9ae1002..90033fd 100644
--- a/templates/emails/redirect.tpl
+++ b/templates/emails/redirect.tpl
@@ -180,7 +180,7 @@
       <form action="emails/redirect" method="post">
         {cycle values="pair,impair" assign=class_combobox}
         {include file="include/emails.combobox.tpl" name="email" val=$email class=$class_combobox error=$error_email i="0"}
-        <tr class=$class_combobox><td colspan="4"><div>
+        <tr class="{$class_combobox}"><td colspan="4"><div>
           <input type="submit" value="ajouter" name="emailop" />
           {xsrf_token_field}
         </div></td></tr>
@@ -262,7 +262,7 @@
   tes emails, Google Calendar, Google Docs&hellip;) sur une adresse polytechnique.org.
 </p>
 
-<table class="bicol" summary="Compte de stockage" id="googleapps">
+<table class="bicol" summary="Compte de stockage">
   <col width="55%" />
   <col width="45%" />
   <tr>
diff --git a/templates/platal/motdepasse.tpl b/templates/platal/motdepasse.tpl
index 79d9faa..0343c03 100644
--- a/templates/platal/motdepasse.tpl
+++ b/templates/platal/motdepasse.tpl
@@ -71,16 +71,18 @@
     </table>
   </form>
   <form action="{$smarty.server.REQUEST_URI}" method="post" id="changepass2">
-  <p>
+  <div>
   {xsrf_token_field}
   <input type="hidden" name="response2"  value="" />
-  </p>
+  </div>
   </form>
 </fieldset>
 
 <p>
   Note bien qu'il s'agit là du mot de passe te permettant de t'authentifier sur le site {#globals.core.sitename#}&nbsp;;
-  le mot de passe te permettant d'utiliser le serveur <a href="./Xorg/SMTPSécurisé">SMTP</a> et <a href="Xorg/NNTPSécurisé">NNTP</a>
+  le mot de passe te permettant d'utiliser le serveur <a
+  href="{"./Xorg/SMTPSécurisé"|urlencode}">SMTP</a>
+  et <a href="{"Xorg/NNTPSécurisé"|urlencode}">NNTP</a>
   de {#globals.core.sitename#} (si tu as <a href="./password/smtp">activé l'accès SMTP et NNTP</a>)
   est indépendant de celui-ci et tu peux le modifier <a href="./password/smtp">ici</a>.
 </p>
-- 
2.1.4