yeswiki icon indicating copy to clipboard operation
yeswiki copied to clipboard

Supression google font

Open RomainLalande opened this issue 6 years ago • 5 comments

Description Supprimer la police google des thèmes de yeswiki.

RomainLalande avatar Dec 08 '18 11:12 RomainLalande

Bon après avoir exploré un peu avec @sylvainboyer , déjà ça concerne l'extension bootstrap3, pas le dépot yeswiki/yeswiki. Ensuite, ce qu'il faudrait faire :

  • Mettre les font dans le dossier theme/font en "dur"
  • Modifier la première ligne des fichiers de css de façon a fabriquer une url qui pointe sur le fichier de police (mais est-ce possible de récupérer bas_url dans le css ? ! )

Du coup c'est à voir quoi...

RomainLalande avatar Apr 08 '19 15:04 RomainLalande

Salut,

Bon après avoir exploré un peu avec @sylvainboyer , déjà ça concerne l'extension bootstrap3, pas le dépot yeswiki/yeswiki. Ensuite, ce qu'il faudrait faire :

Cool, merci d'avoir amorcé le travail dans le dépôt vivant :)

Sur le fond, si une dépendance pose problème, c'est quand même un problème à prendre en compte par Yeswiki : choisir une dépendance, c'est un choix éditorial. Surtout pour la version distribuée par défaut, histoire d'avoir une version de base un minimum fonctionnelle et qui garanti le respect de la légistlation et le service fourni.

Effectivement la dépendance ne se trouve pas directement dans le dépôt mais dans yeswiki-theme-base, plus précisément ici : https://github.com/YesWiki/yeswiki-theme-base/blob/b092fbc23e53bd1681935e5e5fd5fb531c6187eb/styles/base.bootstrap.min.css#L1

Mettre les font dans le dossier theme/font en "dur" Modifier la première ligne des fichiers de css de façon a fabriquer une url qui pointe sur le fichier de police (mais est-ce possible de récupérer bas_url dans le css ? ! )

En principe, modifier l'url dans @import suffit, il faut mettre une référence relative par rapport au fichier css de bootstrap modifié, pas besoin d'utiliser la fonction url(). Quelque chose comme @import ../fonts/fichier cf. yeswiki-theme-margot qui fait ça correctement

Autrement la dernière version de Bootstrap 3.3.7 n'embarque pas la dépendance vers la fonte déposée sur le CDN de Google, peut-être qu'une mise à jour serait suffisante ?

NB : La semaine dernière, la justice allemande a interdit l'utilisation des fontes Google hébergées sur leurs serveurs pour les problèmes de vie privée et géopolitiques que ça pose.

https://social.imirhil.fr/@aeris/107716276397225915

Ca vaut aussi potentiellement pour les CDN américains comme Clouflare comme ceux qu'on trouve sur le site vitrine de Yeswiki Je n'ai pas pris le temps de voir d'où ces références viennent. S'il faut avoir une discussion sur l'usage des CDN américains, pas sûr que je réponde tout de suite. Il me semble que c'est important que prendre en compte le respect de la vie privée dans le design d'un logiciel, en tenir compte au même titre de l'ergonomie et l'ouverture (etc.).

Sebbaz avatar Jan 31 '22 10:01 Sebbaz

Bonjour, Nous sommes ok sur le principe, mais nous manquons de forces vives et de temps pour revoir les dépots en question.
Mon point de vue, c'est que de proposer une pull request sur les thèmes incriminés serait plus rapide que de se lancer dans une grande discussion ou tout le monde est d'accord.

mrflos avatar Jan 31 '22 11:01 mrflos

je m'occuperais de l'appel cdn sur le site de yeswiki.net.
Toute PR sur https://github.com/YesWiki/yeswiki-theme-bootstrap3 sera reviewé rapidement pour mise en ligne sur les dépots de yeswiki.

mrflos avatar Jan 31 '22 11:01 mrflos

(et sur https://github.com/YesWiki/yeswiki-theme-base du coup aussi, et sans doute d'autres à vérifier)

mrflos avatar Jan 31 '22 11:01 mrflos

Je clos cette issue car

  1. le thème margot embarque ses propres polices
  2. la page LookWiki permet de définir des preset de thème avec le thème margot et que la création de ces presets importe automatiquement en local la police concernée pour éviter d'utiliser un CDN peut scrupuleux de la vie privé.

Version concernée yeswiki doryphore 4.3.0 et ultérieures

J9rem avatar Jun 23 '23 07:06 J9rem