Ticket #4547 (closed New Feature: fixed)
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
Change History
comment:2 Changed 4 years ago by fredck
For reference, this idea comes from here:
http://ckeditor.com/blog/CKEditor_Loading_performance_details#comment-109
Changed 4 years ago by garry.yao
- Attachment 4547_firebug_netpanel_snapshots.png added
Result Comparison
comment:3 Changed 4 years ago by garry.yao
- Keywords Review? added
- Owner set to garry.yao
- Status changed from new to assigned
comment:4 Changed 4 years 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.
Note: See
TracTickets for help on using
tickets.
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.