The code and images needs some serious optimization
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Online Tour - OBSOLETE |
Fix Released
|
Medium
|
Barry McGee |
Bug Description
Hello guys,
I'm deploying the 12.10 online tour on our LoCo Team's server, and I see the code really needs some optimization for performance and SEO.
First of all, it uses a LOT of CSS and JavaScript files. Can't we merge all of them into one? Why make the browser make so many HTTP requests, when we could combine the CSS and the JS files and make one of each?
It might loads faster on great speeds, but at Brazil for example, where the average user only have a 1mbps connection that barely works, it makes a LOT of difference.
We also need to minify the JavaScript and the CSS files, so they get smaller and loads faster. And to finish this section of the bug, right now the files are loaded like this: ../css/somecss.css.
All good and stuffs but, we need the URL in there, not the relative patch. That allow us to load everything from a CDN if we want, something that can also increase the performance. On our LoCo Team's server for example, I had to fix it one by one and that was a real pain in the ass.
Since we don't know what is the user's URL, there could be a script that ask what is your site's URL and replace a variable with it. Sadly I'm not a coder, so I do not know how to make it, but it would sure helps a lot. Canonical also should use a CDN on the English tour, for better performance.
More information about the subject:
http://
...
Another thing that need to be changed: the images AREN'T compressed. By compressing all the images using Kraken.io, I managed to get the image folder down from 7,4 MB to 6,7 MB. Might not looks like a lot, but in the web, every byte saved counts.
I attached the compressed images and CSS/JS files, hope you guys can use it. Sorry for my bad English and thanks in advance.
Related branches
- Anthony Dillon: Approve
-
Diff: 4295 lines (+1666/-1487)8 files modified.bzrignore (+1/-0)
Gulpfile.js (+72/-0)
README (+32/-1)
css/style.min.css (+1/-0)
en/index.html (+16/-1486)
en/src/index.html (+1519/-0)
js/script.min.js (+4/-0)
package.json (+21/-0)
Changed in ubuntu-online-tour: | |
assignee: | nobody → Anthony Dillon (ya-bo-ng) |
Changed in ubuntu-online-tour: | |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in ubuntu-online-tour: | |
assignee: | Anthony Dillon (ya-bo-ng) → Barry McGee (barry-mcgee) |
status: | Triaged → In Progress |
Changed in ubuntu-online-tour: | |
status: | In Progress → Fix Committed |
Changed in ubuntu-online-tour: | |
status: | Fix Committed → Fix Released |
Thanks Julian for taking the time to write the bug report and providing the patches.
We use revision control to review and track changes to all of our projects, rather than patches. Would you mind submitting your changes as a branch instead of files?
Here's how you can do it:
1. Check out a copy of the branch, run 'bzr branch lp:ubuntu-online-tour'
2. Apply your changes on your local branch
3. Commit those changes
4. Push the changes to a new branch in Launchpad. E.g. run 'bzr push lp:~julianfernandes/ubuntu-online-tour/bug-1100330'
5. Submit a merge proposal. E.g. go to the Launchpad page of your branch and click on the "Submit merge proposal" link
Let us know if the steps were clear or if you need some help in any of them.
Thanks!