Fix focus layer not showing up in diff view The focus layer was not showing up in the diff view because the `annotated` property was not being set to true. This property is used to determine whether or not the focus layer should be shown. Change-Id: I29f3037653be62e669140c1916601d3a69e3ec85 Release-Notes: skip Google-Bug-Id: b/345126277
diff --git a/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer.ts b/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer.ts index b3d780e..6d55b0f 100644 --- a/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer.ts +++ b/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer.ts
@@ -83,6 +83,7 @@ _line: GrDiffLine, side: Side ) { + this.annotated = true; if (!lineNumberEl || !textEl || !this.diffRangesToFocus) { return; }
diff --git a/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer_test.ts b/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer_test.ts index fc61d21..f76ddef 100644 --- a/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer_test.ts +++ b/polygerrit-ui/app/embed/diff/gr-focus-layer/gr-focus-layer_test.ts
@@ -117,6 +117,11 @@ layer.setRanges(RANGES); }); + test('annotated is true after annotate', () => { + assert.isFalse(hasOutOfFocusClass(1, Side.LEFT)); + assert.isTrue(layer.annotated); + }); + test('line 1-2 are focussed on both sides', () => { assert.isFalse(hasOutOfFocusClass(1, Side.LEFT)); assert.isFalse(hasOutOfFocusClass(2, Side.RIGHT));