restrict vertical resize with sizeToContent #2612
Conversation
* since sizeToContent:true will automatically size the content vertically, don't let the user change the height to start with Note: tried to change the feedback to have e,w arrows but we only have 10px area which is too narrow to show feedback (we have 20x20 for corners that may overlap content which is ok). Showing normal handles for now.
|
Hi @adumesny as far as I can see, this change prevents a user from resizing a widget vertically even if On the demo page (https://gridstackjs.com/demo/sizeToContent.html) it is still possible to resize widget E, but maybe the demo site uses an older version of grid stack? I didn't manage to understand what version is used on the demo site. |
|
It's a bug. Number should let you resize that's the all point. I made the tweak too fast... |
fix gridstack#2612 to allow resize when you have a number (soft max)
fix gridstack#2612 to allow resize when you have a number (soft max)
|
This is a breaking change. I am yet to understand if |
|
@yusufkandemir it is not a bug, but by design, incorrect before. |
Description
Note: tried to change the feedback to have e,w arrows but we only have 10px area which is too narrow to show feedback (we have 20x20 for corners that may overlap content which is ok). Showing normal handles for now.
Checklist
yarn test)