(stable) Fix search results with tables, fatals in Phortune

Summary: Previously we collapsed all table search results, but the new UI doesn't need it. Remove unused methods and fix CSS.

Test Plan: Legalpad Signatures, Phortune Accounts.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

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

src/applications/phortune/controller/PhortuneAccountListController.php

@@ -39,7 +39,7 @@ final class PhortuneAccountListController extends PhortuneController {
         ->setHeader($account->getName())
         ->setHref($this->getApplicationURI($account->getID().'/'))
         ->setObject($account)
-        ->setIcon('fa-credit-card');
+        ->setImageIcon('fa-credit-card');
 
       $payment_list->addItem($item);
     }
@@ -71,7 +71,7 @@ final class PhortuneAccountListController extends PhortuneController {
         ->setHeader($merchant->getName())
         ->setHref($this->getApplicationURI('/merchant/'.$merchant->getID().'/'))
         ->setObject($merchant)
-        ->setIcon('fa-bank');
+        ->setImageIcon('fa-bank');
 
       $merchant_list->addItem($item);
     }

src/applications/phortune/controller/PhortuneProductListController.php

@@ -41,7 +41,7 @@ final class PhortuneProductListController extends PhabricatorController {
         ->setHeader($product->getProductName())
         ->setHref($view_uri)
         ->addAttribute($price->formatForDisplay())
-        ->setIcon('fa-gift');
+        ->setImageIcon('fa-gift');
 
       $product_list->addItem($item);
     }

src/applications/phortune/query/PhortuneMerchantSearchEngine.php

@@ -74,7 +74,7 @@ final class PhortuneMerchantSearchEngine
         ->setHeader($merchant->getName())
         ->setHref('/phortune/merchant/'.$merchant->getID().'/')
         ->setObject($merchant)
-        ->setIcon('fa-bank');
+        ->setImageIcon('fa-bank');
 
       $list->addItem($item);
     }

src/applications/search/controller/PhabricatorApplicationSearchController.php

@@ -270,9 +270,6 @@ final class PhabricatorApplicationSearchController
           if ($list->getContent()) {
             $box->appendChild($list->getContent());
           }
-          if ($list->getCollapsed()) {
-            $box->setCollapsed(true);
-          }
 
           $result_header = $list->getHeader();
           if ($result_header) {

src/applications/search/view/PhabricatorApplicationSearchResultView.php

@@ -12,7 +12,6 @@ final class PhabricatorApplicationSearchResultView extends Phobject {
   private $content = null;
   private $infoView = null;
   private $actions = array();
-  private $collapsed = null;
   private $noDataString;
   private $crumbs = array();
   private $header;
@@ -70,15 +69,6 @@ final class PhabricatorApplicationSearchResultView extends Phobject {
     return $this->actions;
   }
 
-  public function setCollapsed($collapsed) {
-    $this->collapsed = $collapsed;
-    return $this;
-  }
-
-  public function getCollapsed() {
-    return $this->collapsed;
-  }
-
   public function setNoDataString($nodata) {
     $this->noDataString = $nodata;
     return $this;

webroot/rsrc/css/application/search/application-search-view.css

@@ -16,6 +16,11 @@
   border-bottom: 1px solid {$thinblueborder};
 }
 
+.application-search-view .phui-object-box.phui-object-box-collapsed
+  .phui-header-shell {
+  padding: 20px 8px;
+}
+
 .application-search-results
   .phui-profile-header.phui-header-shell .phui-header-header {
   font-size: 20px;