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.

Sign up for GitHub

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

Jump to bottom

Click on scrollbar throws error node.getAscendant is not a function #528

Closed
beatadelura opened this issue Jun 19, 2017 · 3 comments
Closed

Click on scrollbar throws error node.getAscendant is not a function #528

beatadelura opened this issue Jun 19, 2017 · 3 comments
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. resolution:duplicate A duplicate of an already reported issue. status:confirmed An issue confirmed by the development team. type:bug A bug.

Comments

@beatadelura
Copy link
Contributor

Are you reporting a feature or a bug?

Bug

Provide detailed reproduction steps (if any)

  1. Make editor scrollable
  2. Click on scrollbar

Expected result

No error in the console.

Actual result

An error in the console:

Uncaught TypeError: node.getAscendant is not a function

Other details

An error comes from here: https://github.com/ckeditor/ckeditor-dev/blob/master/plugins/tableselection/plugin.js#L281

There is similar issue #515

  • Browser: Chrome 59
  • OS: MacOS
  • CKEditor version: 4.7.0
@beatadelura beatadelura added browser:chrome The issue can only be reproduced in the Chrome browser. type:bug A bug. labels Jun 19, 2017
@mlewand
Copy link
Contributor

mlewand commented Jun 20, 2017

I believe it should be fixed by #515.

@beatadelura beatadelura mentioned this issue Jun 20, 2017
Merged
2 tasks
@beatadelura
Copy link
Contributor Author

@mlewand You are right. #515 fixing this issue. 🎉

@mlewand mlewand added good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. status:confirmed An issue confirmed by the development team. labels Jun 20, 2017
@mlewand
Copy link
Contributor

mlewand commented Jun 20, 2017

I'll close it as a duplicate of #515, and make the former one more generic.

@mlewand mlewand closed this as completed Jun 20, 2017
@mlewand mlewand added the resolution:duplicate A duplicate of an already reported issue. label Jun 20, 2017
@victor-kirillov victor-kirillov mentioned this issue Jul 20, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
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. resolution:duplicate A duplicate of an already reported issue. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mlewand @beatadelura