A lot of libraries ask, why there is less information about the material type, format and literary format shown on the detail page than in the result list. I suggest to match both displays and make it possible to hide with CSS as you can already do on the result list (using the same classes).
Created attachment 131294 [details] [review] Bug 20161: add format, material type, etc to staff details page 1. Make sure DisplayIconsXSLT is on. 2. Look at the results page from the staff client. 3. Notice the results_material_type contains 'Material type', 'Format', 'Literary form', 'Type of visual material' 4. Look at the staff detail page and notice that only 'Material type' will show. 5. Apply patch and restart everything 6. Now look at the detail page and it should match what you see on the results page. 7. Turn off DisplayIconsXSLT 8. Make sure nothing on the results_material_type now displays.
This first patch deals with the staff client. A second one to deal with the OPAC will be incoming ASAP.
Created attachment 131303 [details] [review] Bug 20161: add format, material type, etc to staff/OPAC details page For staff side: 1. Make sure DisplayIconsXSLT is on. 2. Look at the results page from the staff client. 3. Notice the results_material_type contains 'Material type', 'Format', 'Literary form', 'Type of visual material' 4. Look at the staff detail page and notice that only 'Material type' will show. 5. Apply patch and restart everything 6. Now look at the detail page and it should match what you see on the results page. 7. Turn off DisplayIconsXSLT 8. Make sure nothing on the results_material_type now displays. For OPAC: 1. Make sure DisplayOPACiconsXSLT is on. 2. Look at the results page from the OPAC. 3. Notice the results_material_type contains 'Material type', 'Format', 'Literary form', 'Type of visual material' 4. Look at the OPAC detail page and notice that only 'Material type' will show. 5. Apply patch and restart everything 6. Now look at the detail page and it should match what you see on the results page. 7. Turn off DisplayOPACiconsXSLT 8. Make sure nothing on the results_material_type now displays.
Hi Lucas, if we want to display the same information everywhere, maybe a new include file (like for the languages) would be a good idea? It would save some copy and paste and make it easier to keep things aligned in the future. If we make it a template, it would also be easy to pass on some specifics like different class names. What do you think?
(In reply to Katrin Fischer from comment #4) > Hi Lucas, > > if we want to display the same information everywhere, maybe a new include > file (like for the languages) would be a good idea? It would save some copy > and paste and make it easier to keep things aligned in the future. > > If we make it a template, it would also be easy to pass on some specifics > like different class names. > > What do you think? I was curious about this, it is A LOT of repeated code. I think it's a good idea and I'll mark this one as FQA now because of that.