-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
fix(collectionAsync): hidden column does not load edit field selection #742
fix(collectionAsync): hidden column does not load edit field selection #742
Conversation
yeah don't use the |
@@ -1434,7 +1434,7 @@ export class SlickVanillaGridBundle { | |||
|
|||
if (this.slickGrid) { | |||
// find the new column reference pointer & re-assign the new editor to the internalColumnEditor | |||
const columns = this.slickGrid.getColumns(); | |||
const columns = this.columnDefinitions; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove this line and apply it directly as shown below
- const columns = this.columnDefinitions;
if (Array.isArray(columns)) {
- const columnRef = columns.find((col: Column) => col.id === column.id);
+ const columnRef = this.columnDefinitions.find((col: Column) => col.id === column.id);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that seems correct, are you going to be able to fix the unit tests or do you need help? Please don't comment out the unit tests, we need to keep track of them and not forget them please :)
Codecov Report
@@ Coverage Diff @@
## master #742 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 233 233
Lines 16437 16439 +2
Branches 5490 5490
=========================================
+ Hits 16437 16439 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
packages/vanilla-bundle/src/components/__tests__/slick-vanilla-grid.spec.ts
Outdated
Show resolved
Hide resolved
- replicate issue found and fixed in Slickgrid-Universal [PR 742](ghiscoding/slickgrid-universal#742)
- replicate issue found and fixed in Slickgrid-Universal [PR 742](ghiscoding/slickgrid-universal#742)
- replicate issue found and fixed in Slickgrid-Universal [PR 742](ghiscoding/slickgrid-universal#742)
- replicate issue found and fixed in Slickgrid-Universal [PR 742](ghiscoding/slickgrid-universal#742)
#855) * fix(collectionAsync): hidden column does not load edit field selection - replicate issue found and fixed in Slickgrid-Universal [PR 742](ghiscoding/slickgrid-universal#742)
@mcallegario |
Bug: Hidden column does not load edit field with selection options
The "this.slickGrid.getColumns()" method only returns the visible fields of the grid, so if the field that was configured to return a list of elements (collectionAsync) is hidden in the grid, it does not receive the list of elements.
note: Unfortunately after version 1.3.x I am no longer able to perform the "pnpm run build".