Problem/Motivation

As part of #3067216: [meta] Deprecate (in 8.8) and remove (in 9.0) polyfills that are no longer needed the classList asset library was marked deprecated in 8.8.
see: #2955842: Deprecate classList library
All files and usage of classList should be removed in Drupal 9.

Proposed resolution

Remove classList from 9.0.x
Existing change record: classList polyfill is deprecated

Remaining tasks

  1. Post patch removing classList

User interface changes

None

API changes

Data model changes

Release notes snippet

The classList polyfill has been removed from Drupal 9.0 because it is no longer required by any of the browsers supported by Drupal core

CommentFileSizeAuthor
#3 3090010-3.patch5.09 KBzrpnr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zrpnr created an issue. See original summary.

zrpnr’s picture

zrpnr
he/him
Primary language English
Location UTC-7
CreditAttribution: zrpnr at Acquia commented
Title: Remove classList in Drupal 9 » [PP-1] Remove classList in Drupal 9

postponed on #2955842: Deprecate classList library

zrpnr’s picture

zrpnr
he/him
Primary language English
Location UTC-7
CreditAttribution: zrpnr at Acquia commented
Title: [PP-1] Remove classList in Drupal 9 » Remove classList in Drupal 9
Status: Postponed » Needs review
FileSize
3090010-3.patch5.09 KB

This was postponed on #2955842: Deprecate classList library
now that is in 9.0.x, this patch removes:

  • classList asset library
  • classList dependency in core/active-link
  • files in core/assets/vendor/classList
  • deprecation warning suppression in DeprecationListenerTrait
bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
CreditAttribution: bnjmnm at Acquia commented
Status: Needs review » Reviewed & tested by the community
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented

Fix credits.

  • Gábor Hojtsy committed 1e2643a on 9.0.x 
    Issue #3090010 by zrpnr, bnjmnm: Remove classList in Drupal 9
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Reviewed & tested by the community » Fixed

Sitting in person with @nod, who reviewed and nodded that this looked right.

Gábor Hojtsy credited nod_.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented

Status: Fixed » Closed (fixed)

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