Problem/Motivation
Follow up from #3239980: Deprecate Modernizr
From @xjm
The dictionary, libraries, cspell, .eslintrc.json, and vendor-update.js mentions should be removed in the D11 followup. (Maybe it would make sense to file it as postponed now, so we don't miss those parts?) However, I think that the mention in touchevents-test.js should be removed.
Steps to reproduce
NA
Proposed resolution
Remove references to modernizer
Remaining tasks
Merge #3239980: Deprecate Modernizr
Remove references
Review
User interface changes
NA
API changes
NA
Data model changes
NA
Release notes snippet
NA
Issue fork drupal-3334920
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
Comment #2
spokjeComment #3
gábor hojtsyUpdating title, tags and version number based on recent announcement at https://www.drupal.org/about/core/blog/new-drupal-core-branching-scheme-...
Comment #4
quietone commentedWhile this is not a requirement I am adding to the alpha1 meta so we can keep track.
Comment #7
spokjeOpened #3425304: Resync dictionary.txt to resync cspell dictionary.
I _think_ this MR won't have any dictionary changes after that, but let's see.
Comment #8
spokjeThe only remaining references to modernizr in the MR is are
core/misc/touchevents-test.js, and that leads to its appearance incore/misc/cspell/dictionary.txt.According to #3239980-41: Deprecate Modernizr and #3239980-42: Deprecate Modernizr this is fine.
Comment #9
smustgrave commentedDid a search for Modernizr and as mentioned only 2 instances mentioned are the only things I found.
Comment #11
catchCommitted/pushed to 11.x, thanks!
We should revisit whether we really need touchevents any more too, I had a quick check and found #3375181: Deprecate the touchevents JavaScript library and update CSS which is already open.