Opened 10 years ago

Closed 10 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 10 years ago by Piotr Jasiun

Description: modified (diff)

comment:2 Changed 10 years ago by Piotr Jasiun

Milestone: CKEditor 4.4.1

comment:3 Changed 10 years ago by Marek Lewandowski

Status: newconfirmed

comment:4 Changed 10 years ago by Piotr Jasiun

Owner: set to Piotr Jasiun
Status: confirmedassigned

comment:5 Changed 10 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 10 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 tests fail.

Version 0, edited 10 years ago by Piotrek Koszuliński (next)

comment:7 Changed 10 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 10 years ago by Piotrek Koszuliński

Status: reviewreview_passed

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

comment:9 Changed 10 years ago by Piotr Jasiun

Resolution: fixed
Status: review_passedclosed
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy