AMP Validator: Allow attribute 'fetchpriority' for link tag

[buihoanghai]

Description

Im trying improve the LCP by adding attribute fetchpriority suggest by this article https://web.dev/priority-hints/

Alternatives Considered

Allow attribute 'fetchpriority' for link tag

Additional Context

No response

[erwinmombay]

hmm let me check if we even allow for link elements currently

[buihoanghai]

hi @erwinmombay any update?

[erwinmombay]

now that our validators are back in sync we can probably add this.

@alanorozco could we get your feedback if this has any negative performance implications?

@banaag LMK if you have any thoughts on this on the validator side of things.

[westonruter]

The fetchpriority attribute also (more importantly?) needs to be allowed on the img element as well.

could we get your feedback if this has any negative performance implications?

Just merely allowing the fetchpriority would allow sites to begin to use it. This is important for reducing LCP: https://web.dev/fetch-priority/

[erwinmombay]

thanks for the input @westonruter , we are adding this to our current sprint

[banaag]

Validator changes have been submitted, next step is deployment.

[westonruter]

It looks like the validator changes have been deployed, right?

[banaag]

Yes, the changes should be deployed by now.

…

On Thu, Oct 12, 2023, 2:23 PM Weston Ruter ***@***.***> wrote: It looks like the validator changes have been deployed, right? — Reply to this email directly, view it on GitHub <#38715 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABIBG7FQVH5UG4JTIEHA3JTX7BNWZANCNFSM6AAAAAAV3H4SBU> . You are receiving this because you were assigned.Message ID: ***@***.***>