Summary: | The mandatory fields validation popup doesn't appear in IE8 | ||
---|---|---|---|
Product: | Koha | Reporter: | Frédérick Capovilla <fcapovilla> |
Component: | Browser compatibility | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, koha.sekjal, paul.poulain |
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: | ||
Circulation function: | |||
Attachments: |
Corrects a validation problem on addbiblio.pl on IE8
Signed-off patch proposed patch (passed QA) |
Description
Frédérick Capovilla
2011-05-19 14:58:49 UTC
Comment hidden (obsolete)
Created attachment 4192 [details] [review] Signed-off patch Signed-off patch includes one minor edit: I removed the commented-out line which the jQuery line replaces. We have git to track changes for us, we don't have to leave old code in. This patch fixes the problem, which I was able to reproduce in IE8 on Win7. Just a detail : The commented line doesn't seem to have been removed in the signed-off patch... > The commented line doesn't seem to have been removed in the signed-off patch...
Indeed. My git-fu must be failing me today.
Created attachment 4961 [details] [review] proposed patch (passed QA) QA comment As for previous patch for IE8 * Koha not guaranteed to work on IE * I don't have any IE at hand * I've reintroduced the comment owen forgot to upload * I've checked the patch is harmless under FF * Frédérick & Owen says it worked for them Marking passed QA Pushed, please test There have been no further reports of problems so I am marking this bug resolved. |