Paths

Table of Contentst

Add ability to pass `width` arg to dot remarkup interpeter
ClosedPublic
Actions

Authored by dctrwatson on Feb 14 2014, 8:39 PM.

Tags

None

Referenced Files

	F18841570: D8235.id.diff
	Tue, Oct 28, 9:55 AM

	F18838620: D8235.diff
	Mon, Oct 27, 11:47 AM

	F18775248: D8235.id19596.diff
	Fri, Oct 10, 6:25 PM

	F18762507: D8235.id19596.diff
	Oct 6 2025, 7:23 PM

	F18754640: D8235.id19595.diff
	Oct 5 2025, 12:10 AM

	F18735464: D8235.id19598.diff
	Oct 1 2025, 1:42 AM

	F18673545: D8235.id19599.diff
	Sep 25 2025, 11:05 AM

	F18601110: D8235.diff
	Sep 13 2025, 11:32 AM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rPdbe3818d8c90: Add ability to pass `width` arg to dot remarkup interpeter

Summary

Allows to keep really wide graphs inside the div:

dot (width=100%) {{{ }}}

Test Plan

Created a graph that is wider than a phriction page.
Added (width=100%) and now the images stays within the div.

Diff Detail

Repository

rP Phabricator

Branch

dot

Lint

Lint Errors

Severity	Location	Code	Message
Error	src/applications/metamta/garbagecollector/MetaMTAMailReceivedGarbageCollector.php:7	PHL1	Unknown Symbol
Error	src/applications/metamta/garbagecollector/MetaMTAMailSentGarbageCollector.php:7	PHL1	Unknown Symbol
Error	src/applications/people/garbagecollector/PeopleUserLogGarbageCollector.php:7	PHL1	Unknown Symbol

Unit

Tests Passed

Event Timeline

Change to width like {F...} objects

epriestley requested changes to this revision.Feb 14 2014, 9:03 PM

epriestley added inline comments.

src/infrastructure/markup/interpreter/PhabricatorRemarkupBlockInterpreterGraphviz.php
19	This will produce `"width: "` for `dot (width=quack) {{{ ... }}}`.
46	We'll fatal here for `dot {{{ ... }}}` with no `width`, I think.

Make it work without width arg
Allow for more style options in future

Thanks!

Closed by commit rPdbe3818d8c90 (authored by @dctrwatson, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

markup/

interpreter/

PhabricatorRemarkupBlockInterpreterGraphviz.php

3 lines

Diff 19595

View Options

src/infrastructure/markup/interpreter/PhabricatorRemarkupBlockInterpreterGraphviz.php

Add ability to pass `width` arg to dot remarkup interpeterClosedPublicActions