Page MenuHomePhabricator
Differential D15840

Add "wide" remarkup image support for Documents
ClosedPublic
Actions

Authored by chad on May 3 2016, 8:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 4, 7:15 PM
Unknown Object (File)
Wed, May 1, 1:01 AM
Unknown Object (File)
Fri, Apr 12, 5:22 AM
Unknown Object (File)
Mar 5 2024, 12:32 PM
Unknown Object (File)
Feb 12 2024, 6:54 PM
Unknown Object (File)
Feb 10 2024, 10:36 AM
Unknown Object (File)
Feb 8 2024, 6:39 AM
Unknown Object (File)
Feb 8 2024, 6:39 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rPd85386488bf5: Add "wide" remarkup image support for Documents
Summary

Seems to work ok, if you give size=wide to an image file, we blow it out a bit in DocumentPro mode.

Test Plan

Test in Phame and Maniphest.

Screen Shot 2016-05-03 at 1.54.17 PM.png (2×2 px, 1 MB)

Screen Shot 2016-05-03 at 1.54.27 PM.png (2×2 px, 1 MB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 38153.May 3 2016, 8:56 PM
chad retitled this revision from to Add "wide" remarkup image support for Documents.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptMay 3 2016, 8:56 PM
epriestley accepted this revision.May 3 2016, 8:56 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 3 2016, 8:56 PM
chad edited the test plan for this revision. (Show Details)May 3 2016, 8:56 PM
chad edited edge metadata.
chad added inline comments.
webroot/rsrc/css/phui/phui-document.css
106

just... be on the lookout?

epriestley added inline comments.May 3 2016, 8:57 PM
webroot/rsrc/css/phui/phui-document.css
106

haha yeah this might get suspicious

epriestley added a comment.May 3 2016, 8:57 PM

Only thing I can really think of offhand is huge tables.

chad added a comment.May 3 2016, 8:58 PM

lemme do more testing then. I think tables overflow scroll though

epriestley added a comment.May 3 2016, 8:59 PM

Yeah, I'd expect them to be OK.

Maybe a table with a {wide} image inside it? Not sure what users would really expect that to do, though.

Closed by commit rPd85386488bf5: Add "wide" remarkup image support for Documents (authored by chad, committed by chad). · Explain WhyMay 4 2016, 12:27 AM
This revision was automatically updated to reflect the committed changes.
chad added a comment.May 4 2016, 12:29 AM

doitlive

chad added a comment.May 4 2016, 12:29 AM

pls roll secure

Revision Contents
Changeset List

PathSize
resources/
celerity/
14 lines
src/
applications/
files/
markup/
8 lines
webroot/
rsrc/
css/
core/
6 lines
phui/
19 lines
1 line

Diff 38158

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/files/markup/PhabricatorEmbedFileRemarkupRule.php

Loading...
View Options

webroot/rsrc/css/core/remarkup.css

Loading...
View Options

webroot/rsrc/css/phui/phui-document-pro.css

Loading...
View Options

webroot/rsrc/css/phui/phui-document.css

Loading...
Phabricator · Built by Phacility