Clean up Macro view page
ClosedPublic
Actions

Authored by chad on Mar 5 2016, 8:30 PM.

Tags

None

Referenced Files

	F19511679: D15406.diff
	Mon, Jan 12, 6:36 AM

	F19071474: D15406.diff
	Nov 30 2025, 7:40 PM

	F19025358: D15406.diff
	Nov 24 2025, 6:42 AM

	F18998163: D15406.diff
	Nov 20 2025, 5:43 PM

	F18994766: D15406.diff
	Nov 19 2025, 2:47 PM

	F18967356: D15406.diff
	Nov 14 2025, 8:46 PM

	F18848527: D15406.id.diff
	Oct 30 2025, 5:09 AM

	F18848400: D15406.id.diff
	Oct 30 2025, 4:41 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rP090252a89e2b: Clean up Macro view page

Summary

Sets a header icon, makes "Details" not show if empty, simplifies title.

Test Plan

Review a few Macro pages for changes with and without audio.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 37142.Mar 5 2016, 8:30 PM

chad retitled this revision from to Clean up Macro view page.

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 TranscriptMar 5 2016, 8:30 PM

epriestley accepted this revision.Mar 5 2016, 8:41 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Mar 5 2016, 8:41 PM

Closed by commit rP090252a89e2b: Clean up Macro view page (authored by chad, committed by chad). · Explain WhyMar 5 2016, 8:42 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

macro/

controller/

PhabricatorMacroViewController.php

9 lines

Diff 37144

View Options

src/applications/macro/controller/PhabricatorMacroViewController.php

Clean up Macro view pageClosedPublicActions