mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-20 12:30:56 +01:00
Modernize Conduit app a bit
Summary: Remove some AphrontPanels, add some phts, fix some table layouts. Test Plan: Browse many Conduit pages, test a few calls. Reviewers: btrahan, epriestley Reviewed By: epriestley Subscribers: Korvin, epriestley Maniphest Tasks: T7427 Differential Revision: https://secure.phabricator.com/D11957
This commit is contained in:
parent
024a01abe8
commit
423180430e
2 changed files with 20 additions and 28 deletions
|
@ -532,7 +532,6 @@ final class PhabricatorConduitAPIController
|
|||
}
|
||||
|
||||
$param_table = new AphrontTableView($param_rows);
|
||||
$param_table->setDeviceReadyTable(true);
|
||||
$param_table->setColumnClasses(
|
||||
array(
|
||||
'header',
|
||||
|
@ -548,27 +547,20 @@ final class PhabricatorConduitAPIController
|
|||
}
|
||||
|
||||
$result_table = new AphrontTableView($result_rows);
|
||||
$result_table->setDeviceReadyTable(true);
|
||||
$result_table->setColumnClasses(
|
||||
array(
|
||||
'header',
|
||||
'wide',
|
||||
));
|
||||
|
||||
$param_panel = new AphrontPanelView();
|
||||
$param_panel->setHeader('Method Parameters');
|
||||
$param_panel = new PHUIObjectBoxView();
|
||||
$param_panel->setHeaderText(pht('Method Parameters'));
|
||||
$param_panel->appendChild($param_table);
|
||||
|
||||
$result_panel = new AphrontPanelView();
|
||||
$result_panel->setHeader('Method Result');
|
||||
$result_panel = new PHUIObjectBoxView();
|
||||
$result_panel->setHeaderText(pht('Method Result'));
|
||||
$result_panel->appendChild($result_table);
|
||||
|
||||
$param_head = id(new PHUIHeaderView())
|
||||
->setHeader(pht('Method Parameters'));
|
||||
|
||||
$result_head = id(new PHUIHeaderView())
|
||||
->setHeader(pht('Method Result'));
|
||||
|
||||
$method_uri = $this->getApplicationURI('method/'.$method.'/');
|
||||
|
||||
$crumbs = $this->buildApplicationCrumbs()
|
||||
|
@ -578,13 +570,11 @@ final class PhabricatorConduitAPIController
|
|||
return $this->buildApplicationPage(
|
||||
array(
|
||||
$crumbs,
|
||||
$param_head,
|
||||
$param_table,
|
||||
$result_head,
|
||||
$result_table,
|
||||
$param_panel,
|
||||
$result_panel,
|
||||
),
|
||||
array(
|
||||
'title' => 'Method Call Result',
|
||||
'title' => pht('Method Call Result'),
|
||||
));
|
||||
}
|
||||
|
||||
|
|
|
@ -35,6 +35,9 @@ final class PhabricatorConduitLogController
|
|||
}
|
||||
|
||||
$table = $this->renderCallTable($calls, $conns);
|
||||
$box = id(new PHUIObjectBoxView())
|
||||
->setHeaderText(pht('Call Logs'))
|
||||
->appendChild($table);
|
||||
|
||||
$crumbs = $this->buildApplicationCrumbs();
|
||||
$crumbs->addTextCrumb(pht('Call Logs'));
|
||||
|
@ -42,11 +45,11 @@ final class PhabricatorConduitLogController
|
|||
return $this->buildApplicationPage(
|
||||
array(
|
||||
$crumbs,
|
||||
$table,
|
||||
$box,
|
||||
$pager,
|
||||
),
|
||||
array(
|
||||
'title' => 'Conduit Logs',
|
||||
'title' => pht('Conduit Logs'),
|
||||
));
|
||||
}
|
||||
|
||||
|
@ -100,18 +103,17 @@ final class PhabricatorConduitLogController
|
|||
);
|
||||
}
|
||||
|
||||
$table = id(new AphrontTableView($rows))
|
||||
->setDeviceReadyTable(true);
|
||||
$table = id(new AphrontTableView($rows));
|
||||
|
||||
$table->setHeaders(
|
||||
array(
|
||||
'Connection',
|
||||
'User',
|
||||
'Method',
|
||||
'Status',
|
||||
'Error',
|
||||
'Duration',
|
||||
'Date',
|
||||
pht('Connection'),
|
||||
pht('User'),
|
||||
pht('Method'),
|
||||
pht('Status'),
|
||||
pht('Error'),
|
||||
pht('Duration'),
|
||||
pht('Date'),
|
||||
));
|
||||
$table->setColumnClasses(
|
||||
array(
|
||||
|
|
Loading…
Reference in a new issue