The File Entity module transparently leverages the field formatters defined for the 'file' field type. I suggest we do the same for field formatters defined for the 'image' field type. This would allow leverage some nice image specific formatters, for example the Colorbox formatter.

CommentFileSizeAuthor
#2 1876392-leverage-imagefield-formatters-media-1.x-do-not-test.patch1.36 KBDamien Tournoud
#1 1876392-leverage-imagefield-formatters.patch1.19 KBDamien Tournoud
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Active » Needs review
FileSize
1876392-leverage-imagefield-formatters.patch1.19 KB

Here is a patch against 2.x.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
FileSize
1876392-leverage-imagefield-formatters-media-1.x-do-not-test.patch1.36 KB

Backport to Media 1.x.

attiks’s picture

attiks CreditAttribution: attiks commented
Status: Needs review » Reviewed & tested by the community

Nice one, this will also enable the picture formatter

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Project: File Entity (fieldable files) » D7 Media
Version: 7.x-2.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Needs review

thats sweet..just tried colorbox and works:) thanks a lot
i commited #1 http://drupalcode.org/project/file_entity.git/commit/5c37fca
sending to media for 1.x although its feature freeze i think

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Fixed

Tentatively marking this as fixed since Media 1.x is feature frozen.

Feel free to reopen if you disagree (and are able to provide a patch!).

Status: Fixed » Closed (fixed)

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