Differential Action List on Mobile
ClosedPublic
Actions

Authored by chad on Apr 3 2014, 6:58 PM.

Tags

None

Referenced Files

	F13218967: D8691.id20615.diff
	Sat, May 18, 2:15 PM

	F13210587: D8691.diff
	Fri, May 17, 5:01 AM

	F13194298: D8691.diff
	Sun, May 12, 9:28 PM

	F13179106: D8691.diff
	Wed, May 8, 8:56 PM

	Unknown Object (File)
	Sat, May 4, 7:49 PM

	Unknown Object (File)
	Sat, May 4, 6:24 PM

	Unknown Object (File)
	Sun, Apr 28, 12:07 PM

	Unknown Object (File)
	Sun, Apr 28, 12:07 PM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley
btrahan

Maniphest Tasks

T4730: Mobile Differential is missing Action Menu

Commits

Restricted Diffusion Commit
rP7446402f921a: Differential Action List on Mobile

Summary

Make the actions appear in crumbs on mobile

Test Plan

Test action list on a mobile diff layout

Diff Detail

Repository

rP Phabricator

Branch

diff-actions

Lint

Lint Passed

Unit

Tests Passed

Event Timeline

chad updated this revision to Diff 20615.Apr 3 2014, 6:58 PM

chad retitled this revision from to Differential Action List on Mobile.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added reviewers: epriestley, btrahan.

Herald added subscribers: Korvin, epriestley.Apr 3 2014, 6:58 PM

epriestley accepted this revision.Apr 3 2014, 7:00 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 3 2014, 7:00 PM

Closed by commit rP7446402f921a (authored by @chad).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

controller/

DifferentialRevisionViewController.php

1 line

view/

DifferentialRevisionDetailView.php

11 lines

Diff 20615

View Options

src/applications/differential/controller/DifferentialRevisionViewController.php