Taking the liberty of calling this critical.

At the moment if you click 'Add media' in a wysiwyg editor, you (the end user) get told: "Allowed file types: jpg jpeg gif png txt doc xls pdf ppt pps odt ods odp mp3 mov m4v mp4 mpeg avi ogg wmv ico."

This is not true. At the moment, as far as I know, it is not possible for it to be true until #1016376: Expose the media types allowed for WYSIWYG variable in the UI is fixed.

I don't think a 1.0 release should go out claiming functionality that isn't in it, so I think that wording should be taken out.

Not quite sure what is supported though.

Comments

JacobSingh’s picture

JacobSingh commented
Priority: Critical » Normal

Right, the difficulty is you are actually only allowed video and images because that is all that is supported. What extensions are allowed is a matter of how those media types are configured. So there are multiple (annoying) levels of validation. You've got the base list of extensions allowed for upload #1049446: Provide a settings form for general media settings including media dialog theme and file extensions allows you to configure this.

Then you've got the fact that when the WYSIWYG browser is launched, we pass in that only media of type "image" and "video" are allowed. Those are configured in the media types.

I'm downgrading the severity since it is bad but not atrocious. Please suggest an alternate UI for this.

cyberwolf’s picture

cyberwolf commented

Subscribing.

JacobSingh’s picture

JacobSingh commented
Version: 7.x-1.0-beta3 » 7.x-2.x-dev

This is likely to not improve in 7.x-1.x. I'm moving to 7.x-2.x to make sure we remember to not make this confusing there.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Component: Code » WYSIWYG integration
mpgeek’s picture

mpgeek commented
Status: Active » Closed (duplicate)

#1434118: Make a Separate Config for WYSIWYG's Media Browser and #1016376: Expose the media types allowed for WYSIWYG variable in the UI cover this issue. Marking duplicate.