#13447 closed Bug (fixed)
Error while D&D widget by drag handler.
| Reported by: | Artur Delura | Owned by: | Artur Delura |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.5.0 |
| Component: | UI : Widgets | Version: | 4.5.0 Beta |
| Keywords: | Cc: |
Description
- Open: http://tests.ckeditor.dev:10450/tests/plugins/widget/manual/inline
- Try D&D first inline widget (Lena).
There is an error in console: dragRange is not defined: https://github.com/ckeditor/ckeditor-dev/blob/major/plugins/clipboard/plugin.js#L1301-L1302
Change History (11)
comment:1 Changed 9 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 9 years ago by
| Owner: | set to Artur Delura |
|---|---|
| Status: | confirmed → assigned |
comment:3 Changed 9 years ago by
| Status: | assigned → review |
|---|
comment:4 Changed 9 years ago by
I think it might be a good idea to add also checking whether dragRange is defined as well.
comment:6 Changed 9 years ago by
| Status: | review → review_passed |
|---|
comment:7 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
comment:8 Changed 9 years ago by
| Component: | General → UI : Widgets |
|---|---|
| Version: | → 4.5.0 Beta |
comment:11 Changed 9 years ago by
This patch introduces a bug. If dragRange is null then those two properties should be reset. Otherwise old values will be used upon drop what is clearly incorrect. Reported in #13451.
Note: See
TracTickets for help on using
tickets.
Fix related to #13011. Changes in branch:t/13447.