Skip to content

Razor Components: Differ for deltas (renderer) ignores IDs #8954

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.

Sign up for GitHub

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

Jump to bottom
Closed
springy76 opened this issue Mar 31, 2019 · 2 comments
Closed

Razor Components: Differ for deltas (renderer) ignores IDs #8954

springy76 opened this issue Mar 31, 2019 · 2 comments
Labels
area-blazor Includes: Blazor, Razor Components ✔️ Resolution: Duplicate Resolved as a duplicate of another issue

Comments

@springy76
Copy link

springy76 commented Mar 31, 2019 •
edited
Loading

Using 3.0 preview3 of Razor Components (server-side)

Sample: https://github.com/springy76/RazorComponentsProblems/blob/problem/DifferIgnoresIDs/WebApp/Components/Pages/Index.razor

Because the differ does ignore IDs when removing the first node of a list, instead of really removing the first node then all nodes get its ID (and all attributes and children) reassigned and finally the last one is removed.
The side effect of this (wrong) behavior is that not only far too much deltas are transported, but also all sorts of CSS animations and transitions may be run.
@Eilon Eilon added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Apr 1, 2019
@mkArtakMSFT mkArtakMSFT added the area-blazor Includes: Blazor, Razor Components label Apr 1, 2019
@danroth27
Copy link
Member

@SteveSandersonMS Will support for key address this?

@mkArtakMSFT
Copy link
Member

This is dupe of #8232

@mkArtakMSFT mkArtakMSFT added the ✔️ Resolution: Duplicate Resolved as a duplicate of another issue label Apr 3, 2019
@mkArtakMSFT mkArtakMSFT removed area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates labels May 9, 2019
@Andrzej-W Andrzej-W mentioned this issue Jul 5, 2019
Closed
@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-blazor Includes: Blazor, Razor Components ✔️ Resolution: Duplicate Resolved as a duplicate of another issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Eilon @danroth27 @springy76 @mkArtakMSFT