Page MenuHomePhabricator

Scope remarkup highlighting better
ClosedPublic

Authored by chad on May 2 2015, 3:59 PM.
Tags
None
Referenced Files
F16074117: D12657.id30402.diff
Fri, Jun 6, 8:31 AM
Unknown Object (File)
Thu, May 29, 1:17 AM
Unknown Object (File)
Tue, May 27, 5:07 PM
Unknown Object (File)
Apr 30 2025, 3:26 AM
Unknown Object (File)
Apr 25 2025, 1:21 AM
Unknown Object (File)
Apr 23 2025, 7:17 PM
Unknown Object (File)
Apr 22 2025, 4:06 PM
Unknown Object (File)
Apr 15 2025, 7:02 AM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP3da020b4931c: Scope remarkup highlighting better
Summary

These remarkup spans bleed into other areas, but afaik they only seem to be needed on the bright rule. Also normalized the height of the label, which changed when we changed button size.

Test Plan

Review many diffs and inline comments. If this rule altered something else I can't find, let me know?

Diff Detail

Repository
rP Phabricator
Branch
ghost-16
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5676
Build 5695: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad retitled this revision from to Scope remarkup highlighting better.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 2 2015, 4:00 PM
This revision was automatically updated to reflect the committed changes.