archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Display full path in titles of dirs in tree

Browse Source 
Summary: It's confusing with longer trees.

Test Plan: Displayed it.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D3367
This commit is contained in:
vrana
2012-08-22 14:58:52 -07:00
parent d9f241a9ca
commit 0c7b9e14a9
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/applications/differential/controller/DifferentialRevisionViewController.php
View File

@@ -1038,9 +1038,11 @@ final class DifferentialRevisionViewController extends DifferentialController {
$href = null; $href = null;
if ($data) { if ($data) {
$href = '#'.$data->getAnchorName(); $href = '#'.$data->getAnchorName();
$title = $name;
$icon = 'phabricator-filetree-icon-file'; $icon = 'phabricator-filetree-icon-file';
} else { } else {
$name .= '/'; $name .= '/';
$title = $path->getFullPath().'/';
$icon = 'phabricator-filetree-icon-dir'; $icon = 'phabricator-filetree-icon-dir';
} }
@@ -1063,7 +1065,7 @@ final class DifferentialRevisionViewController extends DifferentialController {
array( array(
'href' => $href, 'href' => $href,
'style' => $style, 'style' => $style,
'title' => $name, 'title' => $title,
'class' => 'phabricator-filetree-item', 'class' => 'phabricator-filetree-item',
), ),
$icon.$name_element); $icon.$name_element);