#12006 closed Bug (fixed)
[Nested widgets] Drag and drop of nested block widgets
Reported by: | Olek Nowodziński | Owned by: | Olek Nowodziński |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.5.0 Beta |
Component: | General | Version: | 4.3 Beta |
Keywords: | Cc: |
Description (last modified by )
Part of #10931.
At the moment there's no way to drag and drop any block widget within nested editable because "drop lines " are not shown in nested editables at all.
Change History (11)
comment:1 Changed 11 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 11 years ago by
Description: | modified (diff) |
---|---|
Summary: | [Nested widgets] Drag and drop of nested widgets → [Nested widgets] Drag and drop of nested block widgets |
comment:4 Changed 11 years ago by
Owner: | set to Olek Nowodziński |
---|---|
Status: | confirmed → assigned |
comment:5 Changed 11 years ago by
Status: | assigned → review |
---|
Pushed changes and tests to branch:t/12006:
- Drag and drop of block widgets is enabled for nested editables.
- Drag and drop considers filters (
allowedContent
). If the widget is disallowed in some nested editable, there's no chance it's dropped there (no drop line is displayed). - Prevented pathological drop into widget's own nested editable.
comment:6 Changed 11 years ago by
Milestone: | → CKEditor 4.4.2 |
---|
comment:7 Changed 11 years ago by
Milestone: | CKEditor 4.4.2 → CKEditor 4.5.0 |
---|
This is definitely a ticket for major release, because it changes the behaviour of dnd. Prior to 4.5 nested widgets are not enabled (dnd, insertion etc). Enabling dnd in 4.4.2 would suddenly unlock the ability to nest widgets and that's not safe for a minor release.
comment:9 Changed 11 years ago by
Status: | review → review_passed |
---|
R+. Doesn't work on IE8 but we don't plan to have nested widgets support in IE8 yet, so it's ok.
comment:10 Changed 11 years ago by
Remember to rearrange commits in the branch so tests go after dev.
comment:11 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
git:541f71b landed in major
I made it a block widgets only ticket. Inline widgets dnd will be handled in different ticket.