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/

CommentFileSizeAuthor
#2 0001-Bugfix-closing-wrapper-div.-http-drupal.org-node.patch1.3 KBmikl

Comments

Ozeuss’s picture

Ozeuss commented

You're right, there is indeed a missing closing

. Somehow FF managed to hide it. Will be commited soon

mikl’s picture

mikl
Location Møn
commented
Assigned: Unassigned » mikl
Status: Active » Needs review
StatusFileSize
new 0001-Bugfix-closing-wrapper-div.-http-drupal.org-node.patch1.3 KB

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 :)

Ozeuss’s picture

Ozeuss commented
Status: Needs review » Fixed

Thanks, I've implemented this with a slight modification.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.