Revenir en haut Aller en bas
Le Deal du moment : -34%
-34% LG OLED55B3 – TV OLED 4K 55″ 2023 ...
Voir le deal
919 €

1 résultat trouvé pour partenaire

par Cheshire Cat
le Lun 5 Sep 2016 - 12:46
 
Rechercher dans: Les pages d'accueil
Sujet: [Facile] Page d'accueil #5 - Batty
Réponses: 17
Vues: 2605

[Facile] Page d'accueil #5 - Batty

Coucou miss !

Globalement, c'est respecté je dirai au niveau du visuel. Tu as juste mis ta petite touche perso quoi ! :D

Du coup, ton code (c'est le plus important) :3
Je trouve que niveau contraintes, c'est respecté. Tu as fais la PA en tableau, tu as fusionné des cellules, arrondi les coins. Par contre, tu n'as pas utilisé de div pour les textes. Temps pis mais en soit c'est pas trop grave !

Du coup, dès le départ je remarque un oubli ici
Code:
<table id="tableau" border="1px" cellSpacing=12 width="650px">


Il manque des "", c'est quand même important, et en plus tu as mis une majuscule au S qui fausse encore cette partie du code. Sinon, moi j'aurai pas mis autant d'id dès le départ. Les id, perso, je les utilise pour rendre mon code plus "visible", dans le sens où je repère tout de suite à quoi correspond telle chose. C'est plus rapide pour corriger des bugs, etc.

Maintenant, je t'avoue ne pas trop comprendre cette partie
Code:
<th id="cellule" rowspan="20"><h2 class="texte">NOTRE STAFF </h2>
<img src="http://screenshots.en.sftcdn.net/en/scrn/69661000/69661923/despicable-me-minion-rush-44-100x100.png" id="image"/> <img src="http://screenshots.en.sftcdn.net/en/scrn/69661000/69661923/despicable-me-minion-rush-44-100x100.png" id="image"/> <img src="http://screenshots.en.sftcdn.net/en/scrn/69661000/69661923/despicable-me-minion-rush-44-100x100.png" id="image"/> <img src="http://screenshots.en.sftcdn.net/en/scrn/69661000/69661923/despicable-me-minion-rush-44-100x100.png" id="image"/> <img src="http://screenshots.en.sftcdn.net/en/scrn/69661000/69661923/despicable-me-minion-rush-44-100x100.png" id="image"/> <img src="http://screenshots.en.sftcdn.net/en/scrn/69661000/69661923/despicable-me-minion-rush-44-100x100.png" id="image"/>
<br> <h2 class="texte">NOUVEAUTÉS</h2>
      <h4 id="h4">XX.XX.XX NEWS<br>
      XX.XX.XX NEWS<br>
      XX.XX.XX NEWS<br>
      XX.XX.XX NEWS<br>
      XX.XX.XX NEWS<br>
      XX.XX.XX NEWS<br>
      XX.XX.XX NEWS<br>
      XX.XX.XX NEWS<br>
      XX.XX.XX NEWS</h4>
      </th>


J'utilisais pas trop les TH du temps où je codais en tableau et quand je les vois en template je remarque que c'est surtout utilisé pour les titres de colonne et non pas pour une colonne entière. A ta place, j'aurai mis un td et puis basta.

Sinon, un rowspan à 7 ça suffit, pourquoi mettre 20 ? T'as juste à compter les cellules tu sais x)

Je remarque que dans ton CSS, tu utilises plutôt les id comme tu devrais utiliser des class, ce serait plutôt l'inverse en fait. Sinon, sémantiquement parlant, un id est pour un élément unique tandis qu'une class est un élément qui se répète. Pour cette partie

Code:

      <tr>
<td id="lien"><a href="https://jsfiddle.net/">LIEN</a></td>
<td id="lien"><a href="https://jsfiddle.net/">LIEN</a></td>
<td id="lien"><a href="https://jsfiddle.net/">LIEN</a></td>
  </tr>


Tu aurais faut par exemple dans ta syntaxe. Ensuite, dans cette même partie, tu aurais pu simplement faire ceci

Code:
<tr>
<td>
<a href="https://jsfiddle.net/" class="lien">LIEN</a>
<a href="https://jsfiddle.net/" class="lien">LIEN</a>
<a href="https://jsfiddle.net/" class="lien">LIEN</a>
</td>
  </tr>


Ca évite des colspan que tu n'as pas vraiment l'air de trop maîtriser, surtout que tu as déjà fusionné des cellules juste avant donc te casse pas la tête non plus ^^

Par contre
Code:
<td id="partenaire" cellSpacing=5 cellPadding=5>


Je sais pas depuis quand on applique des cellspacing et cellpadding aux td mais c'est pas bon du tout. Ca sert à rien et en plus tu as encore fais des fautes de syntaxe pour tes attributs HTML. Garde seulement ton id à la limite, mais enlève le reste. Tu as déjà appliqué un cellspacing dans ta balise table en plus ^^

Si tu veux centrer tes images dans #partenaire, applique un text-align:center; qui va les centrer si c'est ce que tu souhaites. Oui c'est étrange mais c'est comme ça XD

Sinon ce que je trouve cool, c'est d'avoir utilisé un p'tit peu les balises de titre. C'est une bonne initiative ici :)

Il te reste pas mal de trucs à revoir cependant, relis bien les tutoriels sur les tableaux et utilise majoritairement des class plutôt en favorisant les id pour les éléments uniques et non répétés. Ca peut être les crédits pas exemple ^^

Après ceci
Code:

  <tr>

  </tr>
      <tr>

  </tr>


Ca sert à rien, vraiment. Supprime ça x)

Bon j'ai encore pas mal de trucs à dire je pense, mais je te laisse déjà regarder un peu ça et on verra une fois ton code corrigé :P

Sauter vers: