Code review comment for lp:~kvilhaugsvik/firefox-extensions/all-in-one-sidebar.ubuntu

thanks. Also thanks for resurrecting the changes that were not committed here.

In future, please use topic branch names and base each upload on the ~ubuntu-dev one. By using distinct topic names (and not the same name as the ~ubuntu-dev brancH) you can keep them around "marked" as merged and don't need to trash them - which is a good thing in general in case there are merge errors. example for branch names would be the package version you prepared: e.g. all-in-one-sidebar.ubuntu.1.4.5-0ubuntu1 could be a good name Thanks.