daybrush / moveable

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

can't resize more than 100px if max-width/height set to 100% #430

Closed thekhairul closed 2 years ago

thekhairul commented 3 years ago

Environments

Description

The resizable module doesn't seem to be working for items that have 'max-width' or 'max-height' set to 100%. I can resize it down below 100px but can't scale it up more than 100px.

https://www.loom.com/share/ea4ad6ad9959441d82e3109ff437dd99

daybrush commented 3 years ago

@thekhairul

thekhairul commented 3 years ago

Ah. Works like a charm. Thanks a bunch for the quick fix :slightly_smiling_face:

thekhairul commented 3 years ago

Hey, found a new issue with the latest release (0.24.1). Resizable not working if the immediate parent of the target has 'display:flex' and the target is normally positioned element (not absolute/fixed)

daybrush commented 3 years ago

@thekhairul

thekhairul commented 3 years ago

Great ! everything's working fine. FYI: a console.log in 'dragControlStart' function

daybrush commented 3 years ago

@thekhairul

Thank you :) i removed console.log.