revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-01-31 00:48:21 +01:00
Code Issues Releases Wiki Activity

Don't show Space monograms in SpaceContextView

Browse source 
Summary: Ref T8449. Get rid of the `S123` stuff in headers, it feels like clutter.

Test Plan: Looked at objects in Spaces.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T8449

Differential Revision: https://secure.phabricator.com/D13243
This commit is contained in:
epriestley 2015-06-11 10:14:22 -07:00
parent d5668ddeae
commit 325f4c863c
2 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/applications/phid/view/PHUIHandleView.php
View file

@ -14,6 +14,7 @@ final class PHUIHandleView
private $handleList;
private $handlePHID;
private $asTag;
private $useShortName;
public function setHandleList(PhabricatorHandleList $list) {
$this->handleList = $list;
@ -30,12 +31,21 @@ final class PHUIHandleView
return $this;
}
public function setUseShortName($short) {
$this->useShortName = $short;
return $this;
}
public function render() {
$handle = $this->handleList[$this->handlePHID];
if ($this->asTag) {
return $handle->renderTag();
} else {
return $handle->renderLink();
if ($this->useShortName) {
return $handle->renderLink($handle->getName());
} else {
return $handle->renderLink();
}
}
}

2
src/applications/spaces/view/PHUISpacesNamespaceContextView.php
View file

@ -41,7 +41,7 @@ final class PHUISpacesNamespaceContextView extends AphrontView {
'class' => 'spaces-name',
),
array(
$viewer->renderHandle($space_phid),
$viewer->renderHandle($space_phid)->setUseShortName(true),
' | ',
));
}