Opened 12 years ago
Closed 12 years ago
#6860 closed Bug (fixed)
[FF] placeholder plugin - JavaScript error
Reported by: | Wiktor Walc | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.1 |
Component: | UI : Dialogs | Version: | 3.5 |
Keywords: | Cc: |
Description (last modified by )
- Open _samples/placeholder.html
- Use the following source:
<p> Sample [[sdfsdf]]</p>
- Double click on the placeholder to open the dialog
- Result: the dialog didn't load the value properly and a JavaScript error occurred
element is null http://ckeditor.local/ckeditor/3.5/_source/plugins/placeholder/dialogs/placeholder.js Line 36
Attachments (2)
Change History (11)
comment:1 Changed 12 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 12 years ago by
Status: | confirmed → pending |
---|
comment:3 Changed 12 years ago by
Description: | modified (diff) |
---|---|
Milestone: | CKEditor 3.5.1 → CKEditor 3.5.2 |
Simplified test case. I still can reproduce it on Firefox 4.0b8.
comment:4 Changed 12 years ago by
Keywords: | FF4 removed |
---|---|
Milestone: | CKEditor 3.5.2 → CKEditor 3.5.1 |
Confirmed also in FF 3.6.13.
comment:5 Changed 12 years ago by
Component: | General → UI : Dialogs |
---|---|
Status: | pending → confirmed |
Changed 12 years ago by
Attachment: | 6860.patch added |
---|
comment:6 Changed 12 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
Summary: | FF4: placeholder plugin - JavaScript error → [FF] placeholder plugin - JavaScript error |
When placeholder being the last element of the block, double click will select the previous node instead of the element itself, it's safe to just not opening the dialog in such case.
comment:7 Changed 12 years ago by
Status: | review → review_passed |
---|
We should file a bug to mozilla. Attaching a reduced TC.
Changed 12 years ago by
Attachment: | reduced_tc.html added |
---|
comment:9 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Firefox bug reported.
WFM with 4.0b8pre.