Page MenuHomePhabricator

Allow form configurations to retitle and reorder forms and add preambles
ClosedPublic

Authored by epriestley on Nov 17 2015, 8:37 PM.
Tags
None
Referenced Files
F15666008: D14503.id.diff
Sun, May 25, 6:36 AM
Unknown Object (File)
Sun, May 18, 3:53 AM
Unknown Object (File)
Sun, May 18, 3:53 AM
Unknown Object (File)
Fri, May 9, 6:51 PM
Unknown Object (File)
Thu, May 8, 7:57 AM
Unknown Object (File)
Wed, May 7, 4:06 PM
Unknown Object (File)
Apr 24 2025, 10:43 PM
Unknown Object (File)
Apr 24 2025, 10:43 PM
Subscribers

Details

Summary

Ref T9132. This just makes edited forms do something, albeit not anything very useful yet.

You can now edit a form and:

  • Retitle it;
  • add a preamble (instructions on top of the form); and
  • reorder the form's fields.
Test Plan

Screen Shot 2015-11-17 at 12.18.22 PM.png (1×1 px, 155 KB)

Screen Shot 2015-11-17 at 12.17.57 PM.png (981×1 px, 129 KB)

Screen Shot 2015-11-17 at 12.18.08 PM.png (1×1 px, 149 KB)

Screen Shot 2015-11-17 at 12.18.06 PM.png (1×1 px, 159 KB)

Screen Shot 2015-11-17 at 12.18.05 PM.png (1×1 px, 164 KB)

Diff Detail

Repository
rP Phabricator
Branch
eengine16
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/applications/base/PhabricatorApplication.php:300XHP86Unsafe Usage of Dynamic String
Errorsrc/applications/base/PhabricatorApplication.php:302XHP86Unsafe Usage of Dynamic String
Warningwebroot/rsrc/js/application/transactions/behavior-reorder-fields.js:1JAVELIN5`javelinsymbols` Not In Path
Unit
Tests Passed
Build Status
Buildable 8898
Build 10418: Run Core Tests
Build 10417: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Allow form configurations to retitle and reorder forms and add preambles.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 18 2015, 6:12 AM

what is this magic

magicfingers

(I gather this editor refactoring is designed to support form editing in Nuance, and as part of that, you're unifying all of the other forms in Phabricator to use the same system)

This revision was automatically updated to reflect the committed changes.