Page MenuHomePhabricator

Tweak the visual style of the ">>" / "<<" depth change indicators slightly
ClosedPublic

Authored by epriestley on Mar 6 2019, 4:36 PM.
Tags
None
Referenced Files
F19514151: D20251.id48359.diff
Tue, Jan 13, 10:46 PM
F19484416: D20251.id48359.diff
Thu, Jan 8, 2:19 AM
F19320249: D20251.id48359.diff
Thu, Dec 25, 8:37 AM
F19082950: D20251.id48359.diff
Dec 2 2025, 12:28 PM
F19047811: D20251.diff
Nov 27 2025, 1:24 PM
F19047671: D20251.diff
Nov 27 2025, 1:14 PM
F19047510: D20251.diff
Nov 27 2025, 1:03 PM
F19033605: D20251.id.diff
Nov 25 2025, 11:23 AM
Subscribers
None

Details

Summary

Ref T13249.

  • When a line has only increased in indent depth, don't red-fill highlight the left side of the diff. Since reading a diff mostly involves focusing on the right side, indent depth changes are generally visible enough without this extra hint. The extra hint can become distracting in cases where there is a large block of indent depth changes.
  • Move the markers slightly to the left, to align them with the gutter.
  • Make them slightly opaque so they're a little less prominent.
Test Plan

See screenshots.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Indent Depth Increased, Old:

Screen Shot 2019-03-06 at 8.24.12 AM.png (425×1 px, 83 KB)

Indent Depth Increased, New:

Screen Shot 2019-03-06 at 8.29.16 AM.png (422×1 px, 77 KB)

Indent Depth Decreased, Old:

Screen Shot 2019-03-06 at 8.30.30 AM.png (395×1 px, 80 KB)

Indent Depth Decreased, New:

Screen Shot 2019-03-06 at 8.30.07 AM.png (391×1 px, 80 KB)

This revision is now accepted and ready to land.Mar 7 2019, 6:53 PM
This revision was automatically updated to reflect the committed changes.