Update Help for newPage
Summary: Swaps over to `newPage` and `newDialog` Test Plan: Unsure how to actually pull these up? Reviewers: epriestley Reviewed By: epriestley Subscribers: Korvin Differential Revision: https://secure.phabricator.com/D15617
This commit is contained in:
@@ -38,14 +38,10 @@ final class PhabricatorHelpDocumentationController
|
||||
$crumbs = $this->buildApplicationCrumbs();
|
||||
$crumbs->addTextCrumb($title);
|
||||
|
||||
return $this->buildApplicationPage(
|
||||
array(
|
||||
$crumbs,
|
||||
$list,
|
||||
),
|
||||
array(
|
||||
'title' => $title,
|
||||
));
|
||||
return $this->newPage()
|
||||
->setTitle($title)
|
||||
->setCrumbs($crumbs)
|
||||
->appendChild($list);
|
||||
}
|
||||
|
||||
|
||||
|
||||
@@ -10,8 +10,7 @@ final class PhabricatorHelpEditorProtocolController
|
||||
public function handleRequest(AphrontRequest $request) {
|
||||
$viewer = $request->getViewer();
|
||||
|
||||
$dialog = id(new AphrontDialogView())
|
||||
->setUser($viewer)
|
||||
return $this->newDialog()
|
||||
->setMethod('GET')
|
||||
->setSubmitURI('/settings/panel/display/')
|
||||
->setTitle(pht('Unsupported Editor Protocol'))
|
||||
@@ -24,9 +23,6 @@ final class PhabricatorHelpEditorProtocolController
|
||||
phutil_tag('tt', array(), 'uri.allowed-editor-protocols')))
|
||||
->addSubmitButton(pht('Change Settings'))
|
||||
->addCancelButton('/');
|
||||
|
||||
return id(new AphrontDialogResponse())
|
||||
->setDialog($dialog);
|
||||
}
|
||||
|
||||
public static function hasAllowedProtocol($uri) {
|
||||
|
||||
@@ -57,14 +57,11 @@ final class PhabricatorHelpKeyboardShortcutController
|
||||
array('class' => 'keyboard-shortcut-help'),
|
||||
$rows);
|
||||
|
||||
$dialog = id(new AphrontDialogView())
|
||||
->setUser($viewer)
|
||||
return $this->newDialog()
|
||||
->setTitle(pht('Keyboard Shortcuts'))
|
||||
->appendChild($table)
|
||||
->addCancelButton('#', pht('Close'));
|
||||
|
||||
return id(new AphrontDialogResponse())
|
||||
->setDialog($dialog);
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user