Missing </div> causes layout troubles in Safari
mikl - March 25, 2009 - 20:34
| Project: | Office Hours |
| Version: | 6.x-1.x-dev |
| Component: | User interface |
| Category: | bug report |
| Priority: | normal |
| Assigned: | mikl |
| Status: | closed |
Description
I've been trying out this module for a project, and I had some issues editing the CCK field in Safari.
Basically, most of the form and half the page was missing:
http://www.quicksnapper.com/mikl/image/untitled-0000
I figured it was a mismatched tag problem, and so I ran it through the HTML validator at http://validator.w3.org/. Here's what it found:
http://www.quicksnapper.com/mikl/full/untitled/
#1
You're right, there is indeed a missing closing . Somehow FF managed to hide it. Will be commited soon
#2
I've made a patch for this, using #prefix and #suffix.
In my opinion that is a safer way to add div-wrappers – if you have it as a form element, form elements can "escape" if their weight is adjusted :)
#3
Thanks, I've implemented this with a slight modification.
#4
Automatically closed -- issue fixed for 2 weeks with no activity.