Page MenuHomePhabricator
Differential D9734

Move Legalpad signature form inline in document
ClosedPublic
Actions

Authored by epriestley on Jun 25 2014, 9:06 PM.
Tags
None
Referenced Files
F18849554: D9734.id23368.diff
Thu, Oct 30, 10:37 AM
F18844149: D9734.id.diff
Wed, Oct 29, 3:28 AM
F18842330: D9734.diff
Tue, Oct 28, 2:59 PM
F18766895: D9734.diff
Oct 7 2025, 7:16 PM
F18654588: D9734.diff
Sep 22 2025, 4:00 PM
F17840126: D9734.id.diff
Jul 26 2025, 8:48 PM
F17804874: D9734.diff
Jul 25 2025, 11:56 AM
Unknown Object (File)
Apr 28 2025, 9:57 PM
View All Files
Subscribers
epriestley

Details

Reviewers
chad
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPb1a8dbade6fe: Move Legalpad signature form inline in document
Summary

This makes it look a little less weird.

Test Plan

Screen_Shot_2014-06-25_at_2.05.59_PM.png (1×1 px, 120 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 23368.Jun 25 2014, 9:06 PM
epriestley retitled this revision from to Move Legalpad signature form inline in document.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 25 2014, 9:06 PM
Harbormaster completed remote builds in B1397: Diff 23368.Jun 25 2014, 9:08 PM
chad commandeered this revision.Jun 25 2014, 9:31 PM
chad edited reviewers, added: epriestley; removed: chad.

I'll take a look after I get through tags.

chad added a comment.Jun 25 2014, 9:33 PM

I have a half-baked, get rid of all those purple one-off headers branch in my sandbox. I think there is a no-bleed setting for DocumentView, but I'd have to check.

chad added a comment.Jun 26 2014, 5:00 AM

I don't know what I was thinking, I can't yoink this, can I?

epriestley edited edge metadata.Jun 26 2014, 5:01 AM

You should be able to now, I think. I can rebase it if arc patch is giving you issues, gimmie a sec.

epriestley commandeered this revision.Jun 26 2014, 5:01 AM
epriestley edited reviewers, added: chad; removed: epriestley.

(Yoinking to rebase for you.)

chad accepted this revision.Jun 26 2014, 5:02 AM
chad edited edge metadata.

Just go ahead and land, I can clean it up tomorrow.

This revision is now accepted and ready to land.Jun 26 2014, 5:02 AM
epriestley added a comment.Jun 26 2014, 5:02 AM

works for me

epriestley closed this revision.Jun 26 2014, 5:02 AM
epriestley updated this revision to Diff 23390.

Closed by commit rPb1a8dbade6fe (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
legalpad/
controller/
47 lines

Diff 23390

View Options

src/applications/legalpad/controller/LegalpadDocumentSignController.php

Loading...
Phabricator · Built by Phacility