Should not show 'Unsaved Changes' after sending tests

[bossanova808]

Hey Ben

Very minor but I think it would be much better UX if Craft didn't show a 'showing unsaved changes' warning every time you send an email test (i.e. when there are no other changes).

Not sure if this is something you can control, though! But I do know it confuses our day to day editors each time the send a test but have not made other changes...

Hope you're well!

[bencroker]

Thanks, I'll take a look.

[bencroker]

Unfortunately, this is currently not possible to prevent. Craft checks whether the value of the serialized form has changed, and since the test email field exists within the form, its value is included in the condition.
Reference: https://github.com/craftcms/cms/blob/b8f30d07cd487c5fbaa140743816e6eb7020e55d/src/web/assets/cp/src/js/ElementEditor.js#L1164C11-L1171

You could create a feature request for this behaviour to be changed in Craft core, so that parts of the DOM can be marked as ignorable, if you like.

[bossanova808]

@bencroker Does Brandon's answer work for you - craftcms/cms#13936 (comment)?

[bencroker]

Yes, I should be able make that work with minimal changes, thanks!

[bencroker]

Implemented in 0df879d for the next release. For the record, that was only happening when adding or removing test contacts, not when sending a test. I generally add default test contacts, which I why I rarely noticed the issue.

You can test this by running composer require "putyourlightson/craft-campaign:dev-develop as 2.10.0".

[bencroker]

Released in 2.10.0.

[bossanova808]

Thanks Ben, appreciated, working as expected/desired now.

However, makes me notice a little bug - based on the text here I have been leaving this blank:

However this always results in:

And the message 'At least one contact must be selected' when hitting the Send Test button (as one would expect).

That should be pre-filled in with my details, shouldn't it?

[bencroker]

Yes, it should. That was only an issue on the campaign edit page but not on the sendout preview page, it seems.

Fixed in 038111e for the next release.

[bencroker]

Released in 2.11.0.