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
Unknown Object (File)
Fri, Dec 13, 1:31 PM
Unknown Object (File)
Sun, Dec 8, 8:07 AM
Unknown Object (File)
Fri, Dec 6, 4:54 AM
Unknown Object (File)
Thu, Dec 5, 10:26 PM
Unknown Object (File)
Wed, Nov 27, 8:31 PM
Unknown Object (File)
Mon, Nov 18, 4:53 PM
Unknown Object (File)
Nov 17 2024, 1:29 AM
Unknown Object (File)
Nov 15 2024, 10:24 AM
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.