Caught a minor issue with the preview image field setting we previously committed.

There is an invisible #states option for the image style, but it was hard coded to be field_video. Have changed this to be relative to the field being edited.

Also edited the preview image source description to be a bit more succinct.

Patch is attached, thanks.

CommentFileSizeAuthor
#2 jw_player-fix_image_field_option-2725119-2.patch2.77 KBron_s
#2 interdiff-2725119-1-2.txt2.13 KBron_s
jw_player-fix_image_field_option-1.patch1.7 KBron_s
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ron_s created an issue. See original summary.

ron_s’s picture

ron_s CreditAttribution: ron_s commented
FileSize
interdiff-2725119-1-2.txt2.13 KB
jw_player-fix_image_field_option-2725119-2.patch2.77 KB

Here is a better version of this patch. Thanks.

  • Berdir committed ba0e4fa on 7.x-2.x authored by ron_s 
    Issue #2725119 by ron_s: Fix preview image field settings
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Version: 7.x-2.x-dev » 8.x-1.x-dev
Status: Needs review » Patch (to be ported)

Committed. Not sure if we'll port this to 8.x on its own or as part of another issue but keeping open for now.

johnchque’s picture

johnchque
Primary language Spanish
Location Bolivia
CreditAttribution: johnchque at MD Systems GmbH commented
Status: Patch (to be ported) » Fixed

Fix of the patch was included in #2713679: Add managed selected preset option.

Status: Fixed » Closed (fixed)

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