No Alt text on images no longer alerts user

silverstripe / cwp-core

CWP basic compatibility module

BSD 3-Clause "New" or "Revised" License

3 stars 12 forks source link

No Alt text on images no longer alerts user #55

Closed gorriecoe closed 5 years ago

gorriecoe commented 5 years ago

In ss3 admin a user would get alerted if alt text wasn't defined. This functionality isn't working with ss4. Is this likely to be implemented? I see the code is still in the repo but is using entwine which I assume no longer works with assets admin.

Alert message

{{ Image name }} has no alt text which is required for non-decorative images.

Are you sure you want to update?

robbieaverill commented 5 years ago

You're probably right. I'll triage this as a high impact bug (due to missing functionality) and suggest that we add a Behat test to cover this functionality to prevent future regressions

robbieaverill commented 5 years ago

Closing as a duplicate of https://github.com/silverstripe/cwp/issues/149