Merge branch...

Merge branch '60369-border-radius-still-showing-up-after-improve-diff-nav-header-was-finished' into 'master'

Breakpoints.isDesktop should not always return false

Closes #60369

See merge request gitlab-org/gitlab-ce!27240

(cherry picked from commit 44be329d)

9dca4f12 Fix isDesktop function to return proper value

app/assets/javascripts/breakpoints.js

@@ -15,7 +15,7 @@ const BreakpointInstance = {
     return breakpoint;
   },
   isDesktop() {
-    return ['lg', 'md'].includes(this.getBreakpointSize);
+    return ['lg', 'md'].includes(this.getBreakpointSize());
   },
 };
 

spec/javascripts/breakpoints_spec.js

@@ -10,4 +10,18 @@ describe('breakpoints', () => {
       expect(bp.getBreakpointSize()).toBe(key);
     });
   });
+
+  describe('isDesktop', () => {
+    it('returns true when screen size is medium', () => {
+      spyOn(bp, 'windowWidth').and.returnValue(breakpoints.md + 10);
+
+      expect(bp.isDesktop()).toBe(true);
+    });
+
+    it('returns false when screen size is small', () => {
+      spyOn(bp, 'windowWidth').and.returnValue(breakpoints.sm + 10);
+
+      expect(bp.isDesktop()).toBe(false);
+    });
+  });
 });