Page MenuHomePhabricator

To improve wrapping behavior of rendered README files, don't use "PHUIDocumentView" in Diffusion
ClosedPublic

Authored by epriestley on May 21 2019, 1:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
May 10 2025, 2:39 AM
Unknown Object (File)
May 8 2025, 3:22 AM
Unknown Object (File)
May 6 2025, 8:12 AM
Unknown Object (File)
May 6 2025, 7:06 AM
Unknown Object (File)
Apr 27 2025, 7:04 AM
Unknown Object (File)
Apr 26 2025, 9:51 AM
Unknown Object (File)
Apr 20 2025, 11:12 PM
Unknown Object (File)
Apr 20 2025, 9:01 AM
Subscribers
None

Details

Summary

See PHI1268. We currently do some weird width handling when rendering Diffusion readmes in a document directory view.

I think this came from D12330, which used PHUIDocumentViewPro to change the font, but we later reverted the font and were left with the DocumentView. Other changes after that modified DocumentView to have fixed-width behavior, but it doesn't make much sense here since the content panel is clearly rendered full-width.

Today, the DocumentView is a more structural element with methods like setCurtain(). Just get rid of it to simplify things, at least as a first step.

Test Plan

Before:

Screen Shot 2019-05-21 at 6.25.59 AM.png (890×1 px, 194 KB)

After:

Screen Shot 2019-05-21 at 6.30.04 AM.png (777×2 px, 239 KB)

Diff Detail

Repository
rP Phabricator
Branch
resize1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22843
Build 31340: Run Core Tests
Build 31339: arc lint + arc unit