-
Notifications
You must be signed in to change notification settings - Fork 27
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
Lists not working in some browsers #139
Comments
The problem happens on the codesandboxes in the readme on mobile. For example: https://codesandbox.io/s/xenodochial-williams-xok6q |
The issue is related to a bug in isBehavior from hareactive. My guess why it still works on your linux machine, is because the chrome version is older than 92, where they introduced a new method on arrays which causes the issue. |
Ah amazing! Glad you have a fix almost ready. When do you expect it to land? Or do you have a suggested workaround? |
I'm working on getting a development environment setup so I can make this fix myself but I'm unable to find a way to get npm to use my local version of hareactive/turbine instead of that from the npm registry. Am I missing a build step or something? Ideally, I want it to auto-reload changes from these dependencies on save, if possible. How do you advise a setting up development environment for hacking on hareactive and turbine? For context, I'm working on gitpod.io instead of locally -- if that helps. (Easier to collaborate) |
After a 2 hours I was able to get local dependencies working! The issue was that I was on node v14.18.3 which seems to require |
While it works in Chrome on my Linux machine, it does not work on Chrome on mobile or Firebox or Chromium on Linux.
Codesandbox: https://codesandbox.io/s/determined-water-oqv6o?file=/src/index.ts
Error:
The text was updated successfully, but these errors were encountered: