From 74d583d344da92d58c74b7144a4c963d24af1caa Mon Sep 17 00:00:00 2001
From: Pavel <flashworking@gmail.com>
Date: Tue, 27 Sep 2016 15:29:52 +0300
Subject: [PATCH] Nested menu position inside the page with scroll

If the page has vertical scroll, the topCoordinate = 0 is not working and we should calculate offset including scroll offset to position long-height menus properly.
---
 contextMenu.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/contextMenu.js b/contextMenu.js
index 0c8905d..4333b37 100644
--- a/contextMenu.js
+++ b/contextMenu.js
@@ -120,8 +120,8 @@ angular.module('ui.bootstrap.contextMenu', [])
             } else if(winHeight <= menuHeight) {
                 // If it really can't fit, reset the height of the menu to one that will fit
                 angular.element($ul[0]).css({"height": winHeight - 5, "overflow-y": "scroll"});
-                // ...then set the topCoordinate height to 0 so the menu starts from the top
-                topCoordinate = 0;
+                // ...then set the topCoordinate height to top of window inclluding scroll offset so the menu starts from the top
+                topCoordinate = (window.pageYOffset || document.documentElement.scrollTop) - (document.documentElement.clientTop || 0);
             } else if(winHeight - topCoordinate < menuHeight) {
                 var reduceThreshold = 5;
                 if(topCoordinate < reduceThreshold) {