-
Notifications
You must be signed in to change notification settings - Fork 108
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
INP still says “waiting for user input” even after I clicked / typed #108
Comments
Codepens are loaded in iframes and Web APIs cannot see into iframes - though it should be noted CrUX does as not limited by Web APIs. You'll also see that if you turn on the extension's console logging, it registers an LCP but it's the outer page's title, even though the inner iframe's title is bigger. So to replicate this you'll need to host this page outside of CodePen (I tried it in full screen mode, but it still does it via an iframe). |
Thanks @tunetheweb for your reply. The fact is there are no I also tried to download the pen and run it locally on my machine, again no |
Apologies for that! I figured it out. INP, like CLS is basically calculated at the end of page life cycle, not on every interaction. The Extension has some logic to broadcast the latest CLS, but the same doesn't exist for INP. So to get you INP, until that's added, you need to "leave" the page. The easiest way to do that is to switch tabs, and then switch back again. Will chat to the team about adding a |
INP is actually updated instantly every time there is a new measured interaction. But thats based on Event Timing, and Event Timing doesn't necessarily measure every single interaction. At the moment, the web-vitals-extension uses the default Besides this, the web-vitals.js has landed several improvements to vitals measurement and INP specifically-- so I think the first task is to roll the the latest web-vitals.js v3.0 here. |
Yeah, what @Mmocy said. The |
Thank you. onINP(console.log, { reportAllChanges: true, durationThreshold: 1 }); With that, I get a report of my clicks on the checkbox and on the This is probably an issue to open in the |
That is working as intended (even if not as desired). Specifically, INP only measures up to the single next paint (or, if there is no need for next paint, until the decision point). If you have async work that updates main content, this will not get captured. I think you may want to use Element Timing for that, or, perhaps you are making the case for a "Interaction to next LCP" type metric... I would encourage filing an issue asking for that :) |
Thanks @mmocny, I must have completely misunderstood the meaning of "paint" while reading all of the explanation in the INP page. I think this issue still useful as a reminder to roll the the latest web-vitals.js v3.0. |
You're not the only one who misunderstood this @verlok ! I did too initially. Some more info here: https://twitter.com/tunetheweb/status/1578065048752242698?s=20&t=FY7tdngVmmGiuXXpJJHW_g |
Oooooooh. So it’s like Interaction to Next ability to Paint. Or Potential Paint. Thank you for this explainer, Barry. I will save the link to your tweet in my bookmarks. I wonder why this isn’t clear by reading the doc on web.dev/inp |
Oh we tweaked that only recently to try to make that more clear: GoogleChrome/web.dev#8686 Open to suggestions on how to further clarify this? |
Describe the bug
After reading through all the INP documentation on web-dev and reading the recommendations to optimise INP, I wanted to try if using a skeleton pattern would help reducing INP.
So I tried and created this pen on Codepen, which displays gray boxes while the page is loading async content.
The problem is that, even opening the page in debug mode (so without the Codepen interface) like this, the web vitals extension still says “waiting for user input” even after I clicked on the “load async content” button or clicked / typed on the input.
To Reproduce
Expected behavior
I expected the extension to show some values in INP after I clicked or interacted with the page.
Version:
and Microsoft Edge Version 105.0.1343.53 (Official build) (x86_64)
Additional context, screenshots, screencasts
Screenshot
The text was updated successfully, but these errors were encountered: