Bug 33798

Summary: Vendor details - improve consistency of edit forms and display
Product: Koha Reporter: David Nind <david>
Component: AcquisitionsAssignee: Jonathan Druart <jonathan.druart>
Status: RESOLVED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: normal    
Priority: P5 - low CC: jonathan.druart, matt.blenkinsop, oleonard
Version: MainKeywords: Academy
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes display errors and improves the consistency of the vendor page and edit forms in acquisitions. Includes adding colons to field labels when adding an interface, making field labels all bold, and addressing some accessibility issues.
Version(s) released in:
23.11.00,23.05.02
Bug Depends on: 33104    
Bug Blocks: 35272    
Attachments: Bug 33798: Add colons after the labels on the edit form
Bug 33798: Improve consistency of the vendor display view
Bug 33798: Add colons after the labels on the edit form
Bug 33798: Improve consistency of the vendor display view
Bug 33798: Improve consistency of the vendor display view

Description David Nind 2023-05-22 20:34:21 UTC
For vendor details in acquisitions, there are some inconsistencies in the display of the edit forms and vendor details.

This could do with some tidy ups.

Specific items:

1. Field labels when displaying vendor details - bold and plain text: for some sections, field labels are in bold, others are just plain text (for example: 'Vendor details' labels are in plain text, for 'Ordering information' they are bold).

2. Edit form for 'Interface details' missing colons after the labels.

2. Accessibility issues: using the WAVE Evaluation Tool browser extension (https://wave.webaim.org/), it shows lots of errors and warnings, such as missing or multiple form labels, and empty buttons and links (some of which may or may not be valid).
Comment 1 Jonathan Druart 2023-06-01 07:09:50 UTC
Created attachment 151893 [details] [review]
Bug 33798: Add colons after the labels on the edit form
Comment 2 Jonathan Druart 2023-06-01 07:09:53 UTC
Created attachment 151894 [details] [review]
Bug 33798: Improve consistency of the vendor display view

It was not consistent: indentation, bold, etc.

This patch is suggesting a new style using .page-section and .rows

Is it what we want?
Comment 3 Jonathan Druart 2023-06-01 07:10:47 UTC
Owen, can I get your opinion on the second patch please?
Comment 4 Owen Leonard 2023-06-01 11:27:00 UTC
(In reply to Jonathan Druart from comment #3)
> Owen, can I get your opinion on the second patch please?

Looks like a good improvement to me!
Comment 5 David Nind 2023-06-06 20:08:09 UTC
Created attachment 152069 [details] [review]
Bug 33798: Add colons after the labels on the edit form

Signed-off-by: David Nind <david@davidnind.com>
Comment 6 David Nind 2023-06-06 20:08:12 UTC
Created attachment 152070 [details] [review]
Bug 33798: Improve consistency of the vendor display view

It was not consistent: indentation, bold, etc.

This patch is suggesting a new style using .page-section and .rows

Is it what we want?

Signed-off-by: David Nind <david@davidnind.com>
Comment 7 David Nind 2023-06-06 20:13:47 UTC
Thanks Jonathan!
Comment 8 Katrin Fischer 2023-06-08 13:15:06 UTC
Created attachment 152191 [details] [review]
Bug 33798: Improve consistency of the vendor display view

It was not consistent: indentation, bold, etc.

This patch is suggesting a new style using .page-section and .rows

Is it what we want?

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 9 Tomás Cohen Arazi 2023-06-09 11:42:49 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 10 Martin Renvoize 2023-07-12 08:24:22 UTC
Thanks for all the hard work!

Pushed to 23.05.x for the next release
Comment 11 Matt Blenkinsop 2023-07-12 09:38:44 UTC
Appears to be dependent on 33104 which has not been backported to 22.11.x - not backporting on this basis