When there is only one location for a node, the location form puts the edit form fieldset in another fieldset with no other content. They also both have the same name by default, so the outer fieldset is just clutter. The attached patch does not use the outer fieldset when the node being edited only allows one location.

CommentFileSizeAuthor
#2 remove_extra_location_fieldset_box.patch1.23 KBcolan
remove_nest_in_location_form.patch614 bytessmulkey

Comments

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented

I marked http://drupal.org/node/108654 as a duplicate of this.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented
StatusFileSize
new remove_extra_location_fieldset_box.patch1.23 KB

That above patch didn't work so well for me out of the box, so I came up with this one instead.

scottrigby’s picture

scottrigby
he/him
Primary language English
Location Brooklyn, NY
commented

great! thanks
Will this be added to next version?

bdragon’s picture

bdragon commented
Title: Fix to single location form display » [master] Fix to single location form display

http://drupal.org/node/211695 marked duplicate.
Marked as master issue.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community

I've been using my above patch for 6 months now (which hasn't given me any trouble), and the only comment so far has been a "thank you" so I think this one should be good to go. Can we commit it?

brmassa’s picture

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

Colan,

its fixed on HEAD. Soon on Location 3

regards,

massa

Anonymous’s picture

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

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