Bug 32076

Summary: Form for editing a basket group is misaligned
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: AcquisitionsAssignee: Lucas Gass <lucas>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: normal    
Priority: P5 - low CC: lucas
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:
22.11.00
Bug Depends on: 30952    
Bug Blocks:    
Attachments: Bug 32076: Add page-section to basketgroup.tt and clean-up markup
Bug 32076: Add page-section to basketgroup.tt and clean-up markup
Bug 32076: Add page-section to basketgroup.tt and clean-up markup

Description Katrin Fischer 2022-11-02 14:45:23 UTC
The input fields on the edit form for basket groups are misaligned and the 'ungrouped baskets' on the right should probably be page-sectioned.
Comment 1 Lucas Gass 2022-11-02 19:01:07 UTC
Created attachment 143026 [details] [review]
Bug 32076: Add page-section to basketgroup.tt and clean-up markup

This patch adds the page-section div to the basket group 'Ungrouped baskets'. It also adds the row class to the fieldset to get the list to display properly.

To test:
1. Apply patch
2. Make sure the page section div for 'Ungrouped baskets' looks good.
3. Look at the markup changes and make sure the make sense.
4. Make sure the form still works as it should.
Comment 2 David Nind 2022-11-03 01:53:58 UTC
Created attachment 143055 [details] [review]
Bug 32076: Add page-section to basketgroup.tt and clean-up markup

This patch adds the page-section div to the basket group 'Ungrouped baskets'. It also adds the row class to the fieldset to get the list to display properly.

To test:
1. Apply patch
2. Make sure the page section div for 'Ungrouped baskets' looks good.
3. Look at the markup changes and make sure the make sense.
4. Make sure the form still works as it should.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Katrin Fischer 2022-11-03 09:36:35 UTC
Created attachment 143068 [details] [review]
Bug 32076: Add page-section to basketgroup.tt and clean-up markup

This patch adds the page-section div to the basket group 'Ungrouped baskets'. It also adds the row class to the fieldset to get the list to display properly.

To test:
1. Apply patch
2. Make sure the page section div for 'Ungrouped baskets' looks good.
3. Look at the markup changes and make sure the make sense.
4. Make sure the form still works as it should.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Katrin Fischer 2022-11-03 09:39:59 UTC
Nice fix, thx Lucas. Could you maybe take a look at bug 31819?
Comment 5 Tomás Cohen Arazi 2022-11-03 14:32:29 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!