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

Delete columns work incorrectly with native selection on Chrome #648

Closed
msamsel opened this issue Jul 14, 2017 · 1 comment
Closed

Delete columns work incorrectly with native selection on Chrome #648

msamsel opened this issue Jul 14, 2017 · 1 comment
Labels
browser:chrome The issue can only be reproduced in the Chrome browser. browser:safari The issue can only be reproduced in the Safari browser. status:confirmed An issue confirmed by the development team. type:bug A bug.

Comments

@msamsel
Copy link
Contributor

msamsel commented Jul 14, 2017

Are you reporting a feature or a bug?

BUG

Check if the issue is already reported

Put all reference links here…

Provide detailed reproduction steps (if any)

Example: https://codepen.io/msamsel/pen/bRZrNV

  1. Open editor without tableselection plugin in Chrome.
  2. Fill some data in table.
  3. Put caret in cell which is not in last column. Caret have to be after last character. In example it might after last D character.
  4. Open context menu.
  5. Choose Column -> Delete Column

Expected result

Only one column will be delete. Column where caret is put

Actual result

2 columns are removed.

Note:

It seems that Chrome wrongly keep selection range, when collapsed selection is put after last character in table cell. Start node is properly set in current cell, but end node is set up to cell from next column.

Other details

  • Browser: Chrome & Safari
  • OS: MacOS
  • CKEditor version: 4.7.1
  • Installed CKEditor plugins: full-all without tableselection
@msamsel msamsel added browser:chrome The issue can only be reproduced in the Chrome browser. browser:safari The issue can only be reproduced in the Safari browser. status:confirmed An issue confirmed by the development team. type:bug A bug. labels Jul 14, 2017
@Comandeer
Copy link
Member

This issue was fixed as a part of #577.

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. browser:safari The issue can only be reproduced in the Safari browser. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

2 participants