Fix application name toolip hover

Summary: Ref T12174. We were always setting a name via builtins so the tooltip was always set. Fix the calls here.

Test Plan: Add "Badges", see tooltip, give "Badges" a name of "Badges", don't see tooltip.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

Maniphest Tasks: T12174

Differential Revision: https://secure.phabricator.com/D17284
This commit is contained in:
Chad Little 2017-01-31 19:34:16 -08:00
parent 7fc8e19786
commit 113bdd9f79
2 changed files with 3 additions and 2 deletions

View file

@ -39,7 +39,7 @@ final class PhabricatorHomeProfileMenuEngine
} }
$properties = array( $properties = array(
'name' => $application->getName(), 'name' => '',
'application' => $application->getPHID(), 'application' => $application->getPHID(),
); );

View file

@ -89,7 +89,8 @@ final class PhabricatorApplicationProfileMenuItem
->setIcon($app->getIcon()); ->setIcon($app->getIcon());
// Don't show tooltip if they've set a custom name // Don't show tooltip if they've set a custom name
if (strlen(($config->getMenuItemProperty('name')))) { $name = $config->getMenuItemProperty('name');
if (!strlen($name)) {
$item->setTooltip($app->getShortDescription()); $item->setTooltip($app->getShortDescription());
} }