ampproject / ampproject/amphtml

Add max-session-gap1s-limit5s CLS metric

Open
#33,733 6 comments 0 reactions 1 assignee Claimed by @erwinmombay View on GitHub
P1: High Priority Type: Feature Request WG: performance
Dominant language
JavaScript
Stars
14.9k
Forks
4.1k
PR merge metrics
No merged PRs in 30d

Description

## Describe the new feature or change to an existing feature you'd like to see

https://web.dev/evolving-cls/ explains an update to CLS. AMP currently sums all layout shifts: https://github.com/ampproject/amphtml/blob/e03dba1b1fe7ff1d4dfeb59ba257ab4d0d93341f/src/service/performance-impl.js#L519-L530

A new tick should be added for [max-session-gap1s-limit5s](https://github.com/mmocny/web-vitals/wiki/Snippets-for-LSN-using-PerformanceObserver#max-session-gap1s-limit5s)

@samouri

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.