Page MenuHomePhabricator

Update how we measure LayoutShift
Closed, ResolvedPublic

Description

Google announced that they are changing how they measure CLS and I think we should follow so we measure the same way as they when they push the change: https://web.dev/evolving-cls/

Event Timeline

@Gilles do you know if there is a new name too?

Not that I know of, but that was still being debated at recent meetings

Krinkle added a subscriber: Gilles.

Change 844911 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[mediawiki/extensions/NavigationTiming@master] Collect Cumulative Layout Shift instead of just Layout Shift.

https://gerrit.wikimedia.org/r/844911

Change 859433 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[schemas/event/secondary@master] Add cumuluative layout schema.

https://gerrit.wikimedia.org/r/859433

Change 859433 abandoned by Phedenskog:

[schemas/event/secondary@master] Add cumuluative layout schema.

Reason:

Lest add this to navtiming

https://gerrit.wikimedia.org/r/859433

Change 859968 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[schemas/event/secondary@master] navtiming: Add cumulative layout shift and largest contentful paint.

https://gerrit.wikimedia.org/r/859968

Change 860842 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/extensions/NavigationTiming@master] Add test for multiple CLS frames and rounding

https://gerrit.wikimedia.org/r/860842

Change 859968 merged by jenkins-bot:

[schemas/event/secondary@master] navtiming: Add cumulative layout shift and largest contentful paint.

https://gerrit.wikimedia.org/r/859968

Change 844911 merged by jenkins-bot:

[mediawiki/extensions/NavigationTiming@master] Collect Cumulative Layout Shift instead of just Layout Shift

https://gerrit.wikimedia.org/r/844911

Change 860842 merged by jenkins-bot:

[mediawiki/extensions/NavigationTiming@master] Add test for multiple CLS frames and rounding

https://gerrit.wikimedia.org/r/860842

Change 861343 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[performance/navtiming@master] Add cumulative layout shift and largest contenful paint.

https://gerrit.wikimedia.org/r/861343

Change 861343 merged by jenkins-bot:

[performance/navtiming@master] Add cumulative layout shift and largest contenful paint.

https://gerrit.wikimedia.org/r/861343

Change 726852 had a related patch set uploaded (by Krinkle; author: Peter Hedenskog):

[operations/puppet@production] eventlogging: Remove obsoleted navtiming schemas

https://gerrit.wikimedia.org/r/726852

Change 879926 had a related patch set uploaded (by Krinkle; author: Krinkle):

[operations/mediawiki-config@master] Remove former EventLogging streams for navtiming

https://gerrit.wikimedia.org/r/879926

Change 879926 merged by jenkins-bot:

[operations/mediawiki-config@master] Remove former EventLogging streams for navtiming

https://gerrit.wikimedia.org/r/879926

Mentioned in SAL (#wikimedia-operations) [2023-02-01T08:17:44Z] <ladsgroup@deploy1002> Started scap: Backport for [[gerrit:879926|Remove former EventLogging streams for navtiming (T281103 T286703 T308621 T323623)]]

Mentioned in SAL (#wikimedia-operations) [2023-02-01T08:19:32Z] <ladsgroup@deploy1002> ladsgroup and krinkle: Backport for [[gerrit:879926|Remove former EventLogging streams for navtiming (T281103 T286703 T308621 T323623)]] synced to the testservers: mwdebug1001.eqiad.wmnet, mwdebug2001.codfw.wmnet, mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2023-02-01T08:27:27Z] <ladsgroup@deploy1002> Finished scap: Backport for [[gerrit:879926|Remove former EventLogging streams for navtiming (T281103 T286703 T308621 T323623)]] (duration: 09m 42s)

Change 726852 merged by Ottomata:

[operations/puppet@production] eventlogging: Remove obsoleted navtiming schemas

https://gerrit.wikimedia.org/r/726852

Change 887425 had a related patch set uploaded (by Krinkle; author: Krinkle):

[schemas/event/secondary@master] Remove elementtiming,firstinputtiming,layoutshift,resourcetiming,rumspeedindex

https://gerrit.wikimedia.org/r/887425

Change 887425 merged by jenkins-bot:

[schemas/event/secondary@master] Remove elementtiming,firstinputtiming,layoutshift,resourcetiming,rumspeedindex

https://gerrit.wikimedia.org/r/887425