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

Nested editable can't be focused under certain circumstances #2655

Closed
mflorea opened this issue Dec 6, 2018 · 3 comments
Closed

Nested editable can't be focused under certain circumstances #2655

mflorea opened this issue Dec 6, 2018 · 3 comments
Assignees
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. plugin:copyformatting The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. support:2 An issue reported by a commercially licensed client. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Milestone

Comments

@mflorea
Copy link
Contributor

mflorea commented Dec 6, 2018

Type of report

Bug

Provide detailed reproduction steps (if any)

  1. Open https://jsfiddle.net/w08o1reL/11/ in Chrome.
  2. Click over "foo". The nested editable is focused as a result.
  3. Click just above the widget (above the black border of the widget). The nested editable looses the focus as a result.
  4. Click again over "foo".

Expected result

The nested editable is focused.

Actual result

The nested editable gets the focus for a fraction of a second but then it looses it.

Other details

  • Browser: Chrome
  • OS: Any
  • CKEditor version: 4.11.1
  • Installed CKEditor plugins: wysiwygarea, widget, copyformatting

Note that:

  • the issue doesn't reproduce if I remove the copyformatting plugin. This makes me think that this issue is related to Widget nested editable triggers blur event on focus in Firefox #2470
  • the issue doesn't reproduce if I replace the padding on the body element with margin. The default CKEditor content CSS is using margin so you may not be affected by this issue if you don't use fullPage or if you use fullPage and you don't have padding on the body tag.
  • the issue doesn't reproduce on Firefox
@mflorea
Copy link
Contributor Author

mflorea commented Dec 6, 2018

You can also check this video https://drive.google.com/file/d/1Voj7yMyy6oEtbFyQNlB_9Sj94vF_joBI/view .

@Comandeer Comandeer self-assigned this Dec 7, 2018
@Comandeer
Copy link
Member

Thanks for the report, I can confirm the issue in Chrome 71.

@Comandeer Comandeer added type:bug A bug. status:confirmed An issue confirmed by the development team. browser:chrome The issue can only be reproduced in the Chrome browser. plugin:copyformatting The plugin which probably causes the issue. labels Dec 7, 2018
@Comandeer Comandeer removed their assignment Dec 7, 2018
@lslowikowska lslowikowska added the support:2 An issue reported by a commercially licensed client. label Feb 14, 2019
@mlewand mlewand added this to the Backlog milestone Feb 14, 2019
@mlewand mlewand added the target:minor Any docs related issue that can be merged into a master or major branch. label Feb 14, 2019
@Comandeer Comandeer self-assigned this Feb 14, 2019
@Comandeer Comandeer added the browser:safari The issue can only be reproduced in the Safari browser. label Feb 14, 2019
@Comandeer
Copy link
Member

The issue was fixed by #2792. The fix will be released in 4.11.3.

@Comandeer Comandeer modified the milestones: Backlog, 4.11.3 Feb 19, 2019
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. plugin:copyformatting The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. support:2 An issue reported by a commercially licensed client. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

4 participants