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

Subscribers

Details

Reviewers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

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.

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

Path

Size

resources/

celerity/

6 lines

webroot/

rsrc/

css/

core/

5 lines

Diff 25847

resources/celerity/map.php

Loading...

webroot/rsrc/css/core/syntax.css

Loading...