| Summary: | Search results are contained into a table html element | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
| Component: | Templates | Assignee: | Jonathan Druart <jonathan.druart> |
| Status: | CLOSED WONTFIX | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | P5 - low | ||
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 8199 : Replace table in opac-results with div
Bug 8199 : Replace table in opac-results with div |
||
|
Description
Jonathan Druart
2012-06-06 14:29:26 UTC
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. |