Bug 9310

Summary: Patron image upload template corrections
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: normal    
Priority: P5 - low CC: chris, jonathan.druart, veron
Version: master   
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:
Attachments: Bug 9310 - Patron image upload template corrections
[SIGNED-OFF] Bug 9310 - Patron image upload template corrections
Bug 9310 - Patron image upload template corrections

Description Owen Leonard 2012-12-20 14:49:43 UTC
The patron image upload template needs a few corrections:

- incorrect grid structure makes page narrower than it needs to be
- information about upload results squeezed into message box illegibly
- upload results should link to patron records
Comment 1 Owen Leonard 2012-12-20 14:56:31 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2012-12-29 09:28:50 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2013-01-15 13:01:18 UTC
QA comment:

This patch improves the ergonomic of the patron image upload page.
Works as expected for zip and png files.

Marked as Passed QA.
Comment 4 Jonathan Druart 2013-01-15 13:01:40 UTC
Created attachment 14598 [details] [review]
Bug 9310 - Patron image upload template corrections

This patch fixes some template structure problems and makes some
improvements:

- Correct grid structure so that page isn't narrower than it needs
  to be.
- Move image upload messages out of message/error dialog and into
  table so that lines are distinct and legible.
- Expand breadcrumbs specificity
- Capitalization corrections

To test: Upload patron images using both single images and zip files.
Test zip file upload with a file which contains valid and invalid
contents (non-existant patron numbers, invalid image files, etc). In all
cases image uploads should work correctly and errors should be legibly
displayed.

Signed-off-by: Marc Veron <veron@veron.ch>

With patch, error messages are displayed  in a nice table.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Tested with zip and png files. Works great.
Comment 5 Jared Camins-Esakov 2013-01-18 02:53:36 UTC
This patch has been pushed to master.
Comment 6 Chris Cormack 2013-02-01 08:03:39 UTC
Pushed to 3.10.x and 3.8.x, will be in 3.10.3 and 3.8.10