Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mouse actions on CKEditor scrollbar throws 'p.getAscendant is not a function' #515

Closed
kdambrauskas opened this issue Jun 15, 2017 · 4 comments
Assignees
Labels
browser:chrome The issue can only be reproduced in the Chrome browser. good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. plugin:tableselection The plugin which probably causes the issue. regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug.
Milestone

Comments

@kdambrauskas
Copy link

kdambrauskas commented Jun 15, 2017

Bug

Mouse events on CKEditor scrollbar results in error 'p.getAscendant is not a function'

Steps to reproduce

  1. Make editor scrollable
  2. Mouse over scrollbar
  3. Profit.

Expected result

No errors

Actual result

Errors

Other details

Clicking the scrollbar also causes an exception.

  • Version Chrome 59.0.3071.86 (Official Build) (64-bit)
  • OS: Win 10
  • CKEditor version: 4.7.0
  • Installed CKEditor plugins: Official site http://ckeditor.com/
@jonira
Copy link

jonira commented Jun 15, 2017

We have experienced same issue. Can be reproduced in http://ckeditor.com/ demos. Using chrome Version 59.0.3071.86 (Official Build) (64-bit) OS: OS X El Capitan

@mlewand mlewand added regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug. plugin:tableselection The plugin which probably causes the issue. labels Jun 16, 2017
@mlewand
Copy link
Contributor

mlewand commented Jun 16, 2017

I can confirm, it's being thrown by tableselection plugin in https://github.com/ckeditor/ckeditor-dev/blob/621199c/plugins/tableselection/plugin.js#L265

Here's the stack from dev version:

plugin.js:265 Uncaught TypeError: target.getAscendant is not a function
    at CKEDITOR.dom.document.fakeSelectionMouseHandler (plugin.js:265)
    at CKEDITOR.dom.document.listenerFirer (event.js:144)
    at CKEDITOR.dom.document.CKEDITOR.event.fire (event.js:290)
    at HTMLDocument.<anonymous> (domobject.js:46)

@mlewand mlewand added browser:chrome The issue can only be reproduced in the Chrome browser. good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. labels Jun 16, 2017
@mlewand mlewand changed the title mouse events on ckeditor scrollbar 'p.getAscendant is not a function' Mouse over on CKEditor scrollbar throws 'p.getAscendant is not a function' Jun 16, 2017
@mlewand mlewand added this to the Backlog milestone Jun 16, 2017
@beatadelura beatadelura self-assigned this Jun 19, 2017
@mlewand mlewand changed the title Mouse over on CKEditor scrollbar throws 'p.getAscendant is not a function' Mouse actions on CKEditor scrollbar throws 'p.getAscendant is not a function' Jun 20, 2017
@mlewand mlewand modified the milestones: Backlog, 4.7.1 Jun 20, 2017
@ror-expert
Copy link

I am also getting same issue when over cursor on scrollbar. see attachment below.
screenshot from 2017-08-04 16-05-54
screenshot from 2017-08-04 16-17-17

@mlewand
Copy link
Contributor

mlewand commented Aug 14, 2017

@rajeev-khangar your first screen is a different issue #527 that is fixed in 4.7.2 which we're about to release.

The second screen was fixed in 4.7.1, please update your editor and you will no longer experience the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:chrome The issue can only be reproduced in the Chrome browser. good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. plugin:tableselection The plugin which probably causes the issue. regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

5 participants