Fix(k-psul): Autocomplete on mobile devices and some other bugs #911

nathan.frison · 2026-07-01T22:16:32+02:00

nathan.frison commented

2026-07-01 22:16:32 +02:00

A few bugs I found :

Make instructions field optional for article creation and edition
Fixed some css width issue on mobile devices
Fixed the autocomplete on some mobile devices, after looking at : https://github.com/Pixabay/JavaScript-autoComplete/issues/71
Made ctrl+a and escape reset all autocomplete fields

A few bugs I found : - Make instructions field optional for article creation and edition - Fixed some css width issue on mobile devices - Fixed the autocomplete on some mobile devices, after looking at : https://github.com/Pixabay/JavaScript-autoComplete/issues/71 - Made ctrl+a and escape reset all autocomplete fields

nathan.frison added the

type -- bug

domain -- kfet

labels

2026-07-01 22:16:32 +02:00

nathan.frison added 4 commits

2026-07-01 22:16:32 +02:00

fix(k-psul): Added blank=True to instructions to make it optional in the creation and update forms 79bec8e9c4

fix(k-psul): fixed css of autocomplete on mobile b3407f4d73

fix(k-psul): fixed autocomplete not working on some mobile devices 9ae6f2f072

fix(k-psul): fixed ctrl+a and escape on article name input now do reset instructions, article nb and stock fields to null, copying the mecanisms of F2 when an article is selected. a712daadb9

nathan.frison requested review from aandres

2026-07-01 22:16:51 +02:00

aandres reviewed

2026-07-01 22:58:06 +02:00

kfet/static/kfet/css/kpsul.css

					
				@ -278,7 +278,7 @@ input[type=number]::-webkit-outer-spin-button {

				}

				#article_autocomplete_container {

Du coup ça déborde par dessous

Du coup ça déborde par dessous ![image](/attachments/27763bd5-4324-4f68-9253-57f791021b48)

image.png

40 KiB

Supprimé sur mobile

👍 1

aandres marked this conversation as resolved

kfet/templates/kfet/kpsul.html Outdated

					
				@ -586,2 +589,3 @@

				        if (e.key === "Escape")

				        if (e.key === "Escape") {

				            updateMatchedArticles("");

				            articleStock.text("");

Quand je regarde en prod c'est déjà géré les lignes que tu rajoutes. Tu avais rencontré un pb ?

Specifiquement, quand tu as déjà sélectionné un article, que donc les stocks/instructions ont une valeur, et que tu reviens sur le champs article pour escape/ctrl+a ça supprime le nom de l'article mais pas le reste

Aaaah si tu es déjà revenu⋅e sur le champ article
Si tu fais echap depuis le champ quantité ça vide déjà tout mais is ok effectivement c'est pertinent. Ça me va

Aaaah si tu es déjà revenu⋅e sur le champ article Si tu fais echap depuis le champ quantité ça vide déjà tout mais is ok effectivement c'est pertinent. Ça me va

aandres marked this conversation as resolved

kfet/templates/kfet/kpsul.html Outdated

					
				@ -592,1 +599,4 @@

				    // Use event 'input' because mobile keyboard do not always send the proper physical keydown event

				    articleSelect.on('input', function (e) {

Le pb avec input c'est qu'il est aussi émis quand tu appuies sur la suppression de caractère, ça rejoue l'auto-complétion et du coup tu es coincé

Il me semble que c'est ce qu'on veut de rejouer l'autocomplétion quand on supprime des caractères, pour réafficher les propositions qui matchent à nouveau, mais peut être que je me trompe

Non parce que si l'auto-complétion a ajouté 5 caractères et que tu t'étais trompé, tu veux en supprimer 6, mais tu les fais un par un et quand tu en supprimes un ça te le re-auto-complete.
Mettons tu veux sirop poire mais tu as tapé "O" après sirop
sirop or -> tu supprimes un caractère
sirop o -> il autocomplète à sirop or et tu es coincé

Non parce que si l'auto-complétion a ajouté 5 caractères et que tu t'étais trompé, tu veux en supprimer 6, mais tu les fais un par un et quand tu en supprimes un ça te le re-auto-complete. Mettons tu veux sirop poire mais tu as tapé "O" après sirop sirop or -> tu supprimes un caractère sirop o -> il autocomplète à sirop or et tu es coincé

nathan.frison added 1 commit

2026-07-02 10:00:56 +02:00

feat(k-psul): Added returnable in the same style as instructions da3ea69e36

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions