ckeditor4 icon indicating copy to clipboard operation
ckeditor4 copied to clipboard

Table columns resize cursor disappears after doing horizontal scroll

Open usmannoor10p opened this issue 4 years ago • 2 comments

Type of report

Bug

Provide detailed reproduction steps (if any)

  • Resize Column 1 you will see the resizing cursor
  • Now move scroll to the extreme right and try to resize Column 4, the resizing cursor will disappear

You can see it from the following attachment:

ezgif com-gif-maker

Expected result

The table column resize cursor should not get disappeared

Actual result

The table column resizing cursor is disappearing

Other details

  • Browser: Version 91.0.4472.164 (Official Build) (64-bit)
  • OS: Windows 10 Version 1090 (OS Build 18363.836)
  • CKEditor version: 4.14.1
  • Installed CKEditor plugins: table, tableresize, tableselection

Code Sample

resize_scroll_focus_bug.zip

usmannoor10p avatar Sep 16 '21 10:09 usmannoor10p

I can confirm the issue. However, it seems to be caused by the fact that resize handlers' positions are not refreshed after scroll:

https://user-images.githubusercontent.com/1078728/133621786-88d1f4c1-a444-4161-9bc7-12df947455b7.mov

After scrolling the handler for the first cell was still in its original position and I was able to resize that cell even if it wasn't in the viewport anymore.

Comandeer avatar Sep 16 '21 13:09 Comandeer

@Comandeer thanks for acknowledging. Will there be any fix for this issue in the future? Or is there any workaround then please do let me know.

Thanks

usmannoor10p avatar Sep 17 '21 05:09 usmannoor10p

~~After some more research I discovered that it's not strictly connected with the scrolling but more with the fact that the table is inside an element with the position: relative style. It seems to somehow alter the results of calculations of pillars positions (probably in the CKEDITOR.dom.element#getDocumentPosition() method).~~

The position: relative is not needed after all, however, the issue is present only in inline editors, I can't reproduce it in an iframe-based one.

Comandeer avatar Aug 29 '22 15:08 Comandeer

Closed in https://github.com/ckeditor/ckeditor4/pull/5318

CKEditorBot avatar Sep 13 '22 09:09 CKEditorBot