Skip to content
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

[resize-observer] The monkey patching of the event loop processing model appears to be outdated #4824

Closed
rniwa opened this issue Mar 4, 2020 · 2 comments · Fixed by #7823

Comments

@rniwa
Copy link

rniwa commented Mar 4, 2020

The entire section about HTML event loop processing model appears to be outdated.

I think this section is trying to monkey patch step 14 of updating the rendering step in the current processing model.

The spec also mentions steps like "recall styles" and "update layout". Since these aren't concepts formally defined anywhere, it's probably best to consolidate as a single step to generate or update the CSS boxes and their geometry or something instead too.

@fantasai fantasai changed the title [css-resizeobserver] The monkey patching of the event loop processing model appears to be outdated [resize-observer] The monkey patching of the event loop processing model appears to be outdated Mar 11, 2020
@domenic
Copy link
Collaborator

domenic commented Mar 24, 2022

This is especially problematic because it is unclear when ResizeObserver callbacks happen in relationship to the other things that happen during "update the rendering", such as:

  • requestAnimationFrame callbacks
  • IntersectionObserver callbacks
  • Paint timing
  • Web animations events
  • scroll and resize events
  • autofocus

If someone could properly upstream this monkeypatch to HTML, with tests, that would be much appreciated.

@domenic
Copy link
Collaborator

domenic commented Oct 4, 2022

/cc @emilio I think this can now be resolved by removing the section as I noted in whatwg/html#8348 (comment) .

emilio added a commit to emilio/csswg-drafts that referenced this issue Oct 4, 2022
emilio added a commit that referenced this issue Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants