-
Notifications
You must be signed in to change notification settings - Fork 102
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
ng-select upgrade #2425
base: develop
Are you sure you want to change the base?
ng-select upgrade #2425
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2425 +/- ##
========================================
Coverage 78.44% 78.44%
========================================
Files 88 88
Lines 7182 7182
========================================
Hits 5634 5634
Misses 1548 1548
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Salut ! Je fais donc suite à #2598 ici. Merci pour cette PR ! Je vois que tu as désactivé le virtualScroll qui est utilisé :
Et j'ai peur que ça pose problème quand il y a beaucoup de données (jeux de données/cadres d'acquisition/observateurs). Y-a-t-il moyen de le remettre à Merci beaucoup ! |
J'avais fait un premier retour sur ce virtualScroll dans un com' ici : #2372 (comment) |
0cdc458
to
a9b1307
Compare
En lien avec #2372.
Permet le multiligne sur les ng-select du module GN2Common.
Il ya des ng-select dans le module metadata mais il faudrait probablement les charger à partir de ceux du GN2Common.
Dans cette PR j'ai amélioré le fonctionnement de la recherche en splitant le terme recherché par les espaces, ce qui permet de recherché via des bout de mot peu importe l'ordre du texte saisie. Au passage la recherche fonctionne sans prendre en compte les accents.
Par contre ce n'est pas mis en place pour le component pnx-areas où la recherche se déroule côté serveur.