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

context prop is optional for no reason? #1181

Open
aleclarson opened this issue Jan 21, 2025 · 1 comment
Open

context prop is optional for no reason? #1181

aleclarson opened this issue Jan 21, 2025 · 1 comment
Labels
enhancement New feature or request

Comments

@aleclarson
Copy link

The context prop for each component in components is optional, even though it will always exist when context is defined on the Virtuoso element. We should replace context?: Context with context: Context in the interfaces.ts module.

@petyosi petyosi added the enhancement New feature or request label Jan 22, 2025
@petyosi
Copy link
Owner

petyosi commented Jan 22, 2025

Agreed, the prop is always passed, even if undefined. I will check as the same thing might be applicable to data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants