launchpad-reviewers team mailing list archive

[Huw Wilkins <huwshimi@xxxxxxxxx>]

Huw Wilkins has proposed merging lp:~huwshimi/launchpad/get-started-text-435779 into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  #435779 "Get started" text anti-climactic
  https://bugs.launchpad.net/bugs/435779

For more details, see:
https://code.launchpad.net/~huwshimi/launchpad/get-started-text-435779/+merge/47958

Changed "Get Started" text to make it a little more engaging. Removed the "Read the user guide" bullet point and made it part of the copy. Also changed the sandbox link to point to http://quastaging.launchpad.net instead of http://staging.launchpad.net.
-- 
https://code.launchpad.net/~huwshimi/launchpad/get-started-text-435779/+merge/47958
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~huwshimi/launchpad/get-started-text-435779 into lp:launchpad.

=== modified file 'lib/lp/app/templates/root-index.pt'
--- lib/lp/app/templates/root-index.pt	2010-12-20 19:31:48 +0000
+++ lib/lp/app/templates/root-index.pt	2011-01-31 02:29:29 +0000
@@ -187,18 +187,18 @@
               <tal:logged_out condition="not:view/user" omit-tag="">
                 <a href="/+login">Creating an account</a> allows you to start
                 working within Launchpad.<br />
-              </tal:logged_out>You can test Launchpad's functionality
-              in our sandbox environment.
-              (<a href="/+help/home-page-staging-help.html" target="help">What's this?</a>)<br />
+              </tal:logged_out>
+              <p>
+                Learn more about Launchpad in the 
+                <a href="https://help.launchpad.net/";>user guide</a>
+                or try it for yourself in our 
+                <a href="https://qastaging.launchpad.net/";>sandbox environment</a>.
+              </p>
               <tal:logged_in condition="view/user" omit-tag=""
                 tal:content="cache:public">
                 If you're ready, you can:
                 <ul tal:define="apphomes view/apphomes">
                   <li>
-                    <a class="sprite info"
-                      href="https://help.launchpad.net/";>Read the user guide</a>
-                  </li>
-                  <li>
                     <a class="sprite add"
                       href="/projects/+new">Register a project</a>
                   </li>