Fix bug where the 'changed' class on the form isn't removed after saving.

[silbinarywolf]

Fix bug where the 'changed' class on the form isn't removed after saving. If the form has a 'changed' class it tells the user that items aren't saved if they try to navigate away from the page.

This removes that class on creation of a gridfield SaveAllButton as the button gets recreated on the save action. Not super across how entwine works so there's probably a better solution, but I figure this will do.

Added to JS:

i(".js-mwm-gridfield-saveall").entwine({
            onmatch: function(){
                // Remove 'changed' state from form anytime a SaveAll button is created.
                i(this[0].form).removeClass('changed');
                this._super();
            }
        }),

[micschk]

Verified & working as it should, would be nice to have this PR merged

[silbinarywolf]

@mi3ll Are you able to action this?

[mellm0]

I don't think this works this way. If there are other fields apart from the GridField, and the save all button removes the changed class from the form, no message will come up when they change page and the changes will be lost.

I have added a flag if you want to have the 'Save All' button remove the changed class instead as a compromise.

Sorry for taking so long to reply.

EDIT: Ah, you probably would need this in ModelAdmin...