commit d8f2f8e05fd6f84af2ff55848271b63c100bb9fd
author Dhruv <dhruvsri@google.com> Thu May 19 17:01:49 2022 +0200
committer Paladox none <thomasmulhall410@yahoo.com> Fri May 20 13:22:01 2022 +0000
tree 9de509f5151c9a3f70504b2f8c3258dc3c5d95ee
parent 24b3a47524cfd2e3c3a5e8b946398345765cda2c

Add check for syntax highlighting in diff prefs changed

Release-Notes: skip
Google-bug-id: b/233199790
Change-Id: I1883813f11801e5bbddc033fc149731c102dc925
(cherry picked from commit 1c425f8f5cea3457d96cc9a32814612e06ec14e4)

polygerrit-ui/app/elements/shared/gr-diff-preferences/gr-diff-preferences.ts

diff --git a/polygerrit-ui/app/elements/shared/gr-diff-preferences/gr-diff-preferences.ts b/polygerrit-ui/app/elements/shared/gr-diff-preferences/gr-diff-preferences.ts
index 5fd7db4..44b6fa2 100644
--- a/polygerrit-ui/app/elements/shared/gr-diff-preferences/gr-diff-preferences.ts
+++ b/polygerrit-ui/app/elements/shared/gr-diff-preferences/gr-diff-preferences.ts
@@ -301,6 +301,8 @@
     // We have to wrap boolean values in Boolean() to ensure undefined values
     // use false rather than undefined.
     return (
+      Boolean(this.originalDiffPrefs?.syntax_highlighting) !==
+        Boolean(this.diffPrefs?.syntax_highlighting) ||
       this.originalDiffPrefs?.context !== this.diffPrefs?.context ||
       Boolean(this.originalDiffPrefs?.line_wrapping) !==
         Boolean(this.diffPrefs?.line_wrapping) ||