Don't link object monograms in object graphs

Summary: Ref T4788.

Test Plan: {F1708372}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T4788

Differential Revision: https://secure.phabricator.com/D16221

Author: epriestley

resources/celerity/map.php

@@ -7,7 +7,7 @@
  */
 return array(
   'names' => array(
-    'core.pkg.css' => 'd04e6f67',
+    'core.pkg.css' => '55d9bb83',
     'core.pkg.js' => 'f2139810',
     'darkconsole.pkg.js' => 'e7393ebb',
     'differential.pkg.css' => '3e81ae60',
@@ -25,7 +25,7 @@
     'rsrc/css/aphront/notification.css' => '3f6c89c9',
     'rsrc/css/aphront/panel-view.css' => '8427b78d',
     'rsrc/css/aphront/phabricator-nav-view.css' => 'ac79a758',
-    'rsrc/css/aphront/table-view.css' => 'aeb66852',
+    'rsrc/css/aphront/table-view.css' => '8df59783',
     'rsrc/css/aphront/tokenizer.css' => '056da01b',
     'rsrc/css/aphront/tooltip.css' => '1a07aea8',
     'rsrc/css/aphront/typeahead-browse.css' => '8904346a',
@@ -536,7 +536,7 @@
     'aphront-list-filter-view-css' => '5d6f0526',
     'aphront-multi-column-view-css' => 'fd18389d',
     'aphront-panel-view-css' => '8427b78d',
-    'aphront-table-view-css' => 'aeb66852',
+    'aphront-table-view-css' => '8df59783',
     'aphront-tokenizer-control-css' => '056da01b',
     'aphront-tooltip-css' => '1a07aea8',
     'aphront-typeahead-control-css' => 'd4f16145',

src/infrastructure/graph/DifferentialRevisionGraph.php

@@ -41,11 +41,13 @@ protected function newTableRow($phid, $object, $trace) {
         array(
           'href' => $object->getURI(),
         ),
-        array(
-          $object->getMonogram(),
-          ' ',
-          $object->getTitle(),
-        ));
+        $object->getTitle());
+
+      $link = array(
+        $object->getMonogram(),
+        ' ',
+        $link,
+      );
     } else {
       $status = null;
       $author = null;
@@ -74,7 +76,7 @@ protected function newTable(AphrontTableView $table) {
           'threads',
           'graph-status',
           null,
-          'wide object-link',
+          'wide pri object-link',
         ));
   }
 

src/infrastructure/graph/ManiphestTaskGraph.php

@@ -54,11 +54,13 @@ protected function newTableRow($phid, $object, $trace) {
         array(
           'href' => $object->getURI(),
         ),
-        array(
-          $object->getMonogram(),
-          ' ',
-          $object->getTitle(),
-        ));
+        $object->getTitle());
+
+      $link = array(
+        $object->getMonogram(),
+        ' ',
+        $link,
+      );
     } else {
       $status = null;
       $assigned = null;
@@ -87,7 +89,7 @@ protected function newTable(AphrontTableView $table) {
           'threads',
           'graph-status',
           null,
-          'wide object-link',
+          'wide pri object-link',
         ));
   }
 

webroot/rsrc/css/aphront/table-view.css

@@ -228,9 +228,14 @@ span.single-display-line-content {
   position: static;
 }
 
+.aphront-table-view tr.closed td.object-link,
+.aphront-table-view tr.alt-closed td.object-link {
+  text-decoration: line-through;
+  color: rgba({$alphablack}, 0.5);
+}
+
 .aphront-table-view tr.closed td.object-link a,
 .aphront-table-view tr.alt-closed td.object-link a {
-  text-decoration: line-through;
   color: rgba({$alphablack}, 0.5);
 }