search

multinet-app / multilink

A node-link visualization built to work with the multinet ecosystem.
https://multinet.app
Apache License 2.0
4 stars 2 forks source link

Tooltip shows correctly towards the edges on the right and the bottom. #379

Closed nishita96 closed 1 year ago

nishita96 commented 1 year ago

Does this PR close any open issues?

Closes #313

Give a longer description of what this PR addresses and why it's needed

The tooltip used to go out of the page when towards the right side and the bottom. Now it shows correctly by taking the height and width of the tooltip dynamically and changing the location of the rectangle towards the opposite side incase it is going out of the screen window. Added 'useElementBounding' and changed 'tooltipPosition'.

Provide pictures/videos of the behavior before and after these changes (optional)

After

Screenshot 2023-08-07 at 3 22 28 PM Screenshot 2023-08-07 at 3 21 11 PM

Are there any additional TODOs before this PR is ready to go?

NO

netlify[bot] commented 1 year ago

Deploy Preview for next-multilink ready!

Name Link
Latest commit 7a1f06bc8a5b67435d016dc55922ea6fd44fca4c
Latest deploy log https://app.netlify.com/sites/next-multilink/deploys/64dbabe9db9ee3000827c52d
Deploy Preview https://deploy-preview-379--next-multilink.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

nishita96 commented 1 year ago
Screenshot 2023-08-14 at 10 33 04 AM

The tooltip was coming way below the desired location when the control panel was displayed. Made that change and pushed.