Bug 6437

Summary: Div tag needed in advsearch
Product: Koha Reporter: Elliott Davis <tdavis>
Component: OPACAssignee: Elliott Davis <tdavis>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: enhancement    
Priority: PATCH-Sent (DO NOT USE) CC: chris, katrin.fischer, magnus
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Added div around advsearch form
Signed-off patch

Description Elliott Davis 2011-05-31 17:31:20 UTC
There is no yui-main id in the advsearch tt file.  Needed to size page since forms are not size-able via css.
Comment 1 Elliott Davis 2011-05-31 18:52:13 UTC Comment hidden (obsolete)
Comment 2 Magnus Enger 2011-06-01 06:04:20 UTC
Did you send the patch to the koha-patches list too? http://wiki.koha-community.org/wiki/Version_Control_Using_Git#Share_with_the_rest_of_the_world I'm marking this as "patch sent" and "needs signoff".
Comment 3 Elliott Davis 2011-06-01 13:31:20 UTC
Magnus, I didn't send it to the patch list.  I'm still trying to work some stuff out on my dev servers email.  Also trying to get BZ to work (slef is working on the cookie issue).

Thanks for marking this correctly.
Comment 4 Owen Leonard 2011-06-01 13:40:22 UTC
Created attachment 4331 [details] [review]
Signed-off patch

Patch adds markup correctly, introducing no validation errors.
Comment 5 Chris Cormack 2011-06-02 23:41:20 UTC
Patch pushed,

Congrats on your first patch in Koha Elliot ;)
Comment 6 Katrin Fischer 2011-08-05 19:13:16 UTC
Div tag with id yui-main now present on advanced search page in opac on current master. Marking fixed.