Page MenuHomePhabricator
Differential D10768

Apply remarkup-code padding to child spans as well
ClosedPublic
Actions

Authored by chad on Oct 31 2014, 5:42 PM.
Tags
None
Referenced Files
F18934902: D10768.diff
Nov 10 2025, 5:10 PM
F18871912: D10768.diff
Nov 5 2025, 1:21 AM
F18854472: D10768.id.diff
Nov 1 2025, 12:19 AM
F18838887: D10768.diff
Oct 27 2025, 1:16 PM
F18803414: D10768.id.diff
Oct 17 2025, 11:32 PM
F18762220: D10768.id.diff
Oct 6 2025, 5:57 PM
F18757408: D10768.diff
Oct 5 2025, 5:21 PM
F18712276: D10768.diff
Sep 29 2025, 5:19 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T6440: Green background on syntax highlighted code for char-level changes has weird padding
Commits
Restricted Diffusion Commit
rP53493ccf9396: Apple remarkup-code padding to child spans as well.
Summary

Fixes T6440. The issue is when a nested span appears, we don't apply the spacing.

Test Plan

Test a new diff in my sandbox that exhibits the issue. Ensure spacing now aligns.

Diff Detail

Repository
rP Phabricator
Branch
diff-padding
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2924
Build 2928: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 25846.Oct 31 2014, 5:42 PM
chad retitled this revision from to Apple remarkup-code padding to child spans as well..
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T6440: Green background on syntax highlighted code for char-level changes has weird padding.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptOct 31 2014, 5:42 PM
Harbormaster completed remote builds in B2924: Diff 25846.Oct 31 2014, 5:43 PM
epriestley accepted this revision.Oct 31 2014, 5:44 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Oct 31 2014, 5:44 PM
epriestley added a comment.Oct 31 2014, 5:44 PM

First time I've seen "apple" used as a verb.

Closed by commit rP53493ccf9396: Apple remarkup-code padding to child spans as well. (authored by chad, committed by chad). · Explain WhyOct 31 2014, 5:45 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Oct 31 2014, 5:46 PM

Yeah, phat phingered.

chad retitled this revision from Apple remarkup-code padding to child spans as well. to Apply remarkup-code padding to child spans as well.Oct 31 2014, 5:47 PM
chad edited edge metadata.

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
webroot/
rsrc/
css/
core/
5 lines

Diff 25846

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/css/core/syntax.css

Loading...
Phabricator · Built by Phacility