Bug 35253 made the materials specified note easier to customize. However I failed to realized that the message is slightly different if 'CircConfirmItemParts' is set to 'Require'. That message should also include the HTML classes added for Bug 35253.
Created attachment 162069 [details] [review] Bug 36074: Add classes to materials specfied message on check in and checkout when CircConfirmItemParts is on. To test: 1. APPLY PATCH 2. Set CircConfirmItemParts to 'Require'. 3. Add a materials specified message to an item. ( 952$3 ) 4. Add the following CSS to your IntranetUserCSS: .mats_spec_label { color: white; background: purple; } .mats_spec_message { color: white; background: green; } 5. Checkout that item. Notice the message should be green and the label (Note about the accompanying materials:) should be purple. 6. Check in that item. Notice the message should be green and the label (Note about the accompanying materials:) should be purple.
Created attachment 162122 [details] [review] Bug 36074: Add classes to materials specfied message on check in and checkout when CircConfirmItemParts is on. To test: 1. APPLY PATCH 2. Set CircConfirmItemParts to 'Require'. 3. Add a materials specified message to an item. ( 952$3 ) 4. Add the following CSS to your IntranetUserCSS: .mats_spec_label { color: white; background: purple; } .mats_spec_message { color: white; background: green; } 5. Checkout that item. Notice the message should be green and the label (Note about the accompanying materials:) should be purple. 6. Check in that item. Notice the message should be green and the label (Note about the accompanying materials:) should be purple. Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Only adds CSS classes, reviewed closely.
(In reply to Katrin Fischer from comment #3) > Only adds CSS classes, reviewed closely. This was meant for another bug. Still only small mark-up changes here. Removed some additional spaces: <span class="mats_spec_label"> Please confirm that the accompanying materials are present: </span>
Pushed for 24.05! Well done everyone, thank you!
I see one last additional space : ... are present:</span> ... are present: </span> Removing it will allow to have online 1 translation :D
Pushed to 23.11.x for 23.11.06
Enhancement will not be backported to 23.05.x