Page MenuHomePhabricator

Remove dedicated "Jump Nav" navigation element
ClosedPublic

Authored by lpriestley on May 26 2014, 5:48 PM.
Tags
None
Referenced Files
F14450499: D9301.id22074.diff
Fri, Dec 27, 3:55 AM
Unknown Object (File)
Thu, Dec 26, 10:39 PM
Unknown Object (File)
Tue, Dec 24, 4:13 AM
Unknown Object (File)
Mon, Dec 23, 9:51 AM
Unknown Object (File)
Thu, Dec 19, 10:03 PM
Unknown Object (File)
Thu, Dec 19, 6:43 PM
Unknown Object (File)
Sat, Dec 7, 6:40 PM
Unknown Object (File)
Thu, Dec 5, 2:06 AM
Subscribers

Details

Summary

Fixes T5175. Not sure if I cleaned out everything, but this seemed like a reasonable first pass. Attempted to delete all code that belonged to Jump Nav feature only.

Test Plan

Open phabricator homepage, verify Jump Nav element is gone, verify the Search bar still autocompletes and jumps to shortcuts.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

lpriestley retitled this revision from to Remove dedicated "Jump Nav" navigation element.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
epriestley edited edge metadata.

This looks like everything to me.

This revision is now accepted and ready to land.May 26 2014, 6:15 PM
epriestley updated this revision to Diff 22075.

Closed by commit rP607f4bfab8d1 (authored by @lpriestley, committed by @epriestley).