Skip to content

Commit 3814af0

Browse files
authored
Merge pull request #444 from UiPath/fix/columns_visibility
fix(grid-visibility-manager): consider only checked columns when computing `isDirty$`
2 parents 09abd13 + f67d016 commit 3814af0

File tree

5 files changed

+8
-5
lines changed

5 files changed

+8
-5
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
# v15.0.19 (2023-12-20)
2+
* **grid** change visibility-manager isDirty$
3+
14
# v15.0.18 (2023-12-20)
25
* **file-picker** rm nrgx/component dependency
36
* **chore** bump version to v15.0.17

package-lock.json

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "angular-components",
3-
"version": "15.0.18",
3+
"version": "15.0.19",
44
"author": {
55
"name": "UiPath Inc",
66
"url": "https://uipath.com"

projects/angular/components/ui-grid/src/managers/visibility-manager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ export class VisibilityManger<T extends IGridDataEntry> {
3737
isDirty$ = this.options$.pipe(
3838
filter(() => !!this._initial),
3939
map(o => ([o.map(this._mapToVisibleDiff), this._initial])),
40-
map(([current, initial]) => !isEqual(current, initial)),
40+
map(([current, initial]) => !isEqual(current?.filter(v => v.checked), initial?.filter(v => v.checked))),
4141
);
4242

4343
set columns(columns: UiGridColumnDirective<T>[]) {

projects/angular/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@uipath/angular",
3-
"version": "15.0.18",
3+
"version": "15.0.19",
44
"license": "MIT",
55
"author": {
66
"name": "UiPath Inc",

0 commit comments

Comments
 (0)