Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Draggable Functionality for Main Menu Positioning #351

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

forumi0721
Copy link
Contributor

This update adds a draggable feature to allow users to reposition the main menu.
The change was made to resolve issues where elements in fixed positions were obstructing visibility.

Changes Made:

  • Implemented draggable functionality for the main menu
  • Ensured the menu retains its position after movement

@polyzium
Copy link
Contributor

My feedback on this:

  • Once the menu is closed, releasing LMB when dragging makes the menu open, and when you drag it to the side, the close button is unreachable because it is out of bounds.
  • Clicking on any part of the menu, including dropdowns, makes it drag (the dropdowns, however, stay). It would be better if you would add a separate "drag" handle on the side of the menu.
  • When resizing the window, the menu may go out of bounds and therefore may be unreachable.
  • Snapping functionality would be very nice to have considering how often you'd be moving this around.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants