Page MenuHomePhabricator

Don't drop "phabricator-remarkup-embed-image" class from Remarkup images with `width` or `height`
ClosedPublic

Authored by epriestley on Jan 9 2016, 9:49 PM.
Tags
None
Referenced Files
F14046543: D14983.id.diff
Wed, Nov 13, 9:47 PM
F14016287: D14983.id36194.diff
Mon, Nov 4, 7:41 AM
F13994688: D14983.id36193.diff
Wed, Oct 23, 7:49 AM
F13986849: D14983.id36194.diff
Mon, Oct 21, 5:56 AM
F13960953: D14983.id.diff
Oct 15 2024, 2:18 AM
Unknown Object (File)
Oct 7 2024, 3:15 PM
Unknown Object (File)
Sep 27 2024, 8:20 PM
Unknown Object (File)
Sep 24 2024, 9:08 AM
Subscribers
None

Details

Summary

Ref T10110. If an image had width or height, we would accidentally not give it an $image_class.

Test Plan

Screen Shot 2016-01-09 at 1.44.52 PM.png (1×967 px, 696 KB)

Screen Shot 2016-01-09 at 1.46.17 PM.png (889×1 px, 321 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Give Remarkup images inline-block layout so "layout=inline" works.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

This is wrong.

epriestley edited edge metadata.
  • Proper fix.
epriestley retitled this revision from Give Remarkup images inline-block layout so "layout=inline" works to Don't drop "phabricator-remarkup-embed-image" class from Remarkup images with `width` or `height`.Jan 9 2016, 9:56 PM
epriestley updated this object.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 9 2016, 10:49 PM
This revision was automatically updated to reflect the committed changes.