Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Avant : on affichait sur la même page 1272 membres actifs + 1161 membres alumnis. Et pour chacun de ces membres, on affichait un avatar.
Résultat : la page est extrêmement longue et complexe (note G sur l'écoxindex, elle fait planter certains navigateur ou téléphone) 😬
Dans cette proposition :
La page est ainsi beaucoup plus légère (ça reste une mauvaise note à l'écoindex, mais il faudrait mettre en place une pagination pour faire mieux).
(PS : le code
search-member.js
est dégueu : il duplique du code de template. Je me demande si on pouvait pas simplement faire du display-none sur les éléments à cacher plutôt que de ré-ecrire tout le html... Mais bon, ça dépasse mes compétences donc je ne change pas le fonctionnement existant dans cette PR)