Ticket #4547 (closed New Feature: fixed)

Opened 11 months ago

Last modified 10 months ago

Load the skin CSS in parallel

Reported by: fredck Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: 3.0
Keywords: Confirmed Cc:

Description

We're currently loading the skin CSS files only after the plugins files get loaded. This is not really necessary, and these files could be loaded in parallel, while the rest of the editor scripts get loaded.

Attachments

4547.patch Download (0.9 KB) - added by garry.yao 11 months ago.
4547_firebug_netpanel_snapshots.png Download (39.4 KB) - added by garry.yao 11 months ago.
Result Comparison

Change History

Changed 11 months ago by fredck

The solution for it could be quite simple. It's enough to (unblocked) load the skin files as the first thing right after loading the configuration file.

Changed 11 months ago by fredck

Changed 11 months ago by garry.yao

Changed 11 months ago by garry.yao

Result Comparison

Changed 11 months ago by garry.yao

  • keywords Review? added
  • owner set to garry.yao
  • status changed from new to assigned

Changed 10 months ago by fredck

  • keywords Review- added; Review? removed

The callback function at line 256 should not be removed. It'll not block for CSS files. It blocks only if the skin.js file needs to be downloaded.

Inside the above callback, we must call loadLang now, removing the call for it at line 130.

Changed 10 months ago by garry.yao

Depends on #4463.

Changed 10 months ago by garry.yao

  • status changed from assigned to closed
  • resolution set to fixed

Fixed by #4463.

Changed 10 months ago by garry.yao

  • keywords Review- removed
Note: See TracTickets for help on using tickets.