I recently made use of a button with #type image_button and the ajax module didn't recognize it as a submit button.

Attached is a patch to fix this.

CommentFileSizeAuthor
#5 ajax_type=image.patch425 bytesMaxkupr
#4 ajax_type=image.patch418 bytesMaxkupr
ajax_image_button_fix.patch559 bytesvoxpelli

Comments

iamjon’s picture

iamjon commented

Hi,
I tried your patch it didn't work.
Can you post your your form api that you used to add the image. Maybe I didn't override the button properly.

iamjon’s picture

iamjon commented

I lied it works.

voxpelli’s picture

voxpelli commented
Status: Needs review » Reviewed & tested by the community

If it works - then we can perhaps set this as reviewed by the community? It's such a simple patch that it shouldn't need a massive review

Maxkupr’s picture

Maxkupr commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new ajax_type=image.patch418 bytes

what about adding handler in ajax.js?

Maxkupr’s picture

Maxkupr commented
StatusFileSize
new ajax_type=image.patch425 bytes

sorry, just discard previous patch. here is correct one

brendoncrawford’s picture

brendoncrawford commented
Status: Needs review » Closed (fixed)

Maximus, patch has been applied. Thanks for your help. Should be available in dev release.

brendoncrawford’s picture

brendoncrawford commented
Status: Closed (fixed) » Fixed

Status: Fixed » Closed (fixed)

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