Make a webform with a file widget. Submit a test, go to results, view. Validate the source code and you'll notice that the last div closing tag is missing. Tends to wreak havok on certain layouts.

CommentFileSizeAuthor
#1 file.inc_.txt676 bytesphl3tch

Comments

phl3tch’s picture

phl3tch commented
Assigned: Unassigned » phl3tch
Status: Active » Needs review
StatusFileSize
new file.inc_.txt676 bytes

Uh, well, that was a pretty simple fix. Patch attached. I've marked it code needs review because I'm not real familiar with the diff tool and the process of submitting patches. Someone let me know if I'm off the mark.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

Thanks! Nice catch. Nice looking patch too. It's good etiquette to not mark your own patches RTBC (ready to be committed) anyway, so good job all around :)

Committed and will be in 1.5.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)