developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 10 years ago

Last modified 9 years ago

#1246 confirmed bug

Fichier CSS et la compression avec plusieurs media en attribut

Reported by: nesswaw Owned by:
Priority: low Milestone:
Component: jelix:plugins:response Version: 1.2RC3
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Bonjour,

Ceci n'est pas trop un bug, mais plutôt quelque chose a améliorer.

Quand on inclut une feuille CSS via la cette fonction dans notre myHTMLResponse:

$this->addCSSLink($designPath.'css/main.css', array(media' => 'screen, handheld, tv, projection'));

Et que l'ont active la compression css, cela aura pour effet de générer 4 balise link dans le head, une pour chaque media:

<link type="text/css" href="/cache/minify/css/b9027ff37ea8be79071d9e16f46f41c8.css?1292946734" rel="stylesheet" media="screen" /> <link type="text/css" href="/cache/minify/css/b9027ff37ea8be79071d9e16f46f41c8.css?1292946734" rel="stylesheet" media="handheld" /> <link type="text/css" href="/cache/minify/css/b9027ff37ea8be79071d9e16f46f41c8.css?1292946734" rel="stylesheet" media="tv" /> <link type="text/css" href="/cache/minify/css/b9027ff37ea8be79071d9e16f46f41c8.css?1292946734" rel="stylesheet" media="projection" />

Le but attendu serait plutôt d'avoir une seule balise <link> avec tout les medias dedans:

<link type="text/css" href="/cache/minify/css/b9027ff37ea8be79071d9e16f46f41c8.css?1292946734" rel="stylesheet" media="screen, handheld, tv, projection" />

Vous trouvez pas?

Je vais voir si je peux corriger cela dans le core de Jelix, je mettrais le code ici si je trouve.

Change History (2)

comment:1 Changed 9 years ago by laurentj

  • Component changed from jelix to jelix:plugins:response
  • Priority changed from normal to low
  • Severity changed from normal to minor

comment:2 Changed 9 years ago by laurentj

  • Status changed from new to confirmed

à mon avis, ça se passe plutôt dans minify... chose que je ne veux pas hacker... Voir si une version de minify corrige le problème.

Note: See TracTickets for help on using tickets.