IgniteUI / igniteui-webcomponents

Ignite UI for Web Components is a complete library of UI components, giving you the ability to build modern web applications using encapsulation and the concept of reusable components in a dependency-free approach.
https://www.infragistics.com/products/ignite-ui-web-components
Other
113 stars 2 forks source link

IgcTreeGrid - resizing the width from scrolled to the right caused errors and the order of the columns was broken #952

Open jsakamotoIGJP opened 8 months ago

jsakamotoIGJP commented 8 months ago

Description

Resizing the width of the IgcTreeGrid from scrolling to the right caused errors, and the order of the columns was broken.

Steps to reproduce

  1. Shrink the width of the IgcTreeGrid.
  2. Scroll to the right of the scrolling area of the IgcTreeGrid.
  3. Widen the width of the IgcTreeGrid.

Result

Some errors were reported in the browser's dev console. And the order of the columns was broken.

Expected result

Nothing should happen.

Attachments

See also the video below.

https://github.com/IgniteUI/igniteui-webcomponents/assets/102948935/f5e54aba-52bf-43f0-923f-2994afed1847

The project that can reproduce this issue is below.

📦c-00233104-html-app1.zip

mddifilippo89 commented 6 months ago

Still occurring latest internal WC. See attached sample using @infragistics/build 23.2.90 (note packages were moved up a folder).

image

overview.zip

kdinev commented 6 months ago

@damyanpetev Which branch are the igc-grids produced from?

dkamburov commented 6 months ago

@damyanpetev Which branch are the igc-grids produced from?

blazor-angular-elements

kdinev commented 6 months ago

In order for the fix to appear then, master needs to be merged into blazor-angular-elements

damyanpetev commented 6 months ago

The branch is quite a bit behind and syncing in full can have some unforeseen effects and require more revalidation - still agree it needs to be done as soon as possible, but for this fix specifically being very small I can cherry pick it directly.

damyanpetev commented 6 months ago

Done. Next internal build should have this fix in

kdinev commented 6 months ago

@damyanpetev Let's plan merging the elements branch at the beginning of 2024. Preferably in January.

damyanpetev commented 6 months ago

Yes, please :)