entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

misc: use autocomplete for data source selection (#75960) #772

Merged
fpeters merged 1 commits from wip/75960-data-source-autocompletion into main 2023-10-27 10:25:58 +02:00
Conversation 1 Commits 1 Files Changed 4 +48 -2
fpeters commented 2023-10-08 08:44:46 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/75960-data-source-autocompletion from 5d40d7b61b to a0f5678378 2023-10-08 10:36:07 +02:00 Compare
fpeters changed title from WIP: misc: use autocomplete for data source selection (#75960) to misc: use autocomplete for data source selection (#75960) 2023-10-08 11:00:22 +02:00
tnoel requested changes 2023-10-09 09:59:08 +02:00
Dismissed
wcs/data_sources.py Outdated
@ -198,6 +198,10 @@ class DataSourceSelectionWidget(CompositeWidget):
value=value.get('type'),
attrs={'data-dynamic-display-parent': 'true'},
)
if len(options) > 15:
tnoel commented 2023-10-09 09:54:43 +02:00
Owner
Copy Link

Je comprends la nécessité de l'autocomplétion quand il y a trop de choix, mais (et c'est un avis perso) je trouve que l'autocomplétion c'est vraiment pas pratique, aussi je proposerais de pousser à 50 ici, histoire que sur la plupart des petits sites tranquilles on garde une liste simple.

Je comprends la nécessité de l'autocomplétion quand il y a trop de choix, mais (et c'est un avis perso) je trouve que l'autocomplétion c'est vraiment pas pratique, aussi je proposerais de pousser à 50 ici, histoire que sur la plupart des petits sites tranquilles on garde une liste simple.
lguerin commented 2023-10-09 10:25:30 +02:00
Owner
Copy Link

je crois que ça rajoute juste un champ de recherche, on a quand même tous les choix présentés dans le select, pas d'autocompletion en ajax

je crois que ça rajoute juste un champ de recherche, on a quand même tous les choix présentés dans le select, pas d'autocompletion en ajax
fpeters commented 2023-10-09 10:46:42 +02:00
Author
Owner
Copy Link

J'ai monté à 50 dans la PR mais oui toutes les options sont de base affichées, pas de chargement asynchrone. (et je peux remettre à 15 si la crainte était ça).

J'ai monté à 50 dans la PR mais oui toutes les options sont de base affichées, pas de chargement asynchrone. (et je peux remettre à 15 si la crainte était ça).
fpeters force-pushed wip/75960-data-source-autocompletion from a0f5678378 to 03a69a5b8f 2023-10-09 10:14:46 +02:00 Compare
fpeters requested review from tnoel 2023-10-24 11:52:56 +02:00
tnoel commented 2023-10-24 14:24:55 +02:00
Owner
Copy Link

J'ai clairement rien compris au patch, je vous laisse décider.

J'ai clairement rien compris au patch, je vous laisse décider.
tnoel refused to review 2023-10-24 14:25:01 +02:00
fpeters dismissed tnoel’s review 2023-10-24 16:45:10 +02:00
Reason:

" J'ai clairement rien compris au patch, je vous laisse décider."

lguerin approved these changes 2023-10-24 16:47:59 +02:00
fpeters force-pushed wip/75960-data-source-autocompletion from 03a69a5b8f to 8b2d75121b 2023-10-27 08:38:54 +02:00 Compare
fpeters merged commit eb4b70ccb2 into main 2023-10-27 10:25:58 +02:00
fpeters deleted branch wip/75960-data-source-autocompletion 2023-10-27 10:25:58 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#772
No description provided.
Delete Branch "wip/75960-data-source-autocompletion"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?