Fix #5912: Resource Manger Context Menu Doesn't reposition after scroll.

dnnsoftware · Jan 15, 2024 · 761ea63 · 761ea63
1 parent 5df0e9d
commit 761ea63
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/...urceManager/ResourceManager.Web/src/components/dnn-rm-folder-list/dnn-rm-folder-list.scss b/...urceManager/ResourceManager.Web/src/components/dnn-rm-folder-list/dnn-rm-folder-list.scss
@@ -1,6 +1,7 @@
 :host {
   display: block;
   margin-left: 1em;
+  position: relative;
 }
 
 button{
@@ -12,7 +13,7 @@ button{
 }
 
 dnn-collapsible{
-  position: fixed;
+  position: absolute;
   display: none;
   border: 1px solid lightgray;
   box-shadow: 2px 2px 4px -2px;

diff --git a/...ourceManager/ResourceManager.Web/src/components/dnn-rm-folder-list/dnn-rm-folder-list.tsx b/...ourceManager/ResourceManager.Web/src/components/dnn-rm-folder-list/dnn-rm-folder-list.tsx
@@ -97,14 +97,17 @@ export class DnnRmFolderList {
 
   private handleContextMenu(e: MouseEvent): void {
     e.preventDefault();
+    this.dismissContextMenu();
+
     this.itemsClient.getFolderItem(state.settings.HomeFolderId)
     .then(item => {
       const collapsible = document.createElement("dnn-collapsible");
       const folderContextMenu = document.createElement("dnn-rm-folder-context-menu");
       collapsible.appendChild(folderContextMenu);
       folderContextMenu.item = item;
-      collapsible.style.left = `${e.pageX}px`;
-      collapsible.style.top = `${e.pageY}px`;
+      const parentPosition = this.el.getBoundingClientRect();
+      collapsible.style.left = `${e.clientX - parentPosition.left}px`;
+      collapsible.style.top = `${e.clientY - parentPosition.top}px`;
       collapsible.style.display = "block";
       this.el.shadowRoot.appendChild(collapsible);
       setTimeout(() => {