Simple patch that will expand the "File Attachments" form_group_collapsible() by default if there are files attached.

CommentFileSizeAuthor
#2 upload.module_2.patch1.19 KBeafarris
upload.module_1.patch1.21 KBeafarris

Comments

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Needs review » Needs work

I think

$has_attachments ? FALSE : TRUE

can be replaced with

!$has_attachments
eafarris’s picture

eafarris commented
Status: Needs work » Needs review
StatusFileSize
new upload.module_2.patch1.19 KB

Agreed. Here's an updated patch.

killes@www.drop.org’s picture

killes@www.drop.org commented
Status: Needs review » Reviewed & tested by the community

Tested, works, please apply. The current behaviour is highly annoying.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed a better solution (I hope) to HEAD.

Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

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