Projet

Général

Profil

Actions

Anomalie #606

fermé

Planete.april.org

Ajouté par Philippe Vayssière il y a plus de 13 ans. Mis à jour il y a plus de 2 ans.

Statut:
Rejeté
Priorité:
Faible
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
30/07/2011
Echéance:
% réalisé:

80%

Temps estimé:
9:00 h
Temps passé:

Description

http://planete.april.org tente de charger beaucoup de fichiers inexistants.

Liste des fichiers 404 :

Les fichiers JS et ie.css sont appelés dans head, les 2 images GIFs sont appelées depuis les 2 fichiers CSS chargés via @import

Script de la page

Puisque jQuery n'est pas chargé, le seul script (inline) de la page ne fonctionne pas non plus :

$(function(){
$('a[@rel$="external"]')
.click(function(){this.target = "_blank";
});
});

Il se trouve en fin d'élément head. Son but est d'ouvrir dans une nouvelle fenêtre/un nouvel onglet les liens externes (et il n'y a que ça sur un planet). Comme ce n'est pas accessible de forcer l'ouverture dans une nouvelle fenêtre sans avertissement, autant tout enlever !

Commentaire conditionnel dans head

<!--[if IE]> <link rel="stylesheet" href="http://www.april.org/sites/default/theme/zen_april/ie.css" type="text/css"> <![endif]-->

Il est à supprimer puisque le fichier n'existe pas (cf. ci-dessus) et que l'affichage est OK avec IE 7.
Note : ne (plus) jamais écrire un Commentaire Conditionnel qui cible toutes les versions d'IE ! IE 9 est respectueux des standards, IE 8 a rarement besoin d'un correctif et en tout cas certainement pas ceux qu'on destine à IE 7 et 6. IE 10 ne prendra plus en compte les Commentaires Conditionnels.
La bonne manière de l'écrire est donc plutôt de cibler IE 7 et moins :
<!--[if IE lte 7]>

Fichiers

110731_planete.zip (137 ko) 110731_planete.zip Philippe Vayssière, 31/07/2011 15:10
planete_css_110805.zip (6,18 ko) planete_css_110805.zip CSS: correction bug d'affichage pour #content présent dans un billet du Planète Philippe Vayssière, 05/08/2011 16:18

Mis à jour par Philippe Vayssière il y a plus de 13 ans

En pièce jointe :
  • deux nouveaux fichiers CSS (version optimisée et normale avec tous les commentaires) dans /htdocs/
  • le fichier template index.html.tmpl dans /templates/
  • des images optimisées en poids dans /htdocs/images/

Le template prend en compte les notes de ce ticket ci-dessus et bien d'autres choses. Les 2 fichiers CSS actuels ne sont plus appelés, c'est druplanet_zen_april.min.css qui l'est maintenant.

Autres modifs:

templates\index.html.tmpl
  • UTF-8 sur infobulle du dernier item de menu
  • indentation
  • avec doctype XHTML : tags auto-fermants avec un slash final ex: <meta />
  • pas de CC 404 pour IE
  • CC vide pour bug IE8
  • aucun script JS
  • footer (c) 2009 => 2011
  • meta viewport pour éventuelle adaptation aux mobiles
  • plus d'éléments style et de @import
  • charge nouvelle CSS via élément link
htdocs/druplanet_zen_april.css
  • reprend 100% de druplanet.css
  • reprend 20% de zen_april.css (gros nettoyage !!)
  • repositionnement du menu et du header via display: table; (rien de prévu pour Fx<4.0)
  • taille titres en em à partir d'une base de 14px (cf. #page)

Mis à jour par Philippe Vayssière il y a plus de 13 ans

J'ai constaté une erreur d'affichage avec un billet qui utilise un ID déjà utilisé par le Planète.
La mise à jour présente dans les 2 fichiers CSS joints corrige le problème pour cet ID là (#content).
Les 2 fichiers CSS sont à placer dans le répertoire /htdocs/ et remplacent les 2 existants.

Pour éviter tout problème à l'avenir, il faudra modifier tous les ID du Planète pour éviter les conflits (ex: id="planete_content" et pas id="content"), à la fois dans les templates et dans le fichier CSS.

Mis à jour par Frédéric Couchet il y a plus de 2 ans

  • Statut changé de Nouveau à Rejeté
Actions

Formats disponibles : Atom PDF