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.

Sign up for GitHub

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

Jump to bottom

fixed the issue related to the tooltip preivew on mouseEvent #9704

Merged
merged 1 commit into from
Oct 30, 2024
Merged

fixed the issue related to the tooltip preivew on mouseEvent #9704

merged 1 commit into from
Oct 30, 2024

Conversation

KaleemNeslit
Copy link
Collaborator

@KaleemNeslit KaleemNeslit commented Oct 28, 2024 •
edited
Loading

Link to Issue

Closes: #9651

Description of Changes

  • Improved tooltip behaviour by conditionally setting anchorEl based on event type, ensuring responsive show/hide functionality.
  • added the
    wrapper in onMouseEnter events was not triggering on that component .

"How We Fixed It"

  • Streamlined the handleInteraction function to toggle anchorEl using the e.type check .
  • Adjusted usage within the QuickTokenLaunchForm page by placing the interaction logic directly on the containing
    , improving encapsulation and simplifying component interaction management

Test Plan

  • goto the communities and go to lunch token and click the randomize more than 5 times you can see the tooltip behaviour tip will appear onMouseLeaveEvent .
Screen.Recording.2024-10-28.at.11.51.58.PM.mov

@KaleemNeslit KaleemNeslit merged commit afeefe9 into master Oct 30, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mzparacha mzparacha mzparacha approved these changes

@masvelio masvelio masvelio approved these changes

@burtonator burtonator Awaiting requested review from burtonator

@Israellund Israellund Awaiting requested review from Israellund

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix tooltip reverse state issues.
3 participants
@KaleemNeslit @masvelio @mzparacha