-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Grid UI component performance and behavior #34225
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
Comments
Hi @onlinebizsoft. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. Please, add a comment to assign the issue:
🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
Hi @onlinebizsoft. Thank you for your report. To help us process this issue please provide us the following information:
It would be good if you provide some screenshots as well related to the issue. Thanks |
Hi @engcom-Hotel. Thank you for working on this issue.
|
@engcom-Hotel I have provided all these information. |
@magento give me 2.4-develop instance |
Hi @pemann. Thank you for your request. I'm working on Magento instance for you. |
Hi @pemann, here is your Magento Instance: https://ae886554601f9303744ea3b1b1cc4b03-2-4-develop.instances.magento-community.engineering |
This might be duplicate to magento/community-features#238 |
I consider this as a bug so it is not a feature request from the community. |
Hello @onlinebizsoft, Thanks for reopening the issue! Are the steps to reproduce is same as mentioned here magento/community-features#238 (comment)? And also which tool you have used to monitor the performance? Thanks |
@engcom-Hotel we use Tideways, you can also easily see with network request in the browser developer tool. |
Hello @onlinebizsoft, Thank you for your reply! Since the issue is quite old, it might have been resolved along with other issues. We request you to try reproducing it in the latest 2.4-develop branch and let us know if the issue is still reproducible for you. Additionally, please confirm the steps to reproduce this issue. Thanks |
Hello @onlinebizsoft , As we have not heard from you regarding this issue for some time, we are assuming that it has been resolved. Therefore, we will be closing this issue. If you are still experiencing any problems, please let us know. Thanks |
Summary (*)
I have watched on some performance monitoring tool and I found this bad behavior, lets say if you have a grid with some kind of sub-queries for each row (custom renderer for each cell for example) then you would see the problem very clearly.
Basically the Grid UI component are loading in 2 stages: first the page loaded from the controller on the GET request then sub-request from Ajax render the grid (mui render). However both seems to process the actual data so users have to wait twice time than normally
Examples (*)
Proposed solution
If on the first loading, we already have data then we could render it from server-side ready, if not then we should load the page fast without processing actual data then let the client-side render it with sub-request ajax (mui render)
Please provide Severity assessment for the Issue as Reporter. This information will help during Confirmation and Issue triage processes.
The text was updated successfully, but these errors were encountered: