Skip to content

Commit

Permalink
Do not trigger changes on setting the same value for visible action p…
Browse files Browse the repository at this point in the history
…roperty (#9051)
  • Loading branch information
andrewtelnov authored Nov 16, 2024
1 parent 225258f commit 17fd13c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 3 additions & 1 deletion packages/survey-core/src/actions/action.ts
Original file line number Diff line number Diff line change
Expand Up @@ -551,7 +551,9 @@ export class Action extends BaseAction implements IAction, ILocalizableOwner {
getRendererContext(locStr: LocalizableString): any { return this.owner ? this.owner.getRendererContext(locStr) : locStr; }

public setVisible(val: boolean): void {
this._visible = val;
if(this.visible !== val) {
this._visible = val;
}
}
public getVisible(): boolean {
return this._visible;
Expand Down
4 changes: 3 additions & 1 deletion packages/survey-core/src/itemvalue.ts
Original file line number Diff line number Diff line change
Expand Up @@ -452,7 +452,9 @@ export class ItemValue extends BaseAction implements ILocalizableOwner, IShortcu
return isVisible && visible;
}
protected setVisible(val: boolean): void {
this._visible = val;
if(this.visible !== val) {
this._visible = val;
}
}
protected getLocTitle(): LocalizableString {
return this.locText;
Expand Down

0 comments on commit 17fd13c

Please sign in to comment.