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

Collision detection for Selector dropdowns #64

Closed
steven-tey opened this issue Jun 29, 2023 · 11 comments
Closed

Collision detection for Selector dropdowns #64

steven-tey opened this issue Jun 29, 2023 · 11 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@steven-tey
Copy link
Owner

Currently, the Color and Node Selectors is unable to detect when it's near the bottom of the screen and has to be opened the other way round:

CleanShot 2023-06-29 at 08 23 35

Using something like a Radix Popover could fix this:

CleanShot.2023-06-29.at.08.25.00.mp4

We'll also want to make sure to limit the height of the dropdown and add overflow-scroll depending on the screen height.

@steven-tey steven-tey added the help wanted Extra attention is needed label Jun 29, 2023
@abhijain1705
Copy link

absolutely agree, that last dropdown its dfficult to scroll webpage and read menu

would be helpful if we set fixed height and overflow: auto

Novel.Notion-style.WYSIWYG.editor.with.AI-powered.autocompletions.-.Google.Chrome.2023-06-29.22-24-42.mp4

can I work on this?

@abhijain1705
Copy link

is that repo is still active?

@steven-tey
Copy link
Owner Author

@abhijain1705 don't think a fixed height is the solution here, since you wouldn't know what position the bubble menu is initiated in the DOM and it might still collide with the screen boundaries

is that repo is still active?

Which repo are you referring to?

@abhijain1705
Copy link

then what can be best the solution here?
can we get current position of whole edit menu?

@steven-tey
Copy link
Owner Author

Converting the selector menu into a Radix popover primitive should fix this – though it might collide with some of the native Tiptap event listeners somehow

@abhijain1705
Copy link

it's like redeveloping the menu using Radix popover
correct me if I am wrong

@steven-tey
Copy link
Owner Author

Exactly!

@abhijain1705
Copy link

can I contribute to this?

@abhijain1705
Copy link

hey @steven-tey you might busy previously
can I contribute to this?

@steven-tey
Copy link
Owner Author

@abhijain1705 yes, of course! :)

@abhijain1705
Copy link

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants