Paths

Table of Contentst

Differential D19457

Add "max-width: 100%;" to stop large images from overflowing the new rendering engine UI
ClosedPublic
Actions

Authored by epriestley on Jun 1 2018, 8:14 PM.

Tags

None

Referenced Files

	F18777026: D19457.id.diff
	Sat, Oct 11, 12:36 AM

	F18768083: D19457.id46548.diff
	Wed, Oct 8, 1:39 AM

	F18761943: D19457.diff
	Mon, Oct 6, 4:57 PM

	F18759693: D19457.diff
	Mon, Oct 6, 5:24 AM

	F18758270: D19457.id46544.diff
	Sun, Oct 5, 9:29 PM

	F18631168: D19457.id.diff
	Sep 16 2025, 2:35 PM

	F18626538: D19457.id46548.diff
	Sep 16 2025, 1:29 AM

	F18621456: D19457.diff
	Sep 15 2025, 8:24 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley
avivey

Maniphest Tasks

T13105: Plans: Rich presentation and diff rendering pipelines for various file types
T13148: css: ginormous images go over preview panel in Files

Commits

rPa894c9993512: Add "max-width: 100%;" to stop large images from overflowing the new rendering…

Summary

Fixes T13148. Ref T13105. The new document rendering engine for images let them overflow the UI bounds.

Add max-width: 100%; to keep them contained.

Test Plan

Viewed a very wide image in Safari, Firefox and Chrome. Saw sensible rendering.
Also viewed a normal image, saw normal behavior.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 1 2018, 8:14 PM

Harbormaster completed remote builds in B20353: Diff 46544.Jun 1 2018, 8:15 PM

epriestley requested review of this revision.Jun 1 2018, 8:15 PM

lgtm

This revision is now accepted and ready to land.Jun 1 2018, 8:24 PM

Closed by commit rPa894c9993512: Add "max-width: 100%;" to stop large images from overflowing the new rendering… (authored by epriestley). · Explain WhyJun 1 2018, 9:53 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

webroot/

rsrc/

css/

phui/

phui-property-list-view.css

1 line

Diff 46548

View Options

resources/celerity/map.php