Page MenuHomePhabricator

Remove an old Differential selection behavior
ClosedPublic

Authored by epriestley on Feb 20 2019, 4:58 AM.
Tags
None
Referenced Files
F16664402: D20196.diff
Tue, Jun 17, 12:52 PM
Unknown Object (File)
May 5 2025, 9:35 PM
Unknown Object (File)
Apr 29 2025, 7:31 PM
Unknown Object (File)
Apr 24 2025, 7:57 PM
Unknown Object (File)
Apr 24 2025, 6:48 PM
Unknown Object (File)
Apr 23 2025, 8:07 PM
Unknown Object (File)
Apr 23 2025, 7:58 PM
Unknown Object (File)
Apr 22 2025, 10:20 AM
Subscribers
None

Details

Summary

Ref T12822. Ref PHI878. This is some leftover code from the old selection behavior that prevented visual selection of the left side of a diff if the user clicked on the right -- basically, a much simpler attack on what ultimately landed in D20191.

I think the change from th to td "broke" it so it didn't interfere with the other behavior, which is why I didn't have to remove it earlier. It's no longer necessary, in any case.

Test Plan

Grepped for behavior name, selected stuff on both sides of a diff.

Diff Detail

Repository
rP Phabricator
Branch
diff2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22080
Build 30160: Run Core Tests
Build 30159: arc lint + arc unit