Closed (won't fix)
Project:
Field Group
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
1 Dec 2013 at 08:59 UTC
Updated:
1 Jun 2024 at 14:16 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
balintbrewsSimple patch that appends the submit button after the multipage buttons are in place.
Comment #2
balintbrewsLet's fix some whitespace issues in the patch.
Btw, it still applies with 1.4, too, and I still think it's a good idea to have this committed to the module. :)
Comment #4
bverc commentedAgreed this should be done. Failed to apply against -dev. Rerolled against latest -dev. Still needs review.
Comment #6
bverc commentedSame as before except I've changed the multipage-button SPAN to a DIV and added form-actions class.
I changed the identifier of the submit buttons from .form-actions to #edit-actions
Comment #7
bverc commentedComment #11
bverc commentedAlright, another attempt.
Comment #13
bverc commentedComment #15
bverc commentedComment #16
bverc commentedComment #18
bverc commentedLooks like I can't get it to pass. Applies fine though. Marked as to not test to change status to Needs Review.
Comment #19
bverc commentedForgot to change the selector in most recent patch.
Comment #20
Richard15 commentedI changed line 33 of multipage.js with this code:
$('.form-actions', $form).remove().appendTo($($controls.parent(), $panes.last()));
simply adding ".parent()" to $controls
It seems working!
Comment #21
chris matthews commentedThe 3 year old patch in #19 to multipage.js does not apply to the latest field_group 7.x-1.x-dev and needs a reroll (if still applicable).
Comment #22
nils.destoop commented