Page MenuHomePhabricator

Make dialogs a little easier to use
ClosedPublic

Authored by epriestley on Mar 20 2014, 1:04 PM.
Tags
None
Referenced Files
F19714448: D8577.diff
Thu, Feb 12, 1:39 AM
F19616430: D8577.diff
Wed, Feb 4, 2:44 AM
F19582594: D8577.diff
Sun, Feb 1, 10:58 PM
F19523702: D8577.id20371.diff
Sat, Jan 17, 6:48 PM
F19470740: D8577.id20353.diff
Jan 5 2026, 10:54 PM
F19357176: D8577.id20371.diff
Dec 26 2025, 7:46 AM
F18944224: D8577.id.diff
Nov 11 2025, 3:35 PM
F18875332: D8577.id.diff
Nov 5 2025, 5:36 PM
Subscribers

Details

Reviewers
btrahan
chad
Commits
Restricted Diffusion Commit
rPd8713f6f0b8c: Make dialogs a little easier to use
Summary
  • Dialog pages currently have no titles or crumbs, and look shoddy. Add titles and crumbs.
  • Dialog titles aren't always great for crumbs, add an optional "short title" for crumbs.
  • AphrontDialogResponse is pure boilerplate. Allow controllers to just return a DialogView instead and get the same effect.
  • Building dialogs requires a bit of boilerplate, and we generally construct them with no explicit "action", which has some issues with T4593. Provide a convenience method to set the viewer and get a reasonable, explict submit URI.
Test Plan
  • Viewed dialog on its own.
  • Viewed dialog as a dialog.

Screen_Shot_2014-03-20_at_6.00.44_AM.png (993×1 px, 103 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Make dialogs a little easier to use.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 20 2014, 3:37 PM
epriestley updated this revision to Diff 20371.

Closed by commit rPd8713f6f0b8c (authored by @epriestley).