#4547 closed New Feature (fixed)
Load the skin CSS in parallel
| Reported by: | Frederico Caldeira Knabben | 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 (2)
Change History (9)
comment:1 Changed 15 years ago by
comment:2 Changed 15 years ago by
For reference, this idea comes from here:
http://ckeditor.com/blog/CKEditor_Loading_performance_details#comment-109
Changed 15 years ago by
| Attachment: | 4547.patch added |
|---|
comment:3 Changed 15 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
comment:4 Changed 15 years ago by
| 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.
comment:7 Changed 15 years ago by
| Keywords: | Review- removed |
|---|
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.