I have a multiple file field to upload audio that's using medialement. A big drawback is that the description doesn't show up, something really usefull when i upload more than just one file.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | show_description_of_file_fix_index_warnings-1154196-13.patch | 2.59 KB | skylord |
| #9 | show_description_of_file-1154196-9.patch | 4.93 KB | vchen |
| #7 | show_description_of_file-1154196-7.patch | 4.78 KB | bocaj |
| #2 | show_description_of_file-1154196-0.patch | 1.79 KB | alotaiba |
Comments
Comment #1
dazz commentedsubscribe
Comment #2
alotaiba commentedSorry to bump this old thread back, but I wanted to share my contribution with the rest of the community.
I fixed the issue, because I had a similar need for one of my drupal installations. Here's a patch the shows the file description, if it's set from the admin panel. Also, it wraps it around
<p class="mediaelement-description">{your-file-description}</p>for easier styling.Comment #3
acranius commentedWorked with me, thanks!
Comment #4
traveller commentedThanks alotaiba, works for me.
Comment #5
zipymonkey commentedPatch worked for me as well.
Comment #6
matt bWorked for me
Comment #7
bocaj commentedThis is an updated patch against the latest dev. Also, it provides a setting so the admin can choose whether the description should be displayed (above or below element) or not.
Comment #8
chris matthews commentedThe patch in #7 no longer applies to 7.x-1.x-dev so setting back to needs work for now.
Comment #9
vchen commentedI updated patch #7 against the latest 7.x-1.x-dev and it's working for me.
Please test this patch (my first patch!).
Comment #10
vchen commentedComment #12
jnettikThe patch looks good to me and works as expected. Committed. Thank you!
Comment #13
skylord commentedHm. This patch doesn't have fallback for old code that knows nothing about descriptions. I.e. when someone calls theme('mediaelement') directly or "Video" module with support for mediaelement formatter. In that case log becames full of "undefined index" messages. I suggest to add more checks.
Comment #15
jnettik@skylord, good catch. I've added this fix. Thank you for the patch!