It would be better to change the structure of opac-results.tt and catalogue/results.tt by replacing "table" elements with a more ergonomic structure (like div, span, etc.)
Created attachment 9956 [details] [review] Bug 8199 : Replace table in opac-results with div Test plan: launch a query from your opac search form and compare results before and after applying this patch. Normally, You don't show a lot of differences.
This patch is a first draft. I would like to have feed back about this patch before to continue (and to adapt to catalogue/results.tt)
I don't think there's anything inherently more ergonomic about showing search results in divs instead of tables, so I'd be interested in seeing a test case that shows how this makes things more flexible. Is the goal to be able to rearrange the elements somehow?
Created attachment 10247 [details] [review] Bug 8199 : Replace table in opac-results with div
Hi Owen, For example: currently (with the table structure) it is not possible to move the image from right to left. With this patch, we can filled the OPACUserCSS syspref with: .result > input { float: none; } .result > span { float: none; margin-left: 5px; } .all_results > div > span.biblio_infos { display: block; max-width: 80%; padding-left: 120px; margin-top: -15px; } .result > span.image { margin-left: 30px; margin-top: -70px; position: absolute; }
I noticed two problems with this: 1. The numbers on the left are run into the main results block. 2. Some of the images (possibly local cover images?) wrap to the next line.
I don't plan to work on it. Reopen if someone is interested.