launchpad-reviewers team mailing list archive

[Nigel Babu <nigelbabu@xxxxxxxxx>]

Nigel Babu has proposed merging lp:~nigelbabu/launchpad/ubuntu-font-787798 into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #787798 in Launchpad itself: "Launchpad should use Google web fonts for the Ubuntu font"
  https://bugs.launchpad.net/launchpad/+bug/787798

For more details, see:
https://code.launchpad.net/~nigelbabu/launchpad/ubuntu-font-787798/+merge/73793

= Description =
Use Google Web Fonts so Launchpad always displays in Ubuntu font

= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/app/templates/base-layout.pt
-- 
https://code.launchpad.net/~nigelbabu/launchpad/ubuntu-font-787798/+merge/73793
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~nigelbabu/launchpad/ubuntu-font-787798 into lp:launchpad.

=== modified file 'lib/lp/app/templates/base-layout.pt'
--- lib/lp/app/templates/base-layout.pt	2011-07-15 14:31:52 +0000
+++ lib/lp/app/templates/base-layout.pt	2011-09-02 12:12:29 +0000
@@ -29,6 +29,7 @@
     <title tal:content="view/fmt:pagetitle">Page Title</title>
     <link rel="shortcut icon" href="/@@/launchpad.png" />
 
+    <link href='http://fonts.googleapis.com/css?family=Ubuntu' rel='stylesheet' type='text/css' />
     <tal:atomfeeds define="feed_links view/feed_links | nothing">
       <link rel="alternate" type="application/atom+xml"
         tal:repeat="link feed_links"