| Summary: | Group and label vendor contact settings | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
| Component: | Templates | Assignee: | Owen Leonard <oleonard> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | fridolin.somers |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Small patch |
| Documentation contact: | Caroline Cyr La Rose | Documentation submission: | https://gitlab.com/koha-community/koha-manual/-/merge_requests/750 |
| Text to go in the release notes: | Version(s) released in: |
23.11.00
|
|
| Circulation function: | |||
| Attachments: |
Bug 34197: Group and label vendor contact settings
Bug 34197: Group and label vendor contact settings Bug 34197: Group and label vendor contact settings |
||
|
Description
Caroline Cyr La Rose
2023-07-04 20:48:01 UTC
Created attachment 153650 [details] [review] Bug 34197: Group and label vendor contact settings This patch changes the layout of vendor contact settings, both in the entry form and in the detail view. Now acquisitions-specific contact settings and serials-specific contact settings are grouped together and labeled as such. The patch also corrects some markup to make the display more consistent. JS handling of some labels has been removed because the structure change eliminated the need for a "for" attribute. The patch also makes a minor unrelated change to the display of discount percentages to ensure that a blank value doesn't display. To test, apply the patch and go to Acquisitions -> New vendor. - In the vendor entry form, in the "Contacts" section, you should see two groups of checkboxes, "Acquisitions options" and "Serials options." Confirm that the labels can be clicked to check checkboxes. - Test the process of adding another contact and confirm that the additional entry fields work correctly. - Confirm that the "Primary acquisitions contact" and "Primary serials contact" checkboxes act like radio buttons between the two contacts: Checking the box under one contact should uncheck the same box under the other contact. - Confirm that all your data is saved correctly. - Test the page during the edit process as well. - View your updated vendor and confirm that the detail view looks correct. Created attachment 153652 [details] [review] Bug 34197: Group and label vendor contact settings This patch changes the layout of vendor contact settings, both in the entry form and in the detail view. Now acquisitions-specific contact settings and serials-specific contact settings are grouped together and labeled as such. The patch also corrects some markup to make the display more consistent. JS handling of some labels has been removed because the structure change eliminated the need for a "for" attribute. The patch also makes a minor unrelated change to the display of discount percentages to ensure that a blank value doesn't display. To test, apply the patch and go to Acquisitions -> New vendor. - In the vendor entry form, in the "Contacts" section, you should see two groups of checkboxes, "Acquisitions options" and "Serials options." Confirm that the labels can be clicked to check checkboxes. - Test the process of adding another contact and confirm that the additional entry fields work correctly. - Confirm that the "Primary acquisitions contact" and "Primary serials contact" checkboxes act like radio buttons between the two contacts: Checking the box under one contact should uncheck the same box under the other contact. - Confirm that all your data is saved correctly. - Test the page during the edit process as well. - View your updated vendor and confirm that the detail view looks correct. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> I love it! It looks much nicer and I think it's clearer what is what. Created attachment 154119 [details] [review] Bug 34197: Group and label vendor contact settings This patch changes the layout of vendor contact settings, both in the entry form and in the detail view. Now acquisitions-specific contact settings and serials-specific contact settings are grouped together and labeled as such. The patch also corrects some markup to make the display more consistent. JS handling of some labels has been removed because the structure change eliminated the need for a "for" attribute. The patch also makes a minor unrelated change to the display of discount percentages to ensure that a blank value doesn't display. To test, apply the patch and go to Acquisitions -> New vendor. - In the vendor entry form, in the "Contacts" section, you should see two groups of checkboxes, "Acquisitions options" and "Serials options." Confirm that the labels can be clicked to check checkboxes. - Test the process of adding another contact and confirm that the additional entry fields work correctly. - Confirm that the "Primary acquisitions contact" and "Primary serials contact" checkboxes act like radio buttons between the two contacts: Checking the box under one contact should uncheck the same box under the other contact. - Confirm that all your data is saved correctly. - Test the page during the edit process as well. - View your updated vendor and confirm that the detail view looks correct. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 23.11. Nice work everyone, thanks! Enhancement not pushed to 23.05.x Screenshot was updated in the manual. |