Description
Lucas Gass (lukeg)
2023-05-16 17:38:32 UTC
Created attachment 151315 [details] [review] Bug 33748: Fix UI issue when importing records from a staged MARC file To test: 1. Set MarcItemFieldsToOrder like this: homebranch: 975$a holdingbranch: 975$b itype: 975$y nonpublic_note: 975$x public_note: 975$z loc: 975$c ccode: 975$8 notforloan: 975$7 quantity: 975$q price: 975$g replacementprice: 975$v uri: 975$u 2. Set MarcFieldsToOrder like this: price: 975$p quantity: 975$q budget_code: 975$h 3. Stage a MARC file that has bibs with items. 4. From acquisitions create a new basket and add 'From a staged file' 5. APPLY PATCH 6. Try 3 - 4 again, the display should now look correct. The patch did help improve the style but I see several issues: In 22.05 the data for Item record 1, item record 2, etc. displayed on the right side of the screen. It was nice to be able to visually connect the item data with the order data displayed directly to the left. It also makes good use of screen real estate and decreases the amount of scrolling down to the bottom in order to save. In 22.05 if there was no matching record nothing was displayed. In 22.11 the word Match: displays with no additional info since there is no match. It seems like this is extraneous and should be removed for non-matches. I attached two screenshots for the display in 22.05 and two for how it is now displaying in 22.11. Created attachment 151323 [details]
22.05 match
Created attachment 151324 [details]
22.05 no match
Created attachment 151325 [details]
22.11 style match
Created attachment 151326 [details]
22.11 style no match
Created attachment 151329 [details] [review] Bug 33748: Fix UI issue when importing records from a staged MARC file Barbara, Can you test my new patch? The display is much better now. Here are several things to take a look at: The three action buttons for the items are sitting right on top of a thin border line. Some space between the buttons would look better. Or, is the line needed at all? Since the item labels are of different widths it makes the input boxes display in a staggered fashion. It would be easier to view if the labels were all one width so that the input boxes could align. It was that way for me in 22.05 but it's possible I put some code in place to do that. If there is no match for the bib then 'Match:' is just hanging above the order info with no purpose. It did not display at all in 22.05 if there was no match. When clicking on the 'Item information' tab the edit_item_form does not display. I've attached a few screenshots. Created attachment 151345 [details]
Original display after updated patch
Created attachment 151346 [details]
Display after suggested tweaks
Created attachment 151347 [details]
Missing edit_item_form
Created attachment 151713 [details] [review] Bug 33748: Fix UI issue when importing records from a staged MARC file This patch attempts to fix a UI issue on addorderiso2709.tt. It removes the <td> which contains the actoin buttons MARC, Card, and Add Order and adds them to the title line. This is so there is more space for the fieldsets. 1. Apply patch 2. Set MarcItemFieldsToOrder like this: homebranch: 975$a holdingbranch: 975$b itype: 975$y nonpublic_note: 975$x public_note: 975$z loc: 975$c ccode: 975$8 notforloan: 975$7 quantity: 975$q price: 975$g replacementprice: 975$v uri: 975$u 3. Stage a MARC file where the bibs have items attached 4. From acquisitions create a new basket and add 'From a staged file'. 5. Add the staged files to a basket. 6. Make sure the display looks correct. 7. Expand the data by clicking on the checkbox on the left hand side, or by clicking 'Select all'. 8. Make sure the display looks correct. 9. Tab over to 'Item informtion' and make sure that display looks correct. 10. Shrink the screen size down to less that 992px, ensuring the display remains correct. Created attachment 151714 [details] [review] Bug 33748: Fix UI issue when importing records from a staged MARC file This patch attempts to fix a UI issue on addorderiso2709.tt. It removes the <td> which contains the actoin buttons MARC, Card, and Add Order and adds them to the title line. This is so there is more space for the fieldsets. 1. Apply patch 2. Set MarcItemFieldsToOrder like this: homebranch: 975$a holdingbranch: 975$b itype: 975$y nonpublic_note: 975$x public_note: 975$z loc: 975$c ccode: 975$8 notforloan: 975$7 quantity: 975$q price: 975$g replacementprice: 975$v uri: 975$u 3. Stage a MARC file where the bibs have items attached 4. From acquisitions create a new basket and add 'From a staged file'. 5. Add the staged files to a basket. 6. Make sure the display looks correct. 7. Expand the data by clicking on the checkbox on the left hand side, or by clicking 'Select all'. 8. Make sure the display looks correct. 9. Tab over to 'Item informtion' and make sure that display looks correct. 10. Shrink the screen size down to less that 992px, ensuring the display remains correct. Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov> Created attachment 151896 [details] [review] Bug 33748: Fix UI issue when importing records from a staged MARC file This patch attempts to fix a UI issue on addorderiso2709.tt. It removes the <td> which contains the actoin buttons MARC, Card, and Add Order and adds them to the title line. This is so there is more space for the fieldsets. 1. Apply patch 2. Set MarcItemFieldsToOrder like this: homebranch: 975$a holdingbranch: 975$b itype: 975$y nonpublic_note: 975$x public_note: 975$z loc: 975$c ccode: 975$8 notforloan: 975$7 quantity: 975$q price: 975$g replacementprice: 975$v uri: 975$u 3. Stage a MARC file where the bibs have items attached 4. From acquisitions create a new basket and add 'From a staged file'. 5. Add the staged files to a basket. 6. Make sure the display looks correct. 7. Expand the data by clicking on the checkbox on the left hand side, or by clicking 'Select all'. 8. Make sure the display looks correct. 9. Tab over to 'Item informtion' and make sure that display looks correct. 10. Shrink the screen size down to less that 992px, ensuring the display remains correct. Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Pushed to master for 23.11. Nice work everyone, thanks! Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to oldstable for 22.11.x |