Created attachment 181654 [details] [review] Bug 39774: Add id to wrapper This makes it so that the established styling (seen from request details page on OPAC, authenticated) is also applied here. To test, before applying patch: 1) Enabled ILLModule and ILLOpacUnauthenticatedRequest 2) Create an unauth OPAC request: logout and visit: http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?backend=Standard&op=add_form 3) Enter the required fields, select type = 'Chapter', add some metadata info such as 'Place of publication', 'Publisher' and others. 4) Notice how the 'Request details' table is rendered, 'Place of publication' and 'Publisher' and misaligned. Apply patch. Refresh.
Created attachment 181664 [details] [review] Bug 39774: Add id to wrapper This makes it so that the established styling (seen from request details page on OPAC, authenticated) is also applied here. To test, before applying patch: 1) Enabled ILLModule and ILLOpacUnauthenticatedRequest 2) Create an unauth OPAC request: logout and visit: http://localhost:8080/cgi-bin/koha/opac-illrequests.pl?backend=Standard&op=add_form 3) Enter the required fields, select type = 'Chapter', add some metadata info such as 'Place of publication', 'Publisher' and others. 4) Notice how the 'Request details' table is rendered, 'Place of publication' and 'Publisher' and misaligned. Apply patch. Refresh. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 181665 [details] Screenshot - ILL request page after the patch I've signed off, as this improves the page. Is the data supposed to be aligned in the centre and formatted as a table, or should it be like the rest of the request above? (See screenshot.)
Created attachment 181667 [details] OPAC logged in request page (In reply to David Nind from comment #3) > I've signed off, as this improves the page. > > Is the data supposed to be aligned in the centre and formatted as a table, > or should it be like the rest of the request above? (See screenshot.) Thank you David. The purpose of the patch is to bring the unauthenticated request page in line with the regular authenticated one (submitted as attachment). Further improvements can be discussed but that was not my goal with that patch specifically.
(In reply to Pedro Amorim from comment #4) > Created attachment 181667 [details] > OPAC logged in request page > > (In reply to David Nind from comment #3) > > I've signed off, as this improves the page. > > > > Is the data supposed to be aligned in the centre and formatted as a table, > > or should it be like the rest of the request above? (See screenshot.) > > Thank you David. > > The purpose of the patch is to bring the unauthenticated request page in > line with the regular authenticated one (submitted as attachment). > Further improvements can be discussed but that was not my goal with that > patch specifically. That makes sense! Thanks. David