Opened 6 years ago

Closed 6 years ago

#6502 closed Task (fixed)

Remove IE6 image preloading

Reported by: garry.yao Owned by: tobiasz.cudnik
Priority: Normal Milestone: CKEditor 3.5
Component: General Version:
Keywords: IE Cc:

Description

Discovered in #6187, in case we can confirm that the following hack is enough to eliminate the IE6 background image duplicated requests bug, we should consider remove the preloader from core.

Attachments (3)

6502.patch (2.7 KB) - added by tobiasz.cudnik 6 years ago.
6502_2.patch (6.0 KB) - added by tobiasz.cudnik 6 years ago.
6502_3.patch (6.6 KB) - added by tobiasz.cudnik 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by garry.yao

  • Keywords IE added
  • Status changed from new to confirmed

comment:2 Changed 6 years ago by tobiasz.cudnik

  • Owner set to tobiasz.cudnik
  • Status changed from confirmed to assigned

Changed 6 years ago by tobiasz.cudnik

comment:3 Changed 6 years ago by tobiasz.cudnik

  • Status changed from assigned to review

After checking IE6 using the released version, with both Fiddler and httpd log i cannot see any difference after the fix from #6187.

Path the removes preloader code completely, as it would be unnecessary bits and we can always go back to it by reverting the patch in the future.

comment:4 Changed 6 years ago by garry.yao

  • Status changed from review to review_failed

Can we include also imagecacher.js along with the "preload" definition in skin file?

Changed 6 years ago by tobiasz.cudnik

comment:5 Changed 6 years ago by tobiasz.cudnik

  • Status changed from review_failed to review

We should also remove imagecacher.js and it's references in packager and loader files, which is what the second patch does.

comment:6 Changed 6 years ago by garry.yao

  • Status changed from review to review_failed

The "preload" definition in skin file is useless at the moment.

Changed 6 years ago by tobiasz.cudnik

comment:7 Changed 6 years ago by tobiasz.cudnik

  • Status changed from review_failed to review

Right, i've missed that.

comment:8 Changed 6 years ago by garry.yao

  • Status changed from review to review_passed

comment:9 Changed 6 years ago by tobiasz.cudnik

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [6100].

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy