-
Notifications
You must be signed in to change notification settings - Fork 1.3k
fix/Multiple-Overlapping-Tooltips-Appear-on-Hover#6445 #6532
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
base: master
Are you sure you want to change the base?
fix/Multiple-Overlapping-Tooltips-Appear-on-Hover#6445 #6532
Conversation
Signed-off-by: Yu Luo <abbieluo.yu@gmail.com>
🚀 Preview for commit 2944749 at: https://684786ba70e65ad94994bfba--layer5.netlify.app |
|
Hi @LibenHailu ! layer5_6445.mov |
@abbiekuma Hmm, it doesn't seem to be working in production. Could you take a look and see what might be causing the issue? |
Just to clarify, when you say it’s not working, are you referring to the table still being missing, or the hover overlap issue still persists? |
The table is missing, here is the preview, https://684786ba70e65ad94994bfba--layer5.netlify.app/service-mesh-landscape |
Can you load the data locally? if not we have to check the backend and update the CORS policy? |
Hello, I I followed the suggestion at Slack to open a new Chrome window with security options disabled. This bypassed the CORS restrictions, and it worked! the data loaded successfully. Here is the updated recording👇 updated_layer5_6445.mov |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abbiekuma Please revert these whitespace, formatting changes
@abbiekuma Add it as an agenda item to the meeting minutes, if you would :) |
🚀 Preview for commit 23e5755 at: https://684ffe1d08ec02c8f6f58afe--layer5.netlify.app |
Signed-off-by: Abbie - Yu Luo <149909888+abbiekuma@users.noreply.github.com>
🚀 Preview for commit e5f02ba at: https://68599115186515835549a12c--layer5.netlify.app |
Here is how it looks in mobile view. @vishalvivekm @vr-varad Screen.Recording.2025-06-23.at.1.37.12.PM.mov |
🚀 Preview for commit 749aa4c at: https://685bcee22a8410e58b79bf5e--layer5.netlify.app |
Description
This PR fixes #6445
Notes for Reviewers
Signed commits