Property constraints validation adding errors to the field element instead of just the property

I uploaded the wrong screenshot, but you get the idea.

Added failing test, which failed as expected.

Probably related:

🐛 The Inline form error of a required, limited multiple file widget is rendered twice Needs work

Left some small comments on the MR

If you are another contributor eager to jump in, please allow the previous poster at least 48 hours to respond to feedback first, so they have the opportunity to finish what they started!

Thanks for the review, resolved comments.

Feedback appears to be addressed.

Leaving the one thread open for committers but concern of causing breakage is there. Maybe a follow up to come up with a generic approach?

I wondered whether this change could break any existing tests or workflows, and so decided to err on the side of caution and only commit this to 11.x. If anyone feels strongly that this should be backported please reopen.

Committed 2be9363 and pushed to 11.x. Thanks!

longwave → closed merge request !11291

Thanks! Added 🐛 Property constraints validation adding errors to the field element instead of just the property Active for exploring other widgets.