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

Don't break interactive components when their parent static content is updated #48764

Closed
SteveSandersonMS opened this issue Jun 13, 2023 · 0 comments · Fixed by #49238
Closed
Assignees
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-full-stack-web-ui Full stack web UI with Blazor
Milestone

Comments

@SteveSandersonMS
Copy link
Member

When streaming SSR, progressively-enhanced navigation, or progressively-enhanced form posts update part of the DOM dynamically, we have to make sure they behave sensibly when part of that DOM structure is an interactive component.

  • Min bar: destroy the old interactive component and activate a new one
  • Better: preserve the old interactive component, but ignore any new params supplied by the updated static content
  • Best: ... also pass in new params from the updated static content

We likely only have capacity for a relatively basic implementation here, so may be limited to the min bar.

@SteveSandersonMS SteveSandersonMS added area-blazor Includes: Blazor, Razor Components feature-full-stack-web-ui Full stack web UI with Blazor labels Jun 13, 2023
@SteveSandersonMS SteveSandersonMS self-assigned this Jun 13, 2023
@SteveSandersonMS SteveSandersonMS added this to the 8.0-rc1 milestone Jun 13, 2023
@mkArtakMSFT mkArtakMSFT added the enhancement This issue represents an ask for new feature or an enhancement to an existing one label Jun 29, 2023
@mkArtakMSFT mkArtakMSFT modified the milestones: 8.0-rc1, 8.0-preview7 Jul 11, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Aug 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-full-stack-web-ui Full stack web UI with Blazor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants