Project

General

Profile

Evolution #3987

Ne pas supprimer les crédits à la compression CSS

Added by tetue tetue almost 2 years ago. Updated 10 months ago.

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

0%

Resolution:
fixed

Description

Je viens de remarquer que la compression CSS supprime les commentaires (c'est bien) y compris les crédits et licence (c'est pas bien) :

Lorsque j'utilise le reset normalise.css, après simple minification, le fichier commence par
/*! normalize.css v7.0.0 | MIT License | github.com/necolas/normalize.css */html{line-height:
(voir ici )

Mais après compression via SPIP, l'information de licence est supprimée :
/* squelettes/css/base.css */
html{line-height:

Les commentaires CSS commençant par /*! (c'est-à-dire avec point d'exclamation à l'ouverture) — couramment utilisés pour indiquer les crédits en première ligne — devraient être préservés.

History

#1 Updated by b b almost 2 years ago

  • Tracker changed from Anomalie to Evolution
  • Project changed from SPIP to Compresseur
  • Priority changed from Haut to Normal

Attribuer le ticket au bon projet, évolution et non anomalie, tout doux sur la priorité :)

Amha, c'est plus dans le paquet.xml qu'il faut mettre en avant les crédits car cela permet de les afficher publiquement sur le site de présentation des plugins et dans l'interface privée des sites qui utilisent le plugin en question.

#2 Updated by tetue tetue almost 2 years ago

Valoriser les crédits dans plugin.xml, c'est super interne (puisque ça concerne une population hyper minoritaire de spipmestres lecteurs de ces infos) mais c'est bien aussi, mais ça ne résout pas le problème qui est pire : on les supprime du code source des pages web !

L'exclamation dans les commentaires CSS /*! */ marque l'importance et doit échapper à la compression, ce que respecte la minification via LESS et SASS par exemple.

Ce n'est certes pas un pur dysfonctionnement technique, mais ça pose un problème moral qui me semblait assez grave pour justifier une priorisation.

Also available in: Atom PDF