Homec4science

Fix show/hide applications on homepage

Authored by epriestley <git@epriestley.com> on Feb 5 2013, 16:43.

Description

Fix show/hide applications on homepage

Summary:
Fixes T2482. After D4792, menus have more formal structure, but previously we were just shoving some <div> into the middle of the thing. This no longer works correctly, since we end up with <div class="nice-formal-div"><div></div>.

Just put IDs on all the items we're going to show/hide instead so we don't have to render any half-tags.

Test Plan: Home page show/hide works again.

Reviewers: btrahan, chad

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2482

Differential Revision: https://secure.phabricator.com/D4810

Details

Committed
epriestley <git@epriestley.com>Feb 5 2013, 16:43
Pushed
aubortJan 31 2017, 17:16
Parents
rPHe4a0c170799c: fix Conpherence sorting
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH742ee44960b0: Fix show/hide applications on homepage (authored by epriestley <git@epriestley.com>).Feb 5 2013, 16:43