#9823 closed bug (fixed)
IE 8 crashes when using background-image on BODY in css
| Reported by: | Owned by: | Timmy Willison | |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.6.3 |
| Component: | support | Version: | 1.6.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
IE 8 crashes when using background-image on the BODY in my css file. Using version 1.6.2. When using version 1.6.1 the browser doesn't crash.
Change History (11)
comment:1 Changed 12 years ago by
| Component: | unfiled → support |
|---|---|
| Owner: | set to cdgrit@… |
| Priority: | undecided → low |
| Status: | new → pending |
comment:2 Changed 12 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
comment:3 Changed 12 years ago by
This is a regression; lines 1281-1282 in 1.6.1:
Set background to avoid IE crashes when removing (#9028) background: "none"
These lines were removed from 1.6.2, causing this crash. Reintroducing them after line 1296 in 1.6.2 solves the crash!
comment:4 Changed 12 years ago by
Lines 1281-1282 properly formatted (my bad):
// Set background to avoid IE crashes when removing (#9028) background: "none"
comment:5 Changed 12 years ago by
| Milestone: | None → 1.7 |
|---|---|
| Priority: | low → blocker |
| Resolution: | invalid |
| Status: | closed → reopened |
Good catch, we'll tackle this.
comment:6 Changed 12 years ago by
| Milestone: | 1.7 → 1.6.3 |
|---|
comment:7 Changed 12 years ago by
| Status: | reopened → pending |
|---|
I have not been able to reproduce this and we have a test for this in the test suite. Is there a test case that causes the crash? Setting background: 'none' should have no effect on any existing body elements since we create a div if a body exists.
comment:8 Changed 12 years ago by
| Owner: | changed from cdgrit@… to Timmy Willison |
|---|---|
| Status: | pending → assigned |
From JorisDebonnet:
Go to http://fiddle.jshell.net/ZKPZq/6/ with IE8. Right click in the result screen, and make it refresh. Boom, crash.
comment:9 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Add back the background: none setting to avoid any possible crashes in IE from the support testElement. Fixes #9823.
Changeset: 5c4a9cc001fcd803efa65ff95571c72cbdafbe69
Thanks for taking the time to contribute to the jQuery project! Please provide a reduced test case on http://jsfiddle.net that reproduces the issue experienced to help us assess your ticket.
Additionally, test against the jQuery (edge) version to ensure the issue still exists.