Opened 3 years ago

Last modified 3 years ago

#14518 pending Bug

On IE11, data-bound elements in dialog are not updated when clicking OK

Reported by: andysd Owned by:
Priority: Normal Milestone:
Component: General Version:
Keywords: IE Cc:

Description

Perhaps related to Issue 14517, but this one only happens on IE. On Safari and Firefox, there is no issue. The testcase uses a very simple example of knockout.js to bind an html input element to a value in a viewModel. It seems that because ckeditor tries to manage all aspects of focus and blur, something is lost (on IE only) when the OK button is clicked and the input field loses focus.

Steps to reproduce

  1. Download the attached plug-in to plugins folder, and add to config.js

config.extraPlugins = 'bugplugin';

  1. Run index.html, adjusting paths to require.js and knockout.js as needed
  2. Follow the simple testcase on-screen, basically change a value and click OK.

Expected result

Alert should show the value you just entered into Input1.

Actual result

On IE11, the current value is not shown . The old value is shown.

Other details (browser, OS, CKEditor version, installed plugins)

Attachments (4)

bugplugin.zip (5.1 KB) - added by andysd 3 years ago.
Plugin
index.html (766 bytes) - added by andysd 3 years ago.
index.html (run this, adjusting paths to scripts)
app.js (323 bytes) - added by andysd 3 years ago.
This is the require.js config referenced from index.html
bower.json (207 bytes) - added by andysd 3 years ago.
bower.json

Download all attachments as: .zip

Change History (7)

Changed 3 years ago by andysd

Attachment: bugplugin.zip added

Plugin

Changed 3 years ago by andysd

Attachment: index.html added

index.html (run this, adjusting paths to scripts)

Changed 3 years ago by andysd

Attachment: app.js added

This is the require.js config referenced from index.html

Changed 3 years ago by andysd

Attachment: bower.json added

bower.json

comment:1 Changed 3 years ago by chung

Keywords: IE added

Any update on this?

comment:2 Changed 3 years ago by Jakub Ś

Status: newpending
Version: 4.5.7

Is there any other way to reproduce this issue without all these extra libraries?

In order to accept the bug we need as simple test case as possible. At the moment problem can be connected to CKEditor but can also be connected to libraries you have mentioned. Is there any way to isolate this issue so that it is reproducible only with the editor?

comment:3 Changed 3 years ago by andysd

Thanks for looking at this -- I could not find a way to reproduce without using knockout, but I am confident the bug is not in knockout. Knockout is like Angular or Backbone -- it's just a way to "bind" UI to a model. I am using requirejs, and I could probably get rid of that dependency by including knockout directly, but as you know the bug isn't going to be in requirejs.

I think if you look at bug 14517 then it might lead to a solution with this bug. Like that bug, it seems like CKEDITOR is a little overzelaous in how it tries to control the tabbing and focus in plugins.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy