Opened 14 years ago
Closed 14 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 14 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 14 years ago by
Status: | confirmed → pending |
---|
comment:3 Changed 14 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 14 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 14 years ago by
Component: | General → UI : Dialogs |
---|---|
Status: | pending → confirmed |
Changed 14 years ago by
Attachment: | 6860.patch added |
---|
comment:6 Changed 14 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 14 years ago by
Status: | review → review_passed |
---|
We should file a bug to mozilla. Attaching a reduced TC.
Changed 14 years ago by
Attachment: | reduced_tc.html added |
---|
comment:9 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Firefox bug reported.
WFM with 4.0b8pre.