Description
Owen Leonard
2021-04-12 10:13:06 UTC
Created attachment 119449 [details] [review] Bug 28134: Replace use of input type number in onboarding templates This patch changes the instances of input of type="number" to input type="text" with attributes inputmode="numeric" and pattern="[0-9]*" To test, apply the patch and run through the process of installing Koha. When you reach onboarding step 2, "Create a patron category," check the enrollment period -> "In months" field. The field should not accept any input except numbers. Continue onboarding until step 5, "Create a new circulation rule." On this page, check that these fields accept only numbers: - Current checkouts allowed - Loan period - Renewals allowed - Renewals period Created attachment 119451 [details] [review] Bug 28134: Replace use of input type number in onboarding templates This patch changes the instances of input of type="number" to input type="text" with attributes inputmode="numeric" and pattern="[0-9]*" To test, apply the patch and run through the process of installing Koha. When you reach onboarding step 2, "Create a patron category," check the enrollment period -> "In months" field. The field should not accept any input except numbers. Continue onboarding until step 5, "Create a new circulation rule." On this page, check that these fields accept only numbers: - Current checkouts allowed - Loan period - Renewals allowed - Renewals period - Holds allowed (total) - Holds allowed (daily) - Holds per record (count) Created attachment 120435 [details] [review] Bug 28134: Replace use of input type number in onboarding templates This patch changes the instances of input of type="number" to input type="text" with attributes inputmode="numeric" and pattern="[0-9]*" To test, apply the patch and run through the process of installing Koha. When you reach onboarding step 2, "Create a patron category," check the enrollment period -> "In months" field. The field should not accept any input except numbers. Continue onboarding until step 5, "Create a new circulation rule." On this page, check that these fields accept only numbers: - Current checkouts allowed - Loan period - Renewals allowed - Renewals period - Holds allowed (total) - Holds allowed (daily) - Holds per record (count) Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Catches all cases and works as expected.. Signing off. I think we can safely use my SO as a PQA. This is a trivial fix on a set of integer inputs so no ambiguity around decimal handling. PQA Pushed to master for 21.05, thanks to everybody involved! Enhancement not pushed to 20.11.x |