Problem/Motivation

In #3439580: Make drupal.tableheader only use CSS for sticky table headers, the tableheader.css library was moved and a moved_files key was added to core.libraries.yml using an incorrect syntax.

Steps to reproduce

Install 10.x-3rc1 and a contributed module or theme that overrides drupal.tableheader (e.g. drupal/stable:2.0.0).

Proposed resolution

Per the fix in #3449302: core.libraries.yml mis-implements moved_files syntax, we need to specify which group the file has moved to as well as the new filename.

Issue fork drupal-3453324

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:

Comments

calebtr created an issue. See original summary.

calebtr’s picture

Issue summary: View changes
calebtr’s picture

calebtr’s picture

Status: Active » Needs review
calebtr’s picture

catch’s picture

Status: Needs review » Reviewed & tested by the community

Yep that's correct. When I worked on the original MR I think my brain just didn't want to believe that much YAML nesting was necessary.

catch’s picture

nod_’s picture

Title: core.libraries.yml mis-implements moved_files syntax, redux » core.libraries.yml mis-implements moved_files syntax

changing title for commit message

  • nod_ committed aa034010 on 10.3.x
    Issue #3453324 by calebtr, catch: core.libraries.yml mis-implements...

  • nod_ committed 22e8a87d on 10.4.x
    Issue #3453324 by calebtr, catch: core.libraries.yml mis-implements...
nod_’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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