There is a need for us to target specific form's instead of all forms on the site. I created a patch that allows for this to happen based on the form id. You just have to go into the hide submit button admin page and add a list of ids to target specific forms, and if left empty it will target all forms.

CommentFileSizeAuthor
#16 edit_target_specific_forms-3038715-16.patch5.32 KBsiddhant.bhosale
#11 hide-submit-configuration.png89.59 KBtanmaykadam
#11 node-edit.png84.7 KBtanmaykadam
#11 hide-submit-settings.png63.02 KBtanmaykadam
#9 edit_target_specific_forms-3038715-9.patch5.16 KBsiddhant.bhosale
#5 hide_submit_d8-target_specific_forms-3038715-5.patch9.69 KBsnehalgaikwad
#3 3038715-3-target-specific-forms.patch7.14 KBmarioangulo
#2 3038715-target-specific-forms.patch7.14 KBmarioangulo

Comments

marioangulo created an issue. See original summary.

marioangulo’s picture

marioangulo commented
StatusFileSize
new 3038715-target-specific-forms.patch7.14 KB
marioangulo’s picture

marioangulo commented
StatusFileSize
new 3038715-3-target-specific-forms.patch7.14 KB

Updated method name that creates array of ids to be more specific to what it does.

snehalgaikwad’s picture

snehalgaikwad commented
Assigned: marioangulo » snehalgaikwad
snehalgaikwad’s picture

snehalgaikwad commented
Assigned: snehalgaikwad » Unassigned
Status: Active » Needs review
StatusFileSize
new hide_submit_d8-target_specific_forms-3038715-5.patch9.69 KB

Here is updated patch with option to enable or disable hide submit button functionality for specific forms.

pratik_kamble’s picture

pratik_kamble
Location Pune, India
commented

@snehalgaikwad I was unable to apply the patch. Please use PHP_EOL to create the array of form ids. I think the current implementation of the logic to pass the current form id needs refactoring.

pratik_kamble’s picture

pratik_kamble
Location Pune, India
commented
Status: Needs review » Needs work
siddhant.bhosale’s picture

siddhant.bhosale commented
Assigned: Unassigned » siddhant.bhosale
siddhant.bhosale’s picture

siddhant.bhosale commented
Assigned: siddhant.bhosale » Unassigned
Status: Needs work » Needs review
StatusFileSize
new edit_target_specific_forms-3038715-9.patch5.16 KB

Fixed the mentioned issue.

tanmaykadam’s picture

tanmaykadam commented
Assigned: Unassigned » tanmaykadam
tanmaykadam’s picture

tanmaykadam commented
StatusFileSize
new hide-submit-settings.png63.02 KB
new node-edit.png84.7 KB
new hide-submit-configuration.png89.59 KB

Please add the configuration settings
/admin/config/content/hide-submit

Please add the form specific settings

Please save the node

Button will disappear.

Patch is working properly.

tanmaykadam’s picture

tanmaykadam commented
Assigned: tanmaykadam » Unassigned
tanmaykadam’s picture

tanmaykadam commented
Status: Needs review » Reviewed & tested by the community
nehajyoti’s picture

nehajyoti commented

For #8 - Does this makes sense to change variables to hide_submit_form_ids & hide_submit_negate to keep consistency with all the existing variables?

siddhant.bhosale’s picture

siddhant.bhosale commented
Assigned: Unassigned » siddhant.bhosale
siddhant.bhosale’s picture

siddhant.bhosale commented
StatusFileSize
new edit_target_specific_forms-3038715-16.patch5.32 KB

I have changed the variable names and uploading the patch for the same.

nehajyoti’s picture

nehajyoti commented
Status: Reviewed & tested by the community » Fixed
marioangulo’s picture

marioangulo commented

Status: Fixed » Closed (fixed)

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