Provide field name when calling insertIntoActiveEditor

Thank you for contributing this patch. I have deployed it in conjunction with #2085699: Insert module support only uses settings from first image field when there is more than one image field in the content type.. It tests successful for me.

Form the perspective of the Insert module, the change seems to be out of place. The EIM module should rather inject the required information by itself. The Insert button could just be disabled as long as the alt field is not filled. (This also is the behaviour of in the module's D8 version where the option to require alt field input is part of Drupal core.)

Anyway, closing this ticket as the EIM module seems to be sort of abandoned, most recent change was five years ago.

I appreciate your reply. Since I am not the original D7 developer, I had another look at the code and I see the issue now. In the sense of separation of concerns I would not extend the event options by adding processing not directly related to the Insert module, as that actually makes the interface less robust. But I wonder why the insertIntoActiveEditor event is not triggered on a meaningful node (which is what I had expected). Indeed, currently, there is no way to access the field environment.

Triggering the event on the Insert button would allow event handlers to easily access the field environment. In my opinion, that would be a sensible solution in that particular case and an improvement in any case. Now, in the event handler, event.target could be used to access the field environment crawling the DOM. If you are fine with that change, I will merge it in.

OK, thanks!