The payment details page (Patron -> Fines -> Account -> Details) needs some markup corrections and for the grid to be converted to Bootstrap.
Created attachment 82757 [details] [review] Bug 21913: Clean up payment details page This patch makes various corrections and improvements to the payment details page: - Grid converted to Bootstrap - Correct title tag - Add patron toolbar - Adding missing patron-related JavaScript - Improve some language To test, apply the patch and locate a patron who has one or more payments on their account. - Open Fines -> Account -> Details of a transaction - Confirm that the page looks correct and adjusts well at various browser widths. - Confirm that patron toolbar items work correctly - Confirm that the payments table only appears when there are payments to list.
İf there isn't any payment, payment section appears. Other functionalities are working as described.
(In reply to Devinim from comment #2) > İf there isn't any payment, payment section appears. Other functionalities > are working as described. Failing QA for that as it is advertised in the commit message.
(In reply to Jonathan Druart from comment #3) > (In reply to Devinim from comment #2) > > İf there isn't any payment, payment section appears. Other functionalities > > are working as described. > > Failing QA for that as it is advertised in the commit message. Hi Jonathan, We didn't understand this :( Can you explain a bit more? Thanks.
Created attachment 83793 [details] [review] Bug 21913: Clean up payment details page This patch makes various corrections and improvements to the payment details page: - Grid converted to Bootstrap - Correct title tag - Add patron toolbar - Adding missing patron-related JavaScript - Improve some language To test, apply the patch and locate a patron who has one or more payments on their account. - Open Fines -> Account -> Details of a transaction - Confirm that the page looks correct and adjusts well at various browser widths. - Confirm that patron toolbar items work correctly
(In reply to Jonathan Druart from comment #3) > (In reply to Devinim from comment #2) > > İf there isn't any payment, payment section appears. Other functionalities > > are working as described. > > Failing QA for that as it is advertised in the commit message. I removed those changes. There wasn't a straightforward way to accomplish it with template changes.
Created attachment 83828 [details] [review] Bug 21913: Clean up payment details page This patch makes various corrections and improvements to the payment details page: - Grid converted to Bootstrap - Correct title tag - Add patron toolbar - Adding missing patron-related JavaScript - Improve some language To test, apply the patch and locate a patron who has one or more payments on their account. - Open Fines -> Account -> Details of a transaction - Confirm that the page looks correct and adjusts well at various browser widths. - Confirm that patron toolbar items work correctly Signed-off-by: Jose-Mario <jose-mario.monteiro-santos@inLibro.com>
Created attachment 83848 [details] [review] Bug 21913: Clean up payment details page This patch makes various corrections and improvements to the payment details page: - Grid converted to Bootstrap - Correct title tag - Add patron toolbar - Adding missing patron-related JavaScript - Improve some language To test, apply the patch and locate a patron who has one or more payments on their account. - Open Fines -> Account -> Details of a transaction - Confirm that the page looks correct and adjusts well at various browser widths. - Confirm that patron toolbar items work correctly Signed-off-by: Jose-Mario <jose-mario.monteiro-santos@inLibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Awesome work all! Pushed to master for 19.05
Enhancement, will not be backported to 18.11.x series.