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:

About issue forks

Comments

calebtr created an issue. See original summary.

calebtr’s picture

calebtr commented
Issue summary: View changes
calebtr’s picture

calebtr commented
Related issues: +#3449302: core.libraries.yml mis-implements moved_files syntax

calebtr opened merge request !8339

calebtr’s picture

calebtr commented
Status: Active » Needs review
calebtr’s picture

calebtr commented
Issue tags: +Contributed project blocker
catch’s picture

catch
he/him
Primary language English
commented
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

catch
he/him
Primary language English
commented
Parent issue: » #3409178: [meta] Release preparation steps for 10.3.0
nod_’s picture

nod_
Primary language French
Location Lille
commented
Title: core.libraries.yml mis-implements moved_files syntax, redux » core.libraries.yml mis-implements moved_files syntax

changing title for commit message

nod_ closed merge request !8339

  • 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

nod_
Primary language French
Location Lille
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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