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

Affiche l'éditeur dès que le DOM est chargé #5483

Merged
merged 1 commit into from
Oct 20, 2019

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Oct 19, 2019

Sur le forum, des membres se sont plaint de la lenteur de l'affichage de l'éditeur. J'ai donc investigué et je me suis rendu compte qu'on charge l'éditeur avec l'événement window.onload, c'est-à-dire après que toutes les ressources de la page (dont les images) soient chargés. Je propose de passer par l'événement document.onDOMContentLoaded pour charger l'éditeur dès que le DOM est prêt. Ainsi, si une ressource met du temps à charger, cela ne bloque pas l'affichage de la page.

QA :

  • Se mettre sur upstream/dev
  • Lancer le serveur
  • Publier un message avec ![Image inaccessible](http://10.255.255.1/test.png)
  • Vérifier que la page est en attente d'une requête et l'éditeur ne s'affiche pas
  • Basculer sur ma branche
  • Générer le front avec make build-front
  • Recharger la page
  • Vérifier que la page est en attente d'un requête mais l'éditeur s'affiche correctement

@Situphen Situphen added the C-Front Concerne l'interface du site label Oct 19, 2019
@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling 74289c2 on Situphen:editor into 6071e13 on zestedesavoir:dev.

@artragis
Copy link
Member

QA OK

@artragis artragis merged commit 2f0b3db into zestedesavoir:dev Oct 20, 2019
@Situphen Situphen deleted the editor branch October 20, 2019 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants