Problem/Motivation

The Remote video field mapping labels at /admin/structure/media/manage/remote_video are:

  1. not consistent with one another
  2. too verbose

Proposed resolution

Remote Video Field Mapping Field Labels

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#5 3222486-5.patch1.75 KBchris matthews
#4 3222486-4.patch1.77 KBchris matthews
#3 3222486-3.patch1.77 KBchris matthews
#2 remove_video_field_mapping_labels.png109.72 KBchris matthews
remove-video-field-mapping-labels.png109.33 KBchris matthews

Issue fork drupal-3222486

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Chris Matthews created an issue. See original summary.

chris matthews’s picture

chris matthews commented
Issue summary: View changes
StatusFileSize
new remove_video_field_mapping_labels.png109.72 KB
chris matthews’s picture

chris matthews commented
Assigned: chris matthews » Unassigned
Status: Active » Needs review
StatusFileSize
new 3222486-3.patch1.77 KB
chris matthews’s picture

chris matthews commented
StatusFileSize
new 3222486-4.patch1.77 KB
chris matthews’s picture

chris matthews commented
StatusFileSize
new 3222486-5.patch1.75 KB
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs review » Needs work

+1 for this change! Let's get it converted to a merge request so that it's ready to go, but I don't see any real reason to block this once tests pass.

Chris Matthews opened merge request !906

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs work » Reviewed & tested by the community

I don't think I have any objections here. RTBC on the assumption that tests will pass. Also, hiding old patches.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed
Issue tags: +String change in 9.3.0

Wow, when you lay them out in a nice table like that, some of these are comically long. 🤣

Thanks so much for the improvement!

Committed and pushed to 9.3.x! Not backporting, since it represents a string change.

  • webchick committed cd796b6 on 9.3.x 
    Issue #3222486 by Chris Matthews: Make remote video field mapping field...

Status: Fixed » Closed (fixed)

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