Make "A" hide or show all inline comments
Summary: Ref T12733. See PHI17.
Test Plan: Pressed "A", then pressed "A".
Reviewers: chad
Reviewed By: chad
Maniphest Tasks: T12733
Differential Revision: https://secure.phabricator.com/D18274
Description
Description
Details
Details
- Provenance
epriestley Authored on Jul 24 2017, 11:30 PM epriestley Pushed on Jul 25 2017, 12:12 PM - Reviewer
- chad
- Differential Revision
- D18274: Make "A" hide or show all inline comments
- Parents
- rP8034b9d819eb: Don't require a device be registered in Almanac to do cluster init/resync steps
- Branches
- Unknown
- Tags
- Tasks
- T12733: (2017 Week 20) Inline Comments Errata / Feedback
- Build Status
Buildable 17799 Build 23903: Run Core Tests
Event Timeline
Comment Actions
Ahh I'm so glad to see this! I wrote a GM script to do this for me and it was quite a hassle to get the scrolling just right. We have some comment chains in our diffs that are like 20 comments long and it really disrupts the reviewing (even collapsed).