Problem/Motivation

Compatibility with D9

Proposed resolution

Update .info.yml include "core_version_requirement: ^8 || ^9"

CommentFileSizeAuthor
#2 readiness-3170167-2.patch401 bytessanjayk
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

sanjayk created an issue. See original summary.

sanjayk’s picture

StatusFileSize
new401 bytes

Fixed d9 readiness

sanjayk’s picture

Status: Active » Needs review

Forgot to move NR.

janmejaig’s picture

I have check the above ticket on Drupal 9.1.x and found that it is working fine .

sanjayk’s picture

Issue summary: View changes
cyberschorsch’s picture

Status: Needs review » Reviewed & tested by the community

Looks good!

mxh made their first commit to this issue’s fork.

mxh’s picture

Status: Reviewed & tested by the community » Needs work

This module depends on the matchMedia polyfill library, which got removed from D9. See change record. The dependency must be either removed or the contrib module needs to be added as dependency in order to make it compatible with D9.

  • mxh committed 63bae95 on 8.x-1.x authored by sanjayk
    Issue #3170167: Drupal 9 Readiness - Theme Breakpoints for Javascript
    

  • mxh committed 1a46298 on 8.x-1.x authored by sanjayk
    Issue #3170167 by Cyberschorsch, sanjayk, mxh, janmejaig: Drupal 9...
mxh’s picture

Status: Needs work » Fixed

The polyfill got removed for now, because D9 does not support any browsers anymore that would need that polyfill. Feel free to open a new issue if the polyfill should be re-added as contrib dependency instead. Thanks!

Status: Fixed » Closed (fixed)

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