drawdb-io / drawdb

Free, simple, and intuitive online database design tool and SQL generator.

https://drawdb.app

MIT License

12.11k stars 856 forks source link

Fixed field circle icons not moving in Safari #35

Closed ClaPalo closed 2 months ago

ClaPalo commented 2 months ago

Fix to #32. For some reason, the problem seems to be related to the opacity of the buttons. I fixed it by setting the opacity back to 1, which doesn't seem to ruin the UX.

vercel[bot] commented 2 months ago

@ClaPalo is attempting to deploy a commit to the dottle's projects Team on Vercel.

A member of the Team first needs to authorize it.

HaecheonLee commented 2 months ago

The issue seems to be related with the style of transform set to g in Canvas.jsx. And the transform state is set by diagram. Personally, removing those style should not be relevant to the position of an element. We may need further investigation to solve it properly.

ClaPalo commented 2 months ago

It seems that by using RGBA for the colors, it's possible to set to opacity without creating the issue.

HaecheonLee commented 2 months ago

This seems to be the solution to fix it. I cannot find any other approach. I created a PR to fix another issue that renders the buttons in the wrong position.

44

HaecheonLee commented 2 months ago

Fix seems to be required for Notes and subject area as well. I will open another issue for this.

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
drawdb	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 14, 2024 7:01am