Skip to content

fix: context menu dynamic positioning, multiple context menus opening issue#1913

Merged
sriramveeraghanta merged 2 commits intodevelopfrom
fix/context_menu
Aug 19, 2023
Merged

fix: context menu dynamic positioning, multiple context menus opening issue#1913
sriramveeraghanta merged 2 commits intodevelopfrom
fix/context_menu

Conversation

@aaryan610
Copy link
Member

@aaryan610 aaryan610 commented Aug 18, 2023

1. Fixed multiple context menus opening on the right click

Before the fix-

Plane._.Simple.extensible.open-source.project.management.tool.and.5.more.pages.-.Personal.-.Microsoft.Edge.2023-08-19.00-33-11.mp4

After the fix-

Plane._.Simple.extensible.open-source.project.management.tool.and.6.more.pages.-.Personal.-.Microsoft.Edge.2023-08-19.00-44-46.mp4

2. Fixed context menu hiding if there's not enough space to render

image image

Other fixes-

  1. Refactored props structure of the context menu component.
  2. Refactored useOutsideClickDetector hook to call the callback function on mousedown instead of click.

@aaryan610 aaryan610 added 🐛bug Something isn't working 🌐frontend labels Aug 18, 2023
@vercel
Copy link

vercel bot commented Aug 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
plane ❌ Failed (Inspect) Aug 18, 2023 7:25pm
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
plane-dev ⬜️ Ignored (Inspect) Visit Preview Aug 18, 2023 7:25pm
plane-space-dev ⬜️ Ignored (Inspect) Visit Preview Aug 18, 2023 7:25pm
plane-space-stage ⬜️ Ignored (Inspect) Visit Preview Aug 18, 2023 7:25pm
plane-staging ⬜️ Ignored (Inspect) Visit Preview Aug 18, 2023 7:25pm

@aaryan610 aaryan610 changed the title fix: context menu dynamic positioning, multiple context menus fix: context menu dynamic positioning, multiple context menus opening issue Aug 18, 2023
@sriramveeraghanta sriramveeraghanta merged commit cebc8bd into develop Aug 19, 2023
@sriramveeraghanta sriramveeraghanta deleted the fix/context_menu branch August 19, 2023 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛bug Something isn't working 🌐frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants