vaadin / patient-portal-demo-flow

0 stars 0 forks source link

Investigate slowness with IE11 #75

Closed pleku closed 6 years ago

pleku commented 6 years ago

Patient Portal PatientsView is currently totally unusable with IE11 since it takes over a minute to load.

This issue is about investigating where does the time go and why.

Interestingly, Bakery StorefrontView does not have the same performance issues, and the test app by Artur http://artur.app.fi/slowgrid-vaadin10-production/ shows that the number of columns has impact on this.

Legioth commented 6 years ago

In my own testing, the loading time wasn't over a minute, but rather around 10 seconds. I suspect there's something fishy with the testing setup, but I'd have a hard time determining whether it's something that actual users might also encounter.

pleku commented 6 years ago

Acceptance Criteria

We have some understanding on why IE11 is so slow with Grid. Whether it is on the web component or Flow side doesn't matter, but we should have issues created for the following steps to improve the situation.

gilberto-torrezan commented 6 years ago

There were some rendering problems related to the webcomponent, that were fixed on vaadin-grid beta4 and beta6, drastically improving the performance on IE11.

The tests weren't limited to patient-portal.

On Flow, there's one particular issue related to hidden columns: https://github.com/vaadin/vaadin-grid-flow/issues/117 , that doesn't affect patient-portal directly, but has some performance impacts on apps that use hidden columns.

pleku commented 6 years ago

Follow up for measuring again: https://github.com/vaadin/patient-portal-demo-flow/issues/78