layer5io
fix/Multiple-Overlapping-Tooltips-Appear-on-Hover#6445
Description
This PR fixes #6445
Notes for Reviewers
- [✅] Yes, I signed my commits.
🚀 Preview for commit 294474974ddf5ba46fbf865b392bb0165a75254a at: https://684786ba70e65ad94994bfba--layer5.netlify.app
Hi @abbiekuma I am not seeing service mesh interface compliance table on the deployed link, can you check?
Hi @abbiekuma I am not seeing service mesh interface compliance table on the deployed link, can you check?
Hi @LibenHailu ! While working on the feature, I noticed that the Service Mesh Interface compliance table doesn’t appear unless data is successfully fetched from the API due to CORS restrictions. To address this and make the component visible during development, I used src/collections/landscape/smi.js/smi_data as mock data to view the table, and the issue was successfully resolved in my local environment.
https://github.com/user-attachments/assets/994bc05b-2923-48e8-aea3-e30454dc6526
local environment.
@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?
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
The reason you're not seeing the SMI table is becuase the data fetch is being blocked by a CORS policy. For now, I’m using mock data locally so I can continue development so that's why it shows up on my end but not yours.
The reason you're not seeing the SMI table is becuase the data fetch is being blocked by a CORS policy. For now, I’m using mock data locally so I can continue development so that's why it shows up on my end but not yours.
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👇
https://github.com/user-attachments/assets/628eeebb-78d7-4b49-9243-878fe889c8b0
@abbiekuma Thank you for your contribution! Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT
Add it as an agenda item to the meeting minutes, if you would :)
🚀 Preview for commit 23e57555b32bc98f78e82af7f8d0d6ff5e85becd at: https://684ffe1d08ec02c8f6f58afe--layer5.netlify.app
🚀 Preview for commit e5f02bab508dae3fa7dc3cf7062ee0bfc317c3a4 at: https://68599115186515835549a12c--layer5.netlify.app
Here is how it looks in mobile view. @vishalvivekm @vr-varad I also updated the format. The previous formatting was changed by Prettier, but I’ve now adjusted it back to the original style.
https://github.com/user-attachments/assets/e564de81-b7d8-4e0f-ad0d-b40d9b072040
🚀 Preview for commit 749aa4c889b56648ee67753300c80bcf2c0f3d60 at: https://685bcee22a8410e58b79bf5e--layer5.netlify.app