Since these fields do not apply when the media widget is being used, we should hide these settings in the field settings UI.

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Title: Hide file and image fields' alt and description settings » Hide file and image field settings that are not yet implemented in the media widget
wmostrey’s picture

wmostrey CreditAttribution: wmostrey commented

Should we hide them, or actually implement them?

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

My thinking behind this is hide the settings until each individual issue can be addressed. I know this would be easy to do at least as a mitigation for now.

wmostrey’s picture

wmostrey CreditAttribution: wmostrey commented

Agreed. Is there a simultaneous effort to release a stable 7.1 and 7.2 version? Is there a discussion or overview of this for people that want to help?

tsvenson’s picture

tsvenson CreditAttribution: tsvenson commented

Info about use cases where these fields are needed can be found in #1241564: [meta] Proposal for features needed for a full 2.0 release of the Media module. Also adding this issue in the list there.

wmostrey’s picture

wmostrey CreditAttribution: wmostrey commented

There is currently a (tested and confirmed working) patch in #1223794: Widget does not respect settings in image_field_instance_settings_form (min/max resolution) that makes these fields work, so there would be no need to work on a temporary fix to hide them. I would mark this issue as a duplicate and focus on getting that patch committed.

michaelfavia’s picture

michaelfavia CreditAttribution: michaelfavia commented
Status: Active » Closed (duplicate)

Marking as duplicate unless there are other settings mot addressed in #1223794: Widget does not respect settings in image_field_instance_settings_form (min/max resolution)?