Opened 4 years ago

Closed 4 years ago

#11823 closed Bug (fixed)

[IE8] TableResize throws error over scrollbar

Reported by: Piotr Jasiun Owned by: Piotr Jasiun
Priority: Normal Milestone: CKEditor 4.4.1
Component: General Version: 4.3.4
Keywords: Cc:

Description (last modified by Piotr Jasiun)

  1. Open table-resize sample.
  2. Add some empty lines so the scrollbar will show.
  3. Drag last pillar and move it so the mouse will be over the scrollbar.

Result: Error multiple times in the console:

Object doesn't support this property or method

First bad commit: git:4991411.

I wasn't able to reproduce it on IE9, IE11 and Chrome.

Change History (9)

comment:1 Changed 4 years ago by Piotr Jasiun

Description: modified (diff)

comment:2 Changed 4 years ago by Piotr Jasiun

Milestone: CKEditor 4.4.1

comment:3 Changed 4 years ago by Marek Lewandowski

Status: newconfirmed

comment:4 Changed 4 years ago by Piotr Jasiun

Owner: set to Piotr Jasiun
Status: confirmedassigned

comment:5 Changed 4 years ago by Piotr Jasiun

Status: assignedreview

It was possible to reproduce this bug also on FF.

The reason was missing events target document when the mouse was outside the editor. We could switch back to using editable instead of document (see first bad commit) but the we would lost benefits of using document (with document D&D works also below the body if body is small).

I decided to check if events target document exist instead and cancel the event if it's not.

Changes in t/11823.

comment:6 Changed 4 years ago by Piotrek Koszuliński

Status: reviewreview_failed

I force pushed rebased branch with additional commit simplifying and clarifying the target validation.

However, R- because tabletools tests fail.

Last edited 4 years ago by Piotrek Koszuliński (previous) (diff)

comment:7 Changed 4 years ago by Piotr Jasiun

Status: review_failedreview

I rebased branch, run tests and tabletools tests were green on IE8, FF and Chrome, but tableresize tests were red. If fixed those tests and pushed both branches.

comment:8 Changed 4 years ago by Piotrek Koszuliński

Status: reviewreview_passed

Ups, my fault - of course I meant tableresize tests.

comment:9 Changed 4 years ago by Piotr Jasiun

Resolution: fixed
Status: review_passedclosed
Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy