Project

General

Profile

Anomalie #4057

Charset des feuilles CSS compilées

Added by marcimat 🌈 over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Category:
css
Target version:
Start date:
12/08/2017
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

Le commit r5618 de 2006 indiquait que Firefox sans déclaration spécifique de charset appliquait un charset iso par défaut avec un type text/html.
Du coup, il a été mis explicitement cet envoi de charset dans tous les CSS envoyés avec cette ligne :

#HTTP_HEADER{Content-Type: text/css; charset=iso-8859-15}

De nos jours, cela pose différents soucis :
- le charset conseillé est utf-8 : https://www.w3.org/International/questions/qa-css-charset
- des propriétés "content: xxx" avec un caractère UTF-8 peut être mal interprété selon le caractère.
- Firefox ne réagit plus du tout comme ça maintenant :) Il garde bien le type CSS et prend par défaut un charset UTF-8.
- accessoirement certaines librairies ajoutent la directive charset "UTF-8"; en tête de fichier, et du coup, ça ne colle pas du tout.

Les fichiers de SPIP et particulièrement donc ceux en questions sont normalement déjà tous enregistrés en utf-8.
Je propose de remplacer donc la directive (et le commentaire associé) par :

[(#REM)<style>/*
    Ce squelette definit les styles de l'espace prive

    Note: l'entete "Vary:" sert a repousser l'entete par
    defaut "Vary: Cookie,Accept-Encoding", qui est (un peu)
    genant en cas de "rotation du cookie de session" apres
    un changement d'IP (effet de clignotement).
*/]
#CACHE{3600*100,cache-client}
#HTTP_HEADER{Content-Type: text/css; charset=utf-8}
#HTTP_HEADER{Vary: Accept-Encoding}

Je ne sais pas si la "note" est toujours valable au passage.

Pas d’objection ?

Associated revisions

Revision 107993 (diff)
Added by marcimat@rezo.net over 1 year ago

Ticket #4057 : Charset des feuilles CSS en utf-8

History

#1 Updated by nicod_ 🐿 over 1 year ago

Aucune objection, au contraire.
Et il me semble que la note est un peu archaïque et concerne les vieux IE... (canal historique)

#2 Updated by nicod_ 🐿 over 1 year ago

Mmmmh... oublie ma remarque sur la note, ça semble toujours d'actualité.

https://blog.stackpath.com/accept-encoding-vary-important

Voir si d'autres le confirment ?

#3 Updated by b b over 1 year ago

Oui, gogogo pour le charset, et pour la note il semble qu'elle soit encore utile.

#4 Updated by marcimat 🌈 over 1 year ago

  • Status changed from Nouveau to Fermé
  • Resolution set to fixed

Donc r23824 dans SPIP modifie le charset CSS.
Appliqué aussi sur tous les plugins-dist par https://zone.spip.org/trac/spip-zone/changeset/107993
Il faudra le faire sur les plugins. Mais comme en général ces squelettes sont des inclusions d’un squelette principal, indiquer le charset dedans ne fait pas grand chose.

On ferme.

Also available in: Atom PDF