Differential D20971

Respect linebreaks in full HTML tables in Remarkup
ClosedPublic
Actions

Authored by epriestley on Feb 6 2020, 10:53 PM.

Tags

None

Referenced Files

	F13095331: D20971.diff
	Thu, Apr 25, 8:30 AM

	Unknown Object (File)
	Fri, Apr 19, 3:20 AM

	Unknown Object (File)
	Fri, Apr 19, 3:20 AM

	Unknown Object (File)
	Fri, Apr 19, 3:20 AM

	Unknown Object (File)
	Thu, Apr 18, 1:23 PM

	Unknown Object (File)
	Tue, Apr 16, 9:19 AM

	Unknown Object (File)
	Thu, Apr 11, 4:30 AM

	Unknown Object (File)
	Mon, Apr 8, 10:48 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T5427: Force a line break in a table cell
Commits: rP2327578adc94: Respect linebreaks in full HTML tables in Remarkup

Summary

Fixes T5427. See PHI1630. See also T13160 and D20568.

In the full HTML table syntax with "<table>", respect linebreaks as literals inside "<td>" cells.

Test Plan

Previewed some full-HTML tables with and without linebreaks, saw what seemed like sensible rendering behavior.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 6 2020, 10:53 PM

Harbormaster completed remote builds in B23793: Diff 49968.Feb 6 2020, 10:54 PM

epriestley requested review of this revision.Feb 6 2020, 10:54 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 6 2020, 11:01 PM

Closed by commit rP2327578adc94: Respect linebreaks in full HTML tables in Remarkup (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP2327578adc94: Respect linebreaks in full HTML tables in Remarkup.

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

markup/

blockrule/

PhutilRemarkupTableBlockRule.php

13 lines

Diff 49969

View Options

src/infrastructure/markup/blockrule/PhutilRemarkupTableBlockRule.php

Respect linebreaks in full HTML tables in RemarkupClosedPublicActions