Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.According to accessibility rules the media browser's iframe should have a title and name attribute.
So instead of
<iframe class="media-modal-frame ui-dialog-content ui-widget-content" src="/media/browser?render=media-popup&types%5Bvideo%5D=video&activePlugins=&schemes%5Bvimeo%5D=vimeo&schemes%5Byoutube%5D=youtube&schemes%5Bpublic%5D=0&schemes%5Bprivate%5D=0&file_directory=video&file_extensions=txt&max_filesize=&uri_scheme=public&plugins=undefined" width="800px" id="mediaBrowser" scrolling="auto" style="width: 800px; min-height: 0px; height: 550px;">
it should be something like
<iframe class="media-modal-frame ui-dialog-content ui-widget-content" src="/media/browser?render=media-popup&types%5Bvideo%5D=video&activePlugins=&schemes%5Bvimeo%5D=vimeo&schemes%5Byoutube%5D=youtube&schemes%5Bpublic%5D=0&schemes%5Bprivate%5D=0&file_directory=video&file_extensions=txt&max_filesize=&uri_scheme=public&plugins=undefined" width="800px" id="mediaBrowser" scrolling="auto" style="width: 800px; min-height: 0px; height: 550px;" name="add_media_window" title="Select a media file"></iframe>
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | media-iframe_attributes-2065153-4.patch | 405 bytes | Sneakyvv |
Comments
Comment #1
Sneakyvv CreditAttribution: Sneakyvv commentedpatch added
Comment #3
Sneakyvv CreditAttribution: Sneakyvv commentedresubmitted patch (removed local git repo reference)
Comment #4
Sneakyvv CreditAttribution: Sneakyvv commentedRerolled against 1.5
Comment #5
Sneakyvv CreditAttribution: Sneakyvv commentedComment #6
Chris Matthews CreditAttribution: Chris Matthews as a volunteer commentedClosing this issue as outdated. However, if you think this issue is still important, please let us know and we will gladly re-open it for review.
sincerely,
- the Drupal Media Team