Wojciech Szymański
Wojciech Szymański
After the discussion of today, if this PR is not a minor version it should not be carried over to milestone `12.1.3`.
I can see that there is one not passing tests 🤯. I'll check it yet.
@budnix It's ready for CR.
> I think that the PR fixes the issue only superficially. The root of the problem is elsewhere. I investigated that and I believe the issue is here, [in that...
> I think that the PR fixes the issue only superficially. The root of the problem is elsewhere. I investigated that and I believe the issue is here, [in that...
> There is still a problem. After your changes, the dataset that does not belong to the merged cells is still updated. > > According to the screenshot: ![Zrzut ekranu...
As @warpech wrote, this bug is only replicable in `iframe`. The bug is related to the fact that drop-down menu (or context menu) has greater height than `iframe` (see below)....
I've tried experimenting with calling `adjustElementsSize` after calling the below `init` which shows scroll. https://github.com/handsontable/handsontable/blob/28ca7c0d8c07eaef6368e5585a6129b95c57bbff/handsontable/src/plugins/contextMenu/menu.js#L264 I also tried to change the below code. I've tried to detect whether there is...
As discussed with @warpech and @budnix I'm changing the assignee. edit: @budnix already did it :)
I would not describe `false` option. We don't do it probably. I would mention default configuration when `true` is handled. Has the plugin any influence on the way how hiding...