Page MenuHomePhabricator

Fix application name toolip hover
ClosedPublic

Authored by chad on Feb 1 2017, 3:36 AM.
Tags
None
Referenced Files
F15513545: D17284.diff
Thu, Apr 17, 7:40 PM
F15508352: D17284.diff
Wed, Apr 16, 5:13 AM
F15503068: D17284.id41579.diff
Mon, Apr 14, 6:29 AM
F15486928: D17284.id41585.diff
Thu, Apr 10, 10:01 AM
F15481361: D17284.id41579.diff
Tue, Apr 8, 8:20 PM
F15481066: D17284.diff
Tue, Apr 8, 5:56 PM
F15468704: D17284.id41579.diff
Fri, Apr 4, 4:42 AM
F15458952: D17284.diff
Mon, Mar 31, 8:34 AM
Subscribers

Details

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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Feb 1 2017, 1:28 PM
This revision was automatically updated to reflect the committed changes.