LearnNewsExamplesServices
Frontmatter
id6210
titlecomponent.Base: update(), promiseUpdate() => critical rendering path
stateClosed
labels
enhancement
assigneestobiu
createdAtJan 13, 2025, 12:05 AM
updatedAtJan 13, 2025, 12:34 AM
githubUrlhttps://github.com/neomjs/neo/issues/6210
authortobiu
commentsCount1
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtJan 13, 2025, 12:34 AM

component.Base: update(), promiseUpdate() => critical rendering path

Closed v8.3.0 enhancement
tobiu
tobiu commented on Jan 13, 2025, 12:05 AM
  • very similar to the recent changes for render()
  • an update can insert new components (not used component types) into the component tree
  • these cmps can have CSS file requirements
  • the update should get delayed, in case the related CSS files are not fully loaded
  • callbacks must not get lost
tobiu added the enhancement label on Jan 13, 2025, 12:05 AM
tobiu assigned to @tobiu on Jan 13, 2025, 12:05 AM
tobiu
tobiu Jan 13, 2025, 12:30 AM
  • this logic can get added inside just 1 spot: updateVdom()
  • the check should only happen in case all other update requirements have successfully passed.

https://github.com/user-attachments/assets/a7d9629a-2787-44c4-b03d-a5461d925853

tobiu referenced in commit 24aac85 - "component.Base: update(), promiseUpdate() => critical rendering path #6210" on Jan 13, 2025, 12:31 AM
tobiu closed this issue on Jan 13, 2025, 12:34 AM