Media picker: Required fields not enforced when media is uploaded inline

PeterBridgerZone commented 5 years ago

A brief description of the issue goes here.

Reproduction

If you're filing a bug, please describe how to reproduce it. Include as much relevant information as possible, such as:

Bug summary

It is possible to setup a Media Type with a required field. This is correctly enforced when media is then uploaded within the media section.

However when media is uploaded inline using the media picker, required fields are not enforced.

Specifics

Using Umbraco 7.12.3
Using Chrome on Windows 10

Steps to reproduce

Create new Media Type with a required fields
Use a Media Picker on a content item
Select the Upload option on the Media Picker
Select the media item to upload
Media item will be uploaded and added to the media library, even though the required field was left blank

Expected result

I would expect the upload to be aborted at the very least, if not allowing the user to specify inputs for the required fields.

Actual result

The media item is successfully added to the media library, with the required fields left blank,

nul800sebastiaan commented 5 years ago

I would expect the upload to be aborted at the very least,

That would not be a very nice experience.. I tried to do something and it turns out it's impossible.

if not allowing the user to specify inputs for the required fields.

We currently don't have the ability of adding custom fields to a media picker's upload field (or media picker in general). I am not even sure how this could work (yes, I can think of ways it would work, but would it be good UX?).

To better understand the requirement here: what exactly needs to be present in that mandatory field?

PeterBridgerZone commented 5 years ago

In this particular scenario, the client is requesting that we make the alt text a required field on images.

So it's a textstring property.

nul800sebastiaan commented 5 years ago

Hi @PeterBridgerZone - We've discussed this at HQ and for now we'll put it on the ideas list for when we overhaul how to work with media in Umbraco, it's a good idea, now to figure out how to implement it nicely, thanks!

GeorgeBR commented 4 years ago

I think this is an important feature. In Europe we now have legal requirements for accessible websites, and alt text is now compulsory for images. You can add an alt text field to a media item, but it is not validated when users upload media. It would be a really useful feature to enforce alt text - or even stop the use of images that don't have an alt text property from being selected

umbrabot commented 2 years ago

Hi there @PeterBridgerZone,

Just wanted to let you know that we noticed that this issue got a bit stale and we haven't been able to get to this idea. We will close this idea for now, as we haven't been able to prioritize it yet.

Once we get time to work on ideas that are in this category we'll review and update existing issues like this one to let you know we're working on it.

Thanks, from your friendly Umbraco GitHub bot :robot: :slightly_smiling_face:

umbraco / Umbraco-CMS