-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CSS causes issue with Turbopack #9261
Comments
Looks like this was originally added in 2011 to make popup tip IE6–8 properly displayed (rotated 45 degrees for the "tip" appearance). 5f63b8d Not sure if we want to care about this anymore cc @Falke-Design. Might consider removing the offending line (or both filter lines) for v1.9.5 — popups should remain accessible even in old IE. |
In Lines 1106 to 1112 in 83fb5a4
Also it looks like Leaflet doesn't work with IE6-8 anymore. With Edge simulation of IE8 I get the following error:
which points to
Line 50 in 83fb5a4
So in my eyes we can drop support for IE8 and lower with 1.9.5 because it already doesn't work anymore. |
@Falke-Design yeah, no one really cares about those versions, and even Leaflet website states IE9+ support on the website for quite some time now. Let's just remove that stuff for v1.9.5. |
+1 let's throw this in the bin. |
This comment was marked as spam.
This comment was marked as spam.
This is a workaround until this Leaflet issue is resolved: Leaflet/Leaflet#9261
Hi, |
Checklist
Steps to reproduce
Use Turbopack to build the project. Observe the build fails to parse the leaflet CSS.
The issue was closed on turbopack's repo siting it is an upstream issue in leaflet. vercel/turbo#2451
Expected behavior
The build should not fail to parse the leaflet CSS.
Current behavior
Turbopack fails to build
Minimal example reproducing the issue
No response
Environment
The text was updated successfully, but these errors were encountered: