Bugix : `Elements.IndexOf(element)` value should be checked to avoid crashes

[pcarret]

When removing elements and adding new one then removing, it can crash

in SVG.razor.cs

        protected override void OnInitialized()
        {

            ElementSubject
                .Buffer(TimeSpan.FromMilliseconds(33))
                .Where(updates => updates.Count > 0)
                .Subscribe(updates =>
                {
                    updates
                        .Distinct()
                        .ToList()
                        .ForEach(element =>
                        {
                            ElementsAsHtml[Elements.IndexOf(element)] = element.ToHtml();
                        });
                    UpdateInput();
                });
        }

Don't you think Elements.IndexOf(element) value should be checked to secure the code and avoid crashes ?

[KristofferStrube]

I think the solution to this problem is to make Elements private and add accessors that ensure ElementsAsHtml is updated accordingly when Elements is changed.

I will look into this.

[KristofferStrube]

I have separated the responsibility of the generated Html to each ISVGElement so that SVG doesn't have to handle that. So some nice Separation of Concerns.