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

scrollable callOnFunctions: true triggers an error when no onScroll is set #5216

Closed
marciogurka opened this issue Sep 12, 2022 · 3 comments
Closed
Labels
bug Something isn't working forum Issues from forum not reproducible OEM OEM customer

Comments

@marciogurka
Copy link

Forum post

When adding the scrollable option with the callOnFunctions property true but don't set an onScroll function, the error of "Maximum call stack size exceeded" occurs

To reproduce that behavior, add the following configuration to the basic Grid demo

scrollable : { callOnFunctions : true },

Screenshot_33.png

@marciogurka marciogurka added bug Something isn't working forum Issues from forum OEM OEM customer labels Sep 12, 2022
@isglass
Copy link
Contributor

isglass commented Sep 13, 2022

While it seems to be a bug, why set callOnFunctions on a Scroller and then not have any on-functions? Perhaps there is some misunderstanding here on what should happen, please clarify purpose

@henriquewerlang
Copy link

We have made a framework to build the application windows, and all events are registered using the "On functions", and in case it is not necessary to spread this configuration throughout the entire application, the framework automatically fills this information in all locations, even if the event does not exist.

@matsbryntse
Copy link
Member

No longer reproducible, please reopen with new instructions if you can still reproduce the issue

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

No branches or pull requests

4 participants