Bug 21505

Summary: Box around 'Additional fields' does not contain the fields
Product: Koha Reporter: Caroline Cyr La Rose <caroline.cyr-la-rose>
Component: SerialsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: colin.campbell, fridolin.somers, josef.moravec, katrin.fischer, martin.renvoize, nick
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:
Bug Depends on: 21523    
Bug Blocks:    
Attachments: Additional fields box
Bug 21505: Box around 'Additional fields' does not contain the fields
Bug 21505: Box around 'Additional fields' does not contain the fields
Bug 21505: Box around 'Additional fields' does not contain the fields

Description Caroline Cyr La Rose 2018-10-05 19:24:12 UTC
Created attachment 80147 [details]
Additional fields box

When you have additional fields in your subscription, the 'Additional fields' title appears in a blue box, but the fields themselves are outside the box.(See attached screen capture)

To reproduce:
1) Go to Serials
2) Click on 'Add subscription fields'
3) Click on 'New fields'
4) Fill in the form and click 'Save'
5) Search for a subscription
6) Click on the title to see the subscription detail page
7) In the 'Information' tab, there is the 'Additional fields' title in its box followed by the additional field
Comment 1 Caroline Cyr La Rose 2018-10-05 19:25:22 UTC
Maybe also change the 'New fields' for 'New field'. All the 'New ...' buttons are singular, and we only add one field at a time.
Comment 2 Owen Leonard 2018-10-09 18:19:02 UTC
Created attachment 80303 [details] [review]
Bug 21505: Box around 'Additional fields' does not contain the fields

This patch modifies the subscription detail page markup to eliminate a
confusing bordered <legend> tag. It is replaced with an <hr> to
delineate the sections and an <h3> tag to serve as the heading.

To test, apply the patch and regenerate the staff client CSS.

 - Configure one or more additional subscription fields.
 - View the detail page for a subscription and confirm that the
   "Additional fields" section is clearly labeled.
Comment 3 Owen Leonard 2018-10-10 11:59:46 UTC
(In reply to Caroline Cyr La Rose from comment #1)
> Maybe also change the 'New fields' for 'New field'. All the 'New ...'
> buttons are singular, and we only add one field at a time.

I created Bug 21531 for this issue.
Comment 4 Claire Gravely 2018-10-11 10:09:00 UTC
Created attachment 80392 [details] [review]
Bug 21505: Box around 'Additional fields' does not contain the fields

This patch modifies the subscription detail page markup to eliminate a
confusing bordered <legend> tag. It is replaced with an <hr> to
delineate the sections and an <h3> tag to serve as the heading.

To test, apply the patch and regenerate the staff client CSS.

 - Configure one or more additional subscription fields.
 - View the detail page for a subscription and confirm that the
   "Additional fields" section is clearly labeled.

Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>
Comment 5 Katrin Fischer 2018-10-11 20:10:39 UTC
I like this much better, thx!
Comment 6 Katrin Fischer 2018-10-11 20:11:12 UTC
Created attachment 80423 [details] [review]
Bug 21505: Box around 'Additional fields' does not contain the fields

This patch modifies the subscription detail page markup to eliminate a
confusing bordered <legend> tag. It is replaced with an <hr> to
delineate the sections and an <h3> tag to serve as the heading.

To test, apply the patch and regenerate the staff client CSS.

 - Configure one or more additional subscription fields.
 - View the detail page for a subscription and confirm that the
   "Additional fields" section is clearly labeled.

Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Nick Clemens 2018-10-19 17:33:10 UTC
Awesome work all!

Pushed to master for 18.11
Comment 8 Martin Renvoize 2018-11-06 10:15:51 UTC
Caused by bug 21523 not in 18.05.x series.
Comment 9 Martin Renvoize 2018-11-06 11:00:59 UTC
Ignore that.. the error is present in 18.05.x series too.. backporting now.
Comment 10 Martin Renvoize 2018-11-06 11:08:36 UTC
Pushed to 18.05.x for 18.05.06
Comment 11 Fridolin Somers 2018-11-28 09:31:04 UTC
I prefer not to change display in 17.11.x