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

Check and improve UI performance #582

Closed
cutwater opened this issue May 25, 2018 · 6 comments
Closed

Check and improve UI performance #582

cutwater opened this issue May 25, 2018 · 6 comments
Assignees
Milestone

Comments

@cutwater
Copy link
Collaborator

Bug Report

SUMMARY

According to users feedback our UI on Galaxy QA site is slow and browser seems to intensively load CPU.

We should profile UI performance and resource utilization and improve it if possible.

@drybjed
Copy link

drybjed commented May 25, 2018

I'm seeing slowdown on Galaxy pages that display lots of imported repositories/content (currently ~134 things imported). This happens on Debian Jessie with Chromium 57, on Intel i7 CPU with 8 GB of RAM. Whenever I click anything on the page, the tab becomes unresponsive for about 30s-1m.

@chouseknecht
Copy link
Contributor

Had the same experience on My Content page when there's a large number of repositories. Reworked the Action menus to use components in PR #603 , which seemed to help. Will test more, after that code gets merged and deployed to QA.

@chouseknecht
Copy link
Contributor

Still work to do on My Content, regarding polling.

@chouseknecht
Copy link
Contributor

Changing this to enhancement, because I'm not sure we want to immediately back port it to v3.0

@newswangerd
Copy link
Member

There are noticeable slowdowns in UI performance on the My Content page for authors that have lots of imported repos. The issue seems to be caused by the import and delete buttons in the in the repository list

screen shot 2018-07-06 at 10 32 46 am

The buttons are implemented as a separate component in angular and as such change detection has to be run on all of them whenever something happens on the page. Disabling change detection doubles the UI responsiveness, but something about rendering all the dropdown menus with the delete button is still causing significant lag whenever any UI actions are triggered. The UI here really needs to be rewritten to include pagination and searching so that we don't have to render so many UI elements.

@chouseknecht
Copy link
Contributor

Agree that pagination and searching is in order.

The buttons/menus are in components in order to make them dynamic.

@newswangerd newswangerd added the status/fix-committed Merged to develop \ release branch label Jul 19, 2018
@cutwater cutwater added status/fix-released Fixed in the latest release and removed status/fix-committed Merged to develop \ release branch labels Jul 20, 2018
@newswangerd newswangerd added status/fix-committed Merged to develop \ release branch and removed status/fix-released Fixed in the latest release labels Jul 26, 2018
@newswangerd newswangerd reopened this Jul 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants