daybrush / moveable

Moveable! Draggable! Resizable! Scalable! Rotatable! Warpable! Pinchable! Groupable! Snappable!
https://daybrush.com/moveable/
MIT License
10.08k stars 618 forks source link

Issue with the translateZ(50px) moveable-control-box on safari #302

Closed tannguyen248 closed 4 years ago

tannguyen248 commented 4 years ago

Environments

I don't know why we should have the translateZ(50px) but on Safari, it works as the z-index and covers all my material-ui popup (with translateZ(0px). The image below describes the issue, do you have any idea with it?

image

daybrush commented 4 years ago

@tannguyen248

I wish there was a demo.

The reason I set translateZ to 50px is that the moveable component should always be shown in front. So that picture is normal.

tannguyen248 commented 4 years ago

@daybrush day so do we have any way to override the translateZ?

tannguyen248 commented 4 years ago

I cannot override the control box translateZ because it's added by every single move, so I override the material-ui to work-around with this issue.

daybrush commented 4 years ago

moveable's new version is released. translateZ prop is added. Thank you :)

tannguyen248 commented 4 years ago

Thank you ;)