Skip to content
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

[BUG] Gap - Cannot read property 'endsWith' of undefined #710

Closed
nabigraphics opened this issue Jul 13, 2022 · 2 comments
Closed

[BUG] Gap - Cannot read property 'endsWith' of undefined #710

nabigraphics opened this issue Jul 13, 2022 · 2 comments
Labels
bug Something isn't working

Comments

@nabigraphics
Copy link
Contributor

Describe the bug

Today, I got this error on sentry issue tracker.
I checked and it seems that undefined is coming into the value of the resolveGapValue function.

Reproduction

https://codesandbox.io/s/kh0fpq?file=/App.js&from-sandpack=true <- Basic Example

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'https://kh0fpq.csb.app/'

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

스크린샷 2022-07-13 오후 10 10 29

Error log with sentry

스크린샷 2022-07-13 오후 6 25 38

스크린샷 2022-07-13 오후 6 26 02

Dev console log

스크린샷 2022-07-13 오후 7 31 58

Desktop (please complete the following information):

  • OS: Android 7
  • Browser: Chrome Mobile WebView 64

Additional context
Add any other context about the problem here.

@efedunin
Copy link
Contributor

Hello there!
I've bumped into this issue with 2.16.6 in IE11.
It seems that method resolveGapValue() here should be patched like in this fix.

@petyosi
Copy link
Owner

petyosi commented Aug 12, 2022

@efedunin I am surprised that virtuoso works in IE11 at all. Why don't you submit a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants