Clean up Macro view page
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F18827773: D15406.id37144.diff
	Fri, Oct 24, 12:02 PM

	F18777964: D15406.diff
	Sat, Oct 11, 6:52 AM

	F18701719: D15406.id37144.diff
	Sat, Sep 27, 8:46 PM

	F18652098: D15406.diff
	Sep 21 2025, 6:36 AM

	F18651563: D15406.diff
	Sep 21 2025, 4:39 AM

	F18589750: D15406.id.diff
	Sep 12 2025, 4:11 AM

	F18586644: D15406.diff
	Sep 11 2025, 7:17 PM

	F18164947: D15406.id37142.diff
	Aug 15 2025, 7:23 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