Opened 15 years ago
Closed 14 years ago
#4514 closed Bug (fixed)
Editor edit area disables on back button
| Reported by: | Ryan Bohn | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | General | Version: | 3.0 |
| Keywords: | Firefox Confirmed Review+ | Cc: |
Description
Environment - Firefox 3.5.3 on Mac OSX Leopard
This is reproducible in the samples, specifically replacebyclass.html
- open up the sample html in FireFox
- now go to another site like google
- then, click back button so your back on the sample
- you are now unable to click in the edit area until you toggle view source
Attachments (2)
Change History (10)
comment:1 Changed 14 years ago by
| Keywords: | Firefox added |
|---|---|
| Resolution: | → wontfix |
| Status: | new → closed |
comment:2 Changed 14 years ago by
| Keywords: | Confirmed added |
|---|---|
| Milestone: | → CKEditor 3.1 |
| Resolution: | wontfix |
| Status: | closed → reopened |
comment:3 Changed 14 years ago by
| Keywords: | Review? added |
|---|
Changed 14 years ago by
| Attachment: | 4514.patch added |
|---|
comment:4 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | reopened → new |
comment:5 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
The patch is ok. I would just investigate other ways to apply onpageshow:
- In the proposed way, onpageshow could be in conflict with other code using it in the page. We must check if it's possible to set it as a normal event.
- We could try adding the onpageshow attribute in the content area instead. We may also try setting it directly to the <body> tag when creating the string we load in the editing area.
If none of the above are good for it, the provided patch would be ok.
Changed 14 years ago by
| Attachment: | 4514_2.patch added |
|---|
comment:6 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|---|
| Status: | new → assigned |
It's much more safe to preserve the original attribute now.
We could try adding the onpageshow attribute in the content area instead.
It's not working since the whole editing frame is cached too.
comment:7 Changed 14 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
- You have reported a Mozilla bug for this issue, but it's not clear it's related to the same thing. Please be sure we have a bug for it there, making a reference to it here.
- Be sure to close #3907 when closing this one.
comment:8 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
A more accurate bug report is delivered to Mozilla.
https://bugzilla.mozilla.org/show_bug.cgi?id=529321
Fixed with [4492].
Note: See
TracTickets for help on using
tickets.
Known limitation of contentEditable in Firefox.