Project

General

Profile

Actions

Anomalie #606

closed

Planete.april.org

Added by Philippe Vayssière over 13 years ago. Updated over 2 years ago.

Status:
Rejeté
Priority:
Faible
Assignee:
-
Category:
-
Target version:
-
Start date:
07/30/2011
Due date:
% Done:

80%

Estimated time:
9:00 h
Spent time:

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]>

Files

110731_planete.zip (137 KB) 110731_planete.zip Philippe Vayssière, 07/31/2011 03:10 PM
planete_css_110805.zip (6.18 KB) planete_css_110805.zip CSS: correction bug d'affichage pour #content présent dans un billet du Planète Philippe Vayssière, 08/05/2011 04:18 PM
Actions #1

Updated by Philippe Vayssière over 13 years ago

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)
Actions #2

Updated by Philippe Vayssière over 13 years ago

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.

Actions #3

Updated by Frédéric Couchet over 2 years ago

  • Status changed from Nouveau to Rejeté
Actions

Also available in: Atom PDF