LearnNewsExamplesServices
Frontmatter
id4683
titlecomponent.Base: explore & fix inconsistencies between the mounted state & having a vnode
stateClosed
labels
enhancementstale
assignees[]
createdAtAug 9, 2023, 3:35 PM
updatedAtSep 13, 2024, 4:29 AM
githubUrlhttps://github.com/neomjs/neo/issues/4683
authortobiu
commentsCount3
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtSep 13, 2024, 4:29 AM

component.Base: explore & fix inconsistencies between the mounted state & having a vnode

Closed v8.1.0 enhancementstale
tobiu
tobiu commented on Aug 9, 2023, 3:35 PM

we can most likely simplify this a lot, in case we use the assumption: if a component has a vnode (real DOM), it has to be mounted.

then we could use afterSetVnode() as the single source of truth to change the value of the mounted config.

tobiu added the enhancement label on Aug 9, 2023, 3:35 PM
tobiu
tobiu Aug 9, 2023, 3:39 PM

hmm, actually there is one exception: you can render a component (creating a vnode) without mounting it. but as soon as we come into the update cycle, the assumption is correct.

bigger question: do we even need render without mounting.

@github-actions - 2024-08-29T02:26:48Z

This issue is stale because it has been open for 90 days with no activity.

  • 2024-08-29T02:26:49Z @github-actions added the stale label

@github-actions - 2024-09-13T02:29:18Z

This issue was closed because it has been inactive for 14 days since being marked as stale.

  • 2024-09-13T02:29:18Z @github-actions closed this issue