Skip to content

first-paint marking steps are missing important details. Should it consider paintable or not. #122

Description

@shubhamg13
  1. Should it check for elements which are paintable? According to current specs it says
    https://www.w3.org/TR/paint-timing/#mark-paint-timing

5.3.2 Mark paint timing
.
.
.
10.

  1. If reportedPaints does not contain "first-paint"... then report paint timing given document, "first-paint"”
  1. What are the exceptions that need to be considered with non-default background? When its opacity is 0, should it be reported?

Observation in Chrome

TC1: data:text/html, <html style="background: yellow; opacity: 0">
FP: Yes.
With opacity: 0, it is the same as the default background. Still we see Fp on Chrome

TC2: data:text/html, <body style="opacity: 0">foo
FP: No
Acc to specs, we create a paintedtextNode. And we reach step 10, and should report first-paint

TC3: data:text/html, <div style="height: 100px; background: yellow; opacity: 0">
FP: Same as TC2

Proposed Behavior:
It should check if the element is paintable and then mark first-paint.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions