Currently the file_entity field formatter requires that both the width and height be both provided, otherwise they're ignored. This prevents e.g. assigning a width of 100% so that the video is scaled to fit the available space.

CommentFileSizeAuthor
#2 file_entity-n2289587-2.patch1.49 KBdamienmckenna
#1 file_entity-n2289587-1.patch993 bytesdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new file_entity-n2289587-1.patch993 bytes

A quick patch to provide the change requested.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new file_entity-n2289587-2.patch1.49 KB

Had to update the theming function too.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Title: Allow width and height to be used independently in field formatter » Allow width and height to be used independently in video field formatter
mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Needs review » Reviewed & tested by the community

Applied clean, let me set width and allow browser to figure out proper height.

  • Dave Reid committed 1e037ad on 7.x-2.x 
    Issue #2289587 by DamienMcKenna: Allow width and height to be used...
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Committed #2 to 7.x-2.x.

Status: Fixed » Closed (fixed)

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