Fix overescaping for Repositories home screen
Summary: Fixes T2623.
Test Plan:
{F34157}
{F34158}
Reviewers: chad, vrana
Reviewed By: chad
CC: aran
Maniphest Tasks: T2623
Differential Revision: https://secure.phabricator.com/D5156
This commit is contained in:
@@ -147,18 +147,27 @@ final class DiffusionHomeController extends DiffusionController {
|
|||||||
|
|
||||||
$repository_tool_uri = PhabricatorEnv::getProductionURI('/repository/');
|
$repository_tool_uri = PhabricatorEnv::getProductionURI('/repository/');
|
||||||
$repository_tool = phutil_tag('a',
|
$repository_tool = phutil_tag('a',
|
||||||
array(
|
array(
|
||||||
'href' => $repository_tool_uri,
|
'href' => $repository_tool_uri,
|
||||||
),
|
),
|
||||||
'repository tool');
|
'repository tool');
|
||||||
$no_repositories_txt = 'This instance of Phabricator does not have any '.
|
$preface = pht('This instance of Phabricator does not have any '.
|
||||||
'configured repositories. ';
|
'configured repositories.');
|
||||||
if ($user->getIsAdmin()) {
|
if ($user->getIsAdmin()) {
|
||||||
$no_repositories_txt .= 'To setup one or more repositories, visit the '.
|
$no_repositories_txt = hsprintf(
|
||||||
$repository_tool.'.';
|
'%s %s',
|
||||||
|
$preface,
|
||||||
|
pht(
|
||||||
|
'To setup one or more repositories, visit the %s.',
|
||||||
|
$repository_tool));
|
||||||
} else {
|
} else {
|
||||||
$no_repositories_txt .= 'Ask an administrator to setup one or more '.
|
$no_repositories_txt = hsprintf(
|
||||||
'repositories via the '.$repository_tool.'.';
|
'%s %s',
|
||||||
|
$preface,
|
||||||
|
pht(
|
||||||
|
'Ask an administrator to setup one or more repositories '.
|
||||||
|
'via the %s.',
|
||||||
|
$repository_tool));
|
||||||
}
|
}
|
||||||
|
|
||||||
$table = new AphrontTableView($rows);
|
$table = new AphrontTableView($rows);
|
||||||
|
|||||||
Reference in New Issue
Block a user