Paths

Table of Contentst

Don't show description box in Pholio when there isnt one
ClosedPublic
Actions

Authored by chad on Jun 16 2014, 8:18 PM.

Tags

None

Referenced Files

	F13229926: D9580.diff
	Mon, May 20, 7:42 PM

	F13227541: D9580.diff
	Mon, May 20, 4:02 AM

	F13223514: D9580.diff
	Sun, May 19, 4:41 AM

	F13205552: D9580.diff
	Wed, May 15, 2:08 AM

	F13201839: D9580.id23013.diff
	Tue, May 14, 5:20 PM

	F13201832: D9580.id23013.diff
	Tue, May 14, 5:12 PM

	F13189765: D9580.diff
	Sat, May 11, 6:21 AM

	Unknown Object (File)
	Tue, May 7, 9:52 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rP125b8dad7bb5: Don't show description box in Pholio when there isnt one

Summary

n/a

Test Plan

Added title and description to 1 of 2 mocks, toggled left and right, saw correct CSS.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

chad updated this revision to Diff 22958.Jun 16 2014, 8:18 PM

chad retitled this revision from to Don't show description box in Pholio when there isnt one.

chad updated this object.

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

chad added a reviewer: epriestley.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJun 16 2014, 8:18 PM

Harbormaster completed remote builds in B1150: Diff 22958.Jun 16 2014, 8:20 PM

JSHint would like var x to appear only once in a function, instead of in each branch:

var x;
if (y) {
  x = a;
} else {
  x = b;
}

This revision is now accepted and ready to land.Jun 17 2014, 11:14 AM

Fix lint

Closed by commit rP125b8dad7bb5 (authored by @chad).

Harbormaster completed remote builds in B1181: Diff 23013.Jun 17 2014, 5:11 PM

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

34 lines

webroot/

rsrc/

js/

application/

pholio/

behavior-pholio-mock-view.js

15 lines

Diff 23014

View Options

resources/celerity/map.php