Bug 29242

Summary:	Flatpickr - Turn autocomplete off
Product:	Koha	Reporter:	Jonathan Druart <jonathan.druart>
Component:	Staff interface	Assignee:	Owen Leonard <oleonard>
Status:	CLOSED FIXED	QA Contact:	Testopia <testopia>
Severity:	normal
Priority:	P5 - low	CC:	gmcharlt, kyle, nugged
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	Trivial patch
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:	21.11.00
Circulation function:
Bug Depends on:
Bug Blocks:	29239
Attachments:	Bug 29242: Flatpickr - Should autocomplete be turned off? Bug 29242: Flatpickr - Should autocomplete be turned off? Bug 29242: Flatpickr - Should autocomplete be turned off?

Description Jonathan Druart 2021-10-14 08:28:49 UTC

Without autocomplete turned off: https://usercontent.irccloud-cdn.com/file/9NZax9ln/image.png

Should we turn it off?

On Chrome it appears right after the input has the focus, on Firefox you have to double click.

Comment 1 Owen Leonard 2021-10-14 17:24:03 UTC

I like having the browser's autocomplete, but that may just be because I'm almost always in testing mode rather that using it live.

We could set the autocomplete attribute upon initialization of the Flatpickr instance and avoid making any changes to the templates.

Another idea is to tie into the "keydown" function we already have and set autocomplete to "on" if the user types a key other than an arrow key. Probably overkill.

Comment 2 Owen Leonard 2021-10-14 18:03:45 UTC

Created attachment 126302 [details] [review]
Bug 29242: Flatpickr - Should autocomplete be turned off?

This patch adds a step to Flatpickr initialization to add an
autocomplete attribute set to "off" so that browsers' built-in
autocomplete menus do not obscure the calendar.

To test you must be using a browser which has form history enabled.

Locate a form that includes a Flatpickr input field, e.g.  Reports
-> Catalog statistics.

Fill the date fields and submit the form. Go back to the form and click
one of the form fields you previously filled out (once in Chrome, twice
in Firefox). The browser should not show its native form history
dropdown.

Comment 3 David Nind 2021-10-16 18:35:33 UTC

Created attachment 126408 [details] [review]
Bug 29242: Flatpickr - Should autocomplete be turned off?

This patch adds a step to Flatpickr initialization to add an
autocomplete attribute set to "off" so that browsers' built-in
autocomplete menus do not obscure the calendar.

To test you must be using a browser which has form history enabled.

Locate a form that includes a Flatpickr input field, e.g.  Reports
-> Catalog statistics.

Fill the date fields and submit the form. Go back to the form and click
one of the form fields you previously filled out (once in Chrome, twice
in Firefox). The browser should not show its native form history
dropdown.

Signed-off-by: David Nind <david@davidnind.com>

Comment 4 Katrin Fischer 2021-10-31 14:42:29 UTC

Created attachment 127140 [details] [review]
Bug 29242: Flatpickr - Should autocomplete be turned off?

This patch adds a step to Flatpickr initialization to add an
autocomplete attribute set to "off" so that browsers' built-in
autocomplete menus do not obscure the calendar.

To test you must be using a browser which has form history enabled.

Locate a form that includes a Flatpickr input field, e.g.  Reports
-> Catalog statistics.

Fill the date fields and submit the form. Go back to the form and click
one of the form fields you previously filled out (once in Chrome, twice
in Firefox). The browser should not show its native form history
dropdown.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Comment 5 Jonathan Druart 2021-11-02 15:54:02 UTC

Pushed to master for 21.11, thanks to everybody involved!