data: add select2 widget for page selection in link list cell (#76059) #79

Merged
vdeniaud merged 1 commits from wip/76059-manager-cellule-lien-lien-intern into main 2023-04-20 09:48:47 +02:00
Owner
No description provided.
vdeniaud force-pushed wip/76059-manager-cellule-lien-lien-intern from 63b26acb70 to 3aa7ab9802 2023-04-13 15:22:16 +02:00 Compare
vdeniaud force-pushed wip/76059-manager-cellule-lien-lien-intern from 3aa7ab9802 to c67dc5061b 2023-04-13 17:07:02 +02:00 Compare
vdeniaud changed title from WIP: data: add select2 widget for page selection in link list cell (#76059) to data: add select2 widget for page selection in link list cell (#76059) 2023-04-13 17:15:07 +02:00
vdeniaud reviewed 2023-04-13 17:16:35 +02:00
@ -37,2 +38,4 @@
<script src="{% static "js/combo.map.js" %}"></script>
<script src="{% static "js/combo.multiselectwidget.js" %}"></script>
<script src="{% static "xstatic/select2.min.js" %}"></script>
<script src="{% static "xstatic/i18n/fr.js" %}"></script>
Author
Owner

Ici on exploite les traductions packagées avec select2, c'est peu explicite mais nettement plus simple que de les prendre à notre charge comme dans wcs et a2.

Ici on exploite les traductions packagées avec select2, c'est peu explicite mais nettement plus simple que de les prendre à notre charge comme dans wcs et a2.
vdeniaud reviewed 2023-04-13 17:19:42 +02:00
@ -1198,0 +1207,4 @@
page_number = 1
pages = Page.get_as_reordered_flat_hierarchy(Page.objects.all())
pages = [x for x in pages if search_term in unicodedata.normalize('NFKC', x.title).casefold()]
Author
Owner

J'ai fait au plus simple pour la partie recherche.

Un autre truc simple et plus performant aurait été le filtrage en SQL via genre __icontains, mais la gestion des accents aurait été moins bonne.

Sinon la meilleure approche ça serait probablement trigram + unaccent mais c'est relou à mettre en place.

D'une manière ou d'une autre cette vue à vocation à devenir générique au détour d'un autre ticket qui nécessite select2 pour un champ d'un cellule.

J'ai fait au plus simple pour la partie recherche. Un autre truc simple et plus performant aurait été le filtrage en SQL via genre `__icontains`, mais la gestion des accents aurait été moins bonne. Sinon la meilleure approche ça serait probablement trigram + unaccent mais c'est relou à mettre en place. D'une manière ou d'une autre cette vue à vocation à devenir générique au détour d'un autre ticket qui nécessite select2 pour un champ d'un cellule.
vdeniaud force-pushed wip/76059-manager-cellule-lien-lien-intern from bfa293f70f to c67dc5061b 2023-04-13 17:44:16 +02:00 Compare
vdeniaud force-pushed wip/76059-manager-cellule-lien-lien-intern from c67dc5061b to a9fc7ca474 2023-04-17 16:36:56 +02:00 Compare
vdeniaud force-pushed wip/76059-manager-cellule-lien-lien-intern from a9fc7ca474 to 2ee91c942b 2023-04-19 11:31:24 +02:00 Compare
vdeniaud added 1 commit 2023-04-19 14:21:39 +02:00
fpeters approved these changes 2023-04-20 08:20:15 +02:00
vdeniaud merged commit de36c6fcd1 into main 2023-04-20 09:48:47 +02:00
vdeniaud deleted branch wip/76059-manager-cellule-lien-lien-intern 2023-04-20 09:48:48 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 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/combo#79
No description provided.