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

Fix slow rendering of catalog filter #1025

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

rastislav-chynoransky
Copy link
Contributor

@rastislav-chynoransky rastislav-chynoransky commented Aug 28, 2024

https://jira.sng.sk/browse/WEBUMENIA-2095

Measuring performance of scripting/rendering we were able to speed up the response time by ~300ms (30%).

before:
Screenshot 2024-08-29 at 11 04 23

after:
Screenshot 2024-08-29 at 11 04 36

@rastislav-chynoransky rastislav-chynoransky changed the title Slow rendering of catalog filter Fix slow rendering of catalog filter Aug 28, 2024
@rastislav-chynoransky
Copy link
Contributor Author

@FrantisekMichalSebestyen bump

Copy link
Contributor

@FrantisekMichalSebestyen FrantisekMichalSebestyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The popover closes on mouseover outside of container:
https://github.com/user-attachments/assets/384e97d3-8f65-451e-997e-0217391c6298

@igor-kamil
Copy link
Member

The popover closes on mouseover outside of container: https://github.com/user-attachments/assets/384e97d3-8f65-451e-997e-0217391c6298

caused by using VMenu instead of VDropdown. This is by the behaviour of the component. Fixed with c96b275

Copy link
Member

@igor-kamil igor-kamil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks ok.
but after testing, the results are basicaly the same - if not even worse.

main
Screenshot 2024-11-21 at 15 02 25

WEBUMENIA-2095
Screenshot 2024-11-21 at 15 01 52

@rastislav-chynoransky
Copy link
Contributor Author

@igor-kamil how is the feel of it though? on my machine it's a visible improvement. maybe on your machine there's no significant delay in both cases.

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

Successfully merging this pull request may close these issues.

3 participants