Skip to content

[Feature Request] Menu: move the logic for updating position to JS #2292

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

Open
capdiem opened this issue Dec 19, 2024 · 1 comment
Open

[Feature Request] Menu: move the logic for updating position to JS #2292

capdiem opened this issue Dec 19, 2024 · 1 comment
Labels
Milestone

Comments

@capdiem
Copy link
Contributor

capdiem commented Dec 19, 2024

Proposal and motivation

When the MMenu component is set to ScrollStrategy as Reposition, frequently rendering the latest UI on the Blazor side during scrolling can be costly, and the UI may experience lag.

Additional context

No response

@capdiem capdiem added type/enhancement New feature or request component/Menu labels Dec 19, 2024
@capdiem capdiem added this to the 1.9.0 Planning milestone Dec 19, 2024
@capdiem capdiem modified the milestones: 1.9.0 Planning, Backlog Feb 8, 2025
@capdiem
Copy link
Contributor Author

capdiem commented Feb 8, 2025

There is a conflict between the timing of setting styles for transition animations and the timing for updating the menu position styles when latency is high. It doesn't look like we can solve this right now, so let's wait for v2. The development branch is feature/menu-position-refactor.

@capdiem capdiem modified the milestones: Backlog, 2.0.0 Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant