NextINpact / Next

18 stars 0 forks source link

Police manquante : source code pro #267

Open francois-dorin opened 11 months ago

francois-dorin commented 11 months ago

La police référencée pour l'inclusion de code dans les articles n'est pas chargée. Résultat des courses : la police est la police classique de l'article, au lieu d'être la police monospace source-code-pro.

Du coup, c'est la police SourceSansProRegular qui est utilisée (la même que pour l'article).

Suggestion :

On peut voir les problèmes de police sur un article comme celui-ci : https://next.ink/117581/guacamole-sur-un-plateau-5-5-on-passe-a-deux-facteurs-et-cest-termine/

Toutes les commandes listées sont sensées être en SourceCodePro.

Cqoicebordel commented 11 months ago

Le problème est pas la police en tant que telle. Ca vient de la feuille de style de Wordpress.

En effet, dans le style.css de Next, y'a

code {
    font-family: source-code-pro, Menlo, Monaco, Consolas, 'Courier New',
        monospace;
}

Le problème vient de la feuille de style style.min.css?ver=6.4.2 qui contient :

.wp-block-code code {
    display: block;
    font-family:inherit;
    overflow-wrap: break-word;
    white-space: pre-wrap
}

Si on commente la ligne font-family:inherit; tout fonctionne correctement.

Une autre solution, moins propre, mais plus simple, probablement, est de rajouter un !important à la find de la ligne sur les fonts :

code {
    font-family: source-code-pro, Menlo, Monaco, Consolas, 'Courier New',
        monospace !important;
}

Mais ça n'empêche pas que, évidemment, la police source-code-pro n'est effectivement pas chargée. Mais au moins, avec ces changements, pour moi ça retombe sur du Courrier New, qui fait le job.

francois-dorin commented 11 months ago

Non seulement tu as tout à fait raison, mais en plus, j'avais vu ce détail. Mais après je suis tellement aller creuser du côté de source-code-pro, pour avoir à quoi elle ressemblait et voir pourquoi elle n'était pas chargée (a priori, elle n'est tout simplement pas incluse, ce n'est pas un problème d'erreur 404) que j'ai fais un énorme raccourci foireux.

Je crois qu'il est temps que je m'arrête et que je prenne des vacances ! Merci à toi pour la correction ;)

hezirem commented 11 months ago

Je crois qu'il est temps que je m'arrête et que je prenne des vacances ! Merci à toi pour la correction ;)

A qui le dis tu ! 👀