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

Lists: Multiple hover cards visible at the same time #2460

Open
sefsh opened this issue Jan 9, 2025 · 3 comments
Open

Lists: Multiple hover cards visible at the same time #2460

sefsh opened this issue Jan 9, 2025 · 3 comments
Assignees
Labels
🐜 bug Something isn't working

Comments

@sefsh
Copy link

sefsh commented Jan 9, 2025

Description

Steps to reproduce

  1. Go to the People page
  2. Create a smart search list
  3. Add some people who have replied to a survey
  4. Add Single survey question column(s)
  5. Quickly hover survey responses

Expected Behaviour

For one hover card to be visible at one time.

Actual Behaviour

Multiple hover cards are visible at one time.

Screenshots (if you have any)

Screen.Recording.2025-01-09.at.13.30.01.mov
@sefsh sefsh added the 🐜 bug Something isn't working label Jan 9, 2025
@helihard
Copy link
Collaborator

helihard commented Jan 14, 2025

@richardolsson I have tried manipulating the delay directly in the useToggleDebounce function, as well as having the function take the delay as an argument, as possible solutions to this issue. Not sure this is the correct approach, also because we have a comment in the code stating that the delay is there for a reason. What do you think?

@helihard helihard self-assigned this Jan 14, 2025
@richardolsson
Copy link
Member

@richardolsson I have tried manipulating the delay directly in the useToggleDebounce function, as well as having the function take the delay as an argument, as possible solutions to this issue. Not sure this is the correct approach, also because we have a comment in the code stating that the delay is there for a reason. What do you think?

I believe the delay is there for a reason, to prevent a popper from closing when the user accidentally moves the cursor outside while going from the cell to the popper.

I believe the only way to solve this is to either have just one popper shared by all cells (which will likely introduce other problems) or to have some sort of shared state (in context).

@helihard
Copy link
Collaborator

Letting this one go because it turned out to be more complicated than it seemed at first glance, will be happy to learn from someone else's solution later on!

@helihard helihard removed their assignment Jan 19, 2025
@awarn awarn self-assigned this Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐜 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants