-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Closed
hrueger/AGLight
#112Labels
P2The issue is important to a large percentage of users, with a workaroundThe issue is important to a large percentage of users, with a workaroundperfThis issue is related to performanceThis issue is related to performance
Description
Reproduction
Steps to reproduce:
- open Chrome dev tools / Performance
- start recording a profile and go to https://next.angular.io/guide/router
Expected Behavior
MatPaginatedTabHeader should not be the perf hot spot / bottleneck for AIO
Actual Behavior
The _checkPaginationEnabled methods triggers a forced synchronous layout (by doing const isEnabled = this._tabList.nativeElement.scrollWidth > this._elementRef.nativeElement.offsetWidth;.
This forced synchronous layout accounts for ~38% of the entire JS processing time on the critical TTI path (see attached recording). Similar bottlenecks were spotted on other profiled sites (ex. https://www.mustakbil.com/)
Splaktar, gkalpak and sylvaindumont
Metadata
Metadata
Assignees
Labels
P2The issue is important to a large percentage of users, with a workaroundThe issue is important to a large percentage of users, with a workaroundperfThis issue is related to performanceThis issue is related to performance