Opened 11 years ago
Closed 11 years ago
#10868 closed Bug (fixed)
IE8: Page crashes when Inline Quotation is used.
Reported by: | Jakub Ś | Owned by: | Piotrek Koszuliński |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.2.2 |
Component: | General | Version: | 4.0 |
Keywords: | IE8 | Cc: |
Description
- Open replacebycode sample and clear page with New Page command or with Ctrl+A Del
- Go to styles dropdown and select Inline Quotation.
- If page hasn't crashed yet then please press Enter key. Page will crash and UIE will reload it.
I was able to reproduce this problem in IE8 only from CKEditor 4.0
Change History (9)
comment:1 Changed 11 years ago by
Status: | new → confirmed |
---|---|
Summary: | IE8: crashes when Inline Quotation is used. → IE8: Page crashes when Inline Quotation is used. |
comment:2 Changed 11 years ago by
comment:3 Changed 11 years ago by
good news - i can no longer reproduce this bug in current version - so it should be solved with 4.3 beta :)
update:
Unfortunately it still occurs. The reason why i haven't experienced it last time, was that ie8 decided that IE7 standards should be applied on sample page replacebycode.html.
If Document Mode is for IE7 then problem does not occurs
If it is for IE8 standards - the bug still exists
comment:4 Changed 11 years ago by
Milestone: | → CKEditor 4.2.2 |
---|
comment:5 Changed 11 years ago by
Owner: | set to Piotrek Koszuliński |
---|---|
Status: | confirmed → assigned |
comment:6 Changed 11 years ago by
First bad commit is git:c8b8caad59. It took mi half an hour to find this because IE is crashing very badly.
comment:7 Changed 11 years ago by
Status: | assigned → review |
---|
Pushed t/10868 with a solution which seems to work. IE8 is crashing when there's an img:hover
style, so I made it invalid for IE8, which does not understand :not()
selector.
comment:8 Changed 11 years ago by
Status: | review → review_passed |
---|
I curious to hear how you figured this out :D
comment:9 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with git:2c044fd on master.
I curious to hear how you figured this out :D
First I did a painful binary lookup in history and then when I found out that this were changes in contents.css what started the problem I tried to find which rules exactly and... it was img:hover
:D
cc