Replace "Phabricator" menu text with phabricator logo

Summary:
I ran a 99designs contest and this gear-eye thing was actually pretty okay. All
this stuff needs tweaks but at least it won't render with a big square on
windows anymore.

Test Plan:
Looked at menu, it seemed slightly more legitimate and designey than before?

Reviewed By: jungejason
Reviewers: tuomaspelkonen, jungejason, aran
CC: aran, epriestley, jungejason
Differential Revision: 363

src/view/page/standard/PhabricatorStandardPageView.php

@@ -272,30 +272,30 @@ class PhabricatorStandardPageView extends AphrontPageView {
     return
       ($console ? '<darkconsole />' : null).
       '<div class="phabricator-standard-page '.$admin_class.'">'.
-        '<div class="phabricator-standard-header">'.
+        '<table class="phabricator-standard-header">'.
-          '<div class="phabricator-login-details">'.
+          '<tr>'.
-            $login_stuff.
+            '<td class="phabricator-logo"><a href="/"> </a></td>'.
-          '</div>'.
+            '<td>'.
-          '<table class="phabricator-primary-navigation">'.
+              '<table class="phabricator-primary-navigation">'.
-            '<tr>'.
+                '<tr>'.
-              '<th class="phabricator-logo">'.
+                  '<th>'.
-                '<a href="/">'.
+                    phutil_render_tag(
-                  "Phabricat\xE2\x9A\x99r".
+                      'a',
-                '</a> '.
+                      array(
-              '</th>'.
+                        'href'  => $this->getBaseURI(),
-              '<th>'.
+                        'class' => 'phabricator-head-appname',
-                phutil_render_tag(
+                      ),
-                  'a',
+                      phutil_escape_html($this->getApplicationName())).
-                  array(
+                  '</th>'.
-                    'href'  => $this->getBaseURI(),
+                  $tabs.
-                    'class' => 'phabricator-head-appname',
+                '</tr>'.
-                  ),
+              '</table>'.
-                  phutil_escape_html($this->getApplicationName())).
+            '</td>'.
-              '</th>'.
+            '<td class="phabricator-login-details">'.
-              $tabs.
+              $login_stuff.
-            '</tr>'.
+            '</td>'.
-          '</table>'.
+          '</tr>'.
-        '</div>'.
+        '</table>'.
         $this->bodyContent.
         '<div style="clear: both;"></div>'.
       '</div>'.

webroot/rsrc/css/application/base/standard-page-view.css

@@ -17,6 +17,13 @@
   color: white;
   overflow: hidden;
   position: relative;
+  width: 100%;
+}
+
+.phabricator-standard-header td {
+  vertical-align: bottom;
+  padding: 0;
+  margin: 0;
 }
 
 .phabricator-admin-page-view .phabricator-standard-header {
@@ -24,8 +31,7 @@
 }
 
 .phabricator-primary-navigation {
-  padding-top: 12px;
+  padding-top: 24px;
-  padding-bottom: 8px;
   padding-left: 24px;
 }
 
@@ -38,14 +44,25 @@
   vertical-align: bottom;
   font-size: 13px;
   border-bottom: 6px solid transparent;
+  padding-top: 14px;
   padding-bottom: 4px;
 }
 
-.phabricator-primary-navigation th.phabricator-logo {
+.phabricator-logo {
-  font-size: 16px;
+  width: 220px;
-  padding-top: 8px;
+}
-  padding-left: 24px;
+
-  padding-right: 6px;
+.phabricator-logo a {
+  display: block;
+  width: 220px;
+  height: 40px;
+  background: url('/rsrc/image/phabricator_logo.png') no-repeat 0 0;
+  padding: 0;
+  margin: 0;
+}
+
+.phabricator-logo a:hover {
+  background-position-y: -40px;
 }
 
 .phabricator-primary-navigation td {
@@ -63,9 +80,9 @@
   text-transform: uppercase;
 }
 
-.phabricator-login-details {
+td.phabricator-login-details {
-  float: right;
+  text-align: right;
-  padding: 6px 24px 0;
+  padding: 0px 24px 6px;
   font-size: 12px;
 }