Page MenuHomePhabricator

[Draft] Toggle inline comments on/off
AbandonedPublic

Authored by chad on May 2 2015, 1:44 AM.
Tags
None
Referenced Files
F19934886: D12650.diff
Sat, Apr 11, 6:53 PM
F19837745: D12650.id30390.diff
Mar 11 2026, 3:21 PM
F19836474: D12650.id30390.diff
Mar 11 2026, 10:44 AM
F19834428: D12650.id30390.diff
Mar 11 2026, 3:57 AM
F19832752: D12650.id30390.diff
Mar 10 2026, 11:56 PM
F19528836: D12650.diff
Jan 19 2026, 12:09 PM
F19515150: D12650.diff
Jan 14 2026, 8:28 AM
F19062288: D12650.id30390.diff
Nov 29 2025, 3:37 PM
Subscribers

Details

Reviewers
epriestley
Summary

Just for testing. Press f to turn on all inline comments.

Test Plan

toggle lots

Diff Detail

Repository
rP Phabricator
Branch
ghost-js-shortcut
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningwebroot/rsrc/js/application/differential/behavior-keyboard-nav.js:189W041JSHintW041
Unit
No Test Coverage
Build Status
Buildable 5666
Build 5685: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad retitled this revision from to [Draft] Toggle inline comments on/off.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.

I can't say this "feels right" either, but I wanted to at least test / play with a keyboard toggle.

My last idea would be to collapse threads, so entire conversations collapse to 1 line, and expand fully with 1-click. But not sure I can fake that with CSS.