Je n'aurais pas dû parler d'exemples PHP avant on dirait, puisque tout le problème vient du HTML utilisé de façon... bizarre.
On va oublier des évidences modernes pas toujours suivies comme "pas de style dans la balise" et plutôt parler de tout le reste :

* PHP boucle pour afficher une liste d'éléments ligne par ligne. En général on crée un tableau dont on boucle les lignes, mais ici non... on boucle en créant un tableau d'une ligne à chaque fois.
* À moins d'être un en-tête, on utilise du <td>, pas du <th>. Dans tous les vieux projets que j'ai maintenu, c'est la première fois que je vois cela.
* Et qu'est ce que ce <h2><br /></h2> ? Ne me dites pas que le contenu du h2 a été vidé, pourquoi garder le <br /> dedans ? à moins de... à moins d'utiliser une balise de titre à taille "suffisante" pour... créer un espacement suffisant... entre chaque "ligne" (techniquement, entre chaque tableau)
* Suivi d'un <p><br /></p> ? le bloc précédent ne faisant pas un espace assez gros, il fallait un léger ajout ??

(pour Folco : et HTML, tu connais ? <h2> est une balise de titre "de 2e importance". <br /> est un retour à la ligne. <p> est un bloc de paragraphe. <th> est une cellule d'en-tête de tableau, <td> une cellule de tableau)