The code and images needs some serious optimization

Bug #1100330 reported by Julian Fernandes
6
This bug affects 1 person
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://developer.yahoo.com/performance/rules.html

...

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

Revision history for this message
Julian Fernandes (julianfernandes) wrote :
Revision history for this message
Julian Fernandes (julianfernandes) wrote :
Revision history for this message
Julian Fernandes (julianfernandes) wrote :
David Planella (dpm)
Changed in ubuntu-online-tour:
assignee: nobody → Anthony Dillon (ya-bo-ng)
Revision history for this message
David Planella (dpm) wrote :

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!

Revision history for this message
Julian Fernandes (julianfernandes) wrote :

Thanks for the reply David :)

I also found some more ways we can improve the tour, like adding Facebook open graph, the Google Plus publisher tag and the Twitter card feature.

Will do that as soon as we launch our new LoCo Team's design and server, in about an hour.

David Planella (dpm)
Changed in ubuntu-online-tour:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Julian Fernandes (julianfernandes) wrote :

My connection here is REALLY bad, so I will upload everything once I get home, on monday :)

Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

Julian, did you manage to add the social tags with the combined and minified css and js? If you have please do provide a merge proposal. If not we will try and do this work as part of the next update (LTS).

Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

The tour will be redeveloped for 16.04 release. I will make sure the images provided or new ones will be optimised.

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
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.