Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Front - Performance / Intégration / Accessibilité / SEO #55

Closed
RiwsPy opened this issue Jun 13, 2024 · 3 comments · Fixed by #58
Closed

Front - Performance / Intégration / Accessibilité / SEO #55

RiwsPy opened this issue Jun 13, 2024 · 3 comments · Fixed by #58

Comments

@RiwsPy
Copy link
Contributor

RiwsPy commented Jun 13, 2024

On a pas facilement la main mais il y a des axes d'amélioration.

  • Performance : 10Mo de chargement pour la page d'accueil (??!)
  • Intégration
    • Nettoyer les feuilles de style du projet
    • Intégrer directement les fonts dans le projet au lieu d'aller chercher chez fonts.googleapis.com
    • Utiliser des variables (notamment pour les fonts et les couleurs)
    • Pas une bonne pratique que de pouvoir overrider les feuilles de style depuis des fichiers python (stylekit) (nécessite trop de temps)
  • Accessibilité
    • Mettre des balises alt aux images + height/width quand possible
  • SEO
    • Ajouter les balises meta (rien de prévu, on oublie)
Copy link
Contributor

Penser à merger la PR 45 qui traite une partie du sujet.

@anquetos
Copy link
Collaborator

Quelques remarques :

  • Fusion et surtout nettoyage des feuilles de styles prévue de mon côté. Il y a pas mal de classes qui sont obsolètes et non utilisées.
  • Pour le stylekit c'est la méthode indiquée par Taipy qui avait été suivie. Tu penses qu'il y a moyen de faire autrement ?
  • Concernant les balises images, les pages ont été créées avec la classe Mardown de Taipy, du coup c'est un peu limitant pour pas mal de chose. Mais je vois qu'il y a une propriété label sur l'élément image. Tu penses que ça peut suffire ?
  • Pour le SEO et les balises meta, avec la méthode du Markdown j'aurais tendance à dire que ça va être compliqué.

@RiwsPy
Copy link
Contributor Author

RiwsPy commented Jun 14, 2024

  • Pour les feuilles de style c'est plutôt les leurs qui posent problème (il y en a beaucoup et elles sont appelées une à une). Cela dit, un brin de ménage sur celles du projet est une bonne idée.
  • Pour le stylekit, je sais que ce qui a été fait est conforme à la doc. C'est quand même une mauvaise pratique de leur part, on override pas le front depuis le back et inversement. J'ai creusé un peu. Visiblement rien n'est prévu pour faire fonctionner taipy autrement que comme eux l'on prévu. Il y a du inline !important un peu partout et pas de moyen de changer l'ordre d'appel des feuilles de style sauf à réécrire toutes les méthodes : c'est un enfer.
  • label va ajouter un tag aria-label c'est toujours ça de pris. Il y a des balises <a> sans alt tout de même, par exemple le logo de D4G.
  • Effectivement, rien n'est prévu pour les balises meta et rien n'est prévu pour overrider leurs fichiers html…

Les marges de manœuvre sont donc plus que très réduites. Je mets à jour le ticket.

edit : s'il est compliqué de passer outre le stylekit, celui-ci défini tout de même des variables utilisables dans le css.

stylekit = {
  "color_primary": "#021978"
}

va créer

:root {
  --color-primary: #021978;
}

(note le _ qui est devenu -)

et le rend utilisable dans toutes les feuilles de style :

  background-color: var(--primary-color);

@anquetos anquetos linked a pull request Jul 3, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants