Media display should be templated so that changes can be made without hacking the module. Currently only the thumbnail is templated.

Example use case: Move the media-gallery-controls to a location above the image, not below, so users can easily page through a sequence of images. Placing the controls below an image makes their location dependent on the depth of the image, which can vary.

Comments

drupa11y’s picture

drupa11y commented

Guess this is a good idea.

ergophobe’s picture

ergophobe commented

Actually, I came here to make exactly this request.
There is a lot that is hard-coded in this module and it makes it really hard to use. I understand it's in early development, but this one change would help a lot with adoption.

In my case, I want my galleries to have a lot of additional fields in addition to the ones that are built in. If I use the default template, they don't display (and in any case would not be formatted the way I want if they did). If I switch to using node--media-gallery.tpl.php I have to rip all the guts out of the module and recreate the display (unless I'm missing something).

The hard-coded fields are also an issue in that I want to throw the Gallery Settings in a vertical tab and I can only do that with a THEME_form_alter call (which is fine, but wouldn't normally be necessary if the fields weren't completely hardcoded).

The thing is, when done this module will finally bring top-notch image handling to Drupal, so it would great to see wider adoption and get more brains working on it, but I have to say I'm a bit lost with a lot of what's going on this module so I'm afraid I'm not a lot of help :-(

lsolesen’s picture

lsolesen commented

Marked #1174512: Why use node.tpl to view the "all galleries" page? as duplicate.

lsolesen’s picture

lsolesen commented

Patches are welcome. Any takers?

ivnish’s picture

ivnish
he/him
commented
Issue summary: View changes
Status: Active » Closed (outdated)