Opened 11 years ago
Closed 11 years ago
#11597 closed Bug (fixed)
[IE11] Error thrown when trying to open preview using keyboard
| Reported by: | Piotrek Koszuliński | Owned by: | Marek Lewandowski |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.3.4 |
| Component: | General | Version: | 4.3.2 |
| Keywords: | Cc: |
Description (last modified by )
- open any sample with CKEditor (i.e. samples/replacebyclass.html)
- focus editor
- press alt+f10 to focus toolbar
- navigate to preview button
- press button using space
Related issue: #11408.
Our goal is to avoid exception. Popup blocker thing is another issue.
Change History (7)
comment:1 Changed 11 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 11 years ago by
| Owner: | set to Marek Lewandowski |
|---|---|
| Status: | confirmed → assigned |
comment:3 Changed 11 years ago by
| Status: | assigned → review |
|---|
comment:4 Changed 11 years ago by
| Description: | modified (diff) |
|---|
comment:5 Changed 11 years ago by
I tested it and everything works fine now (on Windows 7 and Windows 8.1).
comment:6 Changed 11 years ago by
| Status: | review → review_passed |
|---|
comment:7 Changed 11 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with git:7838eb1c98 (merged to master) at dev.
Note: See
TracTickets for help on using
tickets.
Commit which caused the issue: git:93803693adf82d3467f926a68325904b3a360f3b
With that commit we started to use window.location (for IE) to assign that tricky js code (instead of passing it to window.open). I created a trivial fix which prevents the exception from being thrown.
Pushed to t/11597 at dev.