Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.
And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.
matchAll Polyfill
See original GitHub issueWe’ve been seeing some client side error logs re .matchAll() calls evaluating to undefined. After a bit of digging, we realized that all of those exceptions originate on older-ish browsers. Nonetheless, this is an issue for us (and I suspect others, too). Specifically, the issue stems from this line: https://github.com/dcastil/tailwind-merge/blob/a703309b6811b8b798b1a8cd2d1db3ac06811b36/src/lib/merge-classlist.ts#L83
I’m curious if you’ve given thought to adding a matchAll polyfill, or using a different approach to matching?
Issue Analytics
- State:
- Created a year ago
- Comments:8 (3 by maintainers)
Top Results From Across the Web
Top Related Medium Post
Top Related StackOverflow Question
Troubleshoot Live Code
Top Related Reddit Thread
Top Related Hackernoon Post
Top Related Tweet
Top Related Dev.to Post
Top Related Hashnode Post
Confirmed working on latest v1.6.0. Thanks so much, @dcastil !
Right – I’ve impl’d that polyfill and it seems to work for us. But it took us an hour to realize this was the issue. Figured I’d post here to save others some time.
I’m able to reproduce this on Chrome 70.x on Mac. It’s difficult to run multiple Chrome versions on the same machine, but you can download an old build of Chromium and easily run that side-by-side. Here’s the version I’m using: version: 70.0.3538.124
Here’s a sample stacktrace, although, the issue is rather straight forward: