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

Consolidate tooltip components #75

Closed
dmfalke opened this issue Jun 8, 2022 · 3 comments · Fixed by VEuPathDB/web-monorepo#932
Closed

Consolidate tooltip components #75

dmfalke opened this issue Jun 8, 2022 · 3 comments · Fixed by VEuPathDB/web-monorepo#932
Assignees

Comments

@dmfalke
Copy link
Member

dmfalke commented Jun 8, 2022

We can start with moving this component to this repo.

A related follow-up task will be to convert all existing tooltips to use this one.

@jernestmyers suggests moving to a newer MUI version - might be better placement mechanism

@asizemore
Copy link
Member

See also #100

@dmfalke
Copy link
Member Author

dmfalke commented Jan 17, 2023

Should one of these issues be closed?

@chowington chowington self-assigned this Feb 6, 2024
@bobular bobular changed the title New component: tooltip Consolidate tooltip components Feb 14, 2024
@dmfalke
Copy link
Member Author

dmfalke commented Feb 14, 2024

Note that this issue includes a reference to improving the performance of tooltips when we have a large list of elements with tooltips: #127 (comment)

ETA I'm not suggesting that we address this at the same time as consolidation. I just want to keep the idea in our heads.

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 a pull request may close this issue.

3 participants