Page MenuHomePhabricator

Restore "h" to hide or show files, and modernize file visibility toggling
ClosedPublic

Authored by epriestley on May 17 2017, 7:54 PM.
Tags
None
Referenced Files
F19434354: D17940.id43172.diff
Sun, Dec 28, 2:26 PM
F19259089: D17940.diff
Mon, Dec 22, 3:13 PM
F19096197: D17940.diff
Dec 4 2025, 8:57 AM
F19095485: D17940.diff
Dec 4 2025, 5:14 AM
F18926918: D17940.id43172.diff
Nov 9 2025, 8:29 PM
F18915139: D17940.id.diff
Nov 9 2025, 7:30 AM
F18863274: D17940.diff
Nov 2 2025, 10:37 PM
F18808522: D17940.id.diff
Oct 19 2025, 6:56 AM
Subscribers
None

Details

Summary

Ref T12616. This puts "h" back to collapse or expand the current file.

This removes some very complicated/messy code around following links in the table of contents and getting files auto-expanded. I suspect no one will miss this, but we can restore it if ayone notices.

Test Plan

Pressed "h" to collapse/expand a file. Also used the menu items.

Diff Detail

Repository
rP Phabricator
Branch
inline24
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17068
Build 22824: Run Core Tests
Build 22823: arc lint + arc unit

Event Timeline

webroot/rsrc/css/application/differential/changeset-view.css
328

This at least makes the weird collapsed state a little less weird/goofy looking (see T12688).

webroot/rsrc/js/application/diff/DiffChangesetList.js
369–386

Oops, some copy-paste slipup nonsense here, one sec.

  • Copy/paste with greater finesse.
This revision is now accepted and ready to land.May 18 2017, 3:02 PM
This revision was automatically updated to reflect the committed changes.