Another minor observation on a handy module.

Provide for an alternative HTML5 based template, i.e.,

<figure class="image_field">
  <?php print $image; ?>
  <figcaption class="image-field-caption">
    <?php print $caption; ?>
  </figcaption>
</figure>

reference: http://html5doctor.com/the-figure-figcaption-elements/

CommentFileSizeAuthor
#2 image_field_caption-change_to_HTML5_figure_figcaption_2031435.patch990 bytesBadfaerie
#1 image_field_caption-change_to_HTML5_figure_figcaption_2031435.patch990 bytesBadfaerie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Badfaerie’s picture

Badfaerie CreditAttribution: Badfaerie as a volunteer commented
Issue summary: View changes
FileSize
image_field_caption-change_to_HTML5_figure_figcaption_2031435.patch990 bytes

Patch does the above, but does not add a specific class. It inserts the default field-item classes and you can add any other classes you need or want.

Badfaerie’s picture

Badfaerie CreditAttribution: Badfaerie as a volunteer commented
FileSize
image_field_caption-change_to_HTML5_figure_figcaption_2031435.patch990 bytes
Badfaerie’s picture

Badfaerie CreditAttribution: Badfaerie as a volunteer commented
iStryker’s picture

iStryker CreditAttribution: iStryker at University of Waterloo commented
Status: Active » Closed (outdated)

I feel like this could break things. If people have css for this then it would break, as all elements would change

Marking all 7.x as out of date. If the problem persists, then only a new issue.