Opened 15 years ago
Last modified 15 years ago
#5221 closed Bug
[IE7 3.2.x branch] Have to click the Table dialog's OK button twice when there is no content. — at Initial Version
Reported by: | Joe Kavanagh | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.3 |
Component: | General | Version: | SVN (CKEditor) - OLD |
Keywords: | IBM Confirmed Review+ | Cc: | Damian |
Description
- Open the Ajax sample. Notice that the elements path is empty.
- Click the Table toolbar button.
- Click the Table dialog's OK button.
Observe the javascript error: 'type' is null or not an object.
This does not occur on the nightly 3.2 demo as the content area gets focus before the dialog appears. "body p" appears in the elements path.
The error is occurring in core/dom/walker.js guardRTL function.
I believe the error is occurring because the node to replace is null. The wysiwyg plugin's onInsertElement function thinks it is in an empty p element and attempts to replace it with the table element. See the if statement following the below comment in the onInsertFunction function:
If we're in an empty block which indicate a new paragraph, simply replace it with the inserting block.(#3664)