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.

Comments

dazz’s picture

subscribe

alotaiba’s picture

Status: Active » Needs review
StatusFileSize
new1.79 KB

Sorry 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.

acranius’s picture

Version: 7.x-1.0 » 7.x-1.2

Worked with me, thanks!

traveller’s picture

Thanks alotaiba, works for me.

zipymonkey’s picture

Patch worked for me as well.

matt b’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Worked for me

bocaj’s picture

Version: 7.x-1.2 » 7.x-1.x-dev
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new4.78 KB

This 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.

chris matthews’s picture

Status: Needs review » Needs work

The patch in #7 no longer applies to 7.x-1.x-dev so setting back to needs work for now.

vchen’s picture

StatusFileSize
new4.93 KB

I updated patch #7 against the latest 7.x-1.x-dev and it's working for me.

Please test this patch (my first patch!).

vchen’s picture

Status: Needs work » Needs review

  • vchen authored 3d856aa on 7.x-1.x
    Issue #1154196 by vchen, alotaiba, bocaj, acranius, dazz, traveller,...
jnettik’s picture

Status: Needs review » Fixed

The patch looks good to me and works as expected. Committed. Thank you!

skylord’s picture

Status: Fixed » Needs review
StatusFileSize
new2.59 KB

Hm. 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.

  • skylord authored f7c97b9 on 7.x-1.x
    Issue #1154196 by skylord: Show Description of file
    
jnettik’s picture

Status: Needs review » Fixed

@skylord, good catch. I've added this fix. Thank you for the patch!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.