Charset des feuilles CSS compilées #4057

Closed
opened 5 years ago by marcimat · 4 comments
marcimat commented 5 years ago
Owner

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)
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 : <pre> #HTTP_HEADER{Content-Type: text/css; charset=iso-8859-15} </pre> 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 : <pre> [(#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} </pre> Je ne sais pas si la "note" est toujours valable au passage. Pas d’objection ?
Owner

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

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

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 ?

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 ?
b_b commented 5 years ago
Owner

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

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

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.
Statut changé à Fermé

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. **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.