Page MenuHomePhabricator

When printing, wrap all content in Remarkup tables more aggressively
ClosedPublic

Authored by epriestley on Sep 28 2020, 4:22 PM.
Tags
None
Referenced Files
F18108524: D21472.id51108.diff
Mon, Aug 11, 7:23 AM
F18103469: D21472.id51107.diff
Sun, Aug 10, 5:41 AM
F18102390: D21472.id.diff
Sat, Aug 9, 8:30 PM
F18097673: D21472.diff
Fri, Aug 8, 2:55 PM
Unknown Object (File)
Jun 5 2025, 5:04 PM
Unknown Object (File)
Jun 2 2025, 4:27 AM
Unknown Object (File)
Jun 1 2025, 5:43 AM
Unknown Object (File)
May 31 2025, 5:50 PM
Subscribers
None

Details

Summary

Ref T13564. See PHI1798. Earlier efforts here (see D21439) still leave us with:

  • Incorrect behavior for long URIs, like http://www.example.com/MMMMM....
  • Incorrect beahvior for long text blocks, like MMMMMM....
  • Undesirable behavior for monospaced text in non-printing contexts (it wraps when we'd prefer it not wrap).

Apply the wrapping rules to all "<td>" content to resolve these three prongs.

Test Plan
  • Viewed long URIs, text blocks, and monospaced text in and out of tables, while printed and not printed, in Safari, Firefox, and Chrome.
  • All browser behavior now appears to be correct ("all content is preserved in printed document").
  • Some browser behavior when making wrapping choices is questionable, but I can't find an automatic solution for that.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Here's the remaining goofy behavior:

Screen Shot 2020-09-28 at 8.59.24 AM.png (447×762 px, 43 KB)

This revision was not accepted when it landed; it landed in state Needs Review.Sep 28 2020, 4:47 PM
This revision was automatically updated to reflect the committed changes.