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
Unknown Object (File)
Fri, Feb 14, 3:52 AM
Unknown Object (File)
Tue, Feb 11, 3:22 PM
Unknown Object (File)
Sat, Feb 8, 11:49 PM
Unknown Object (File)
Fri, Feb 7, 3:45 AM
Unknown Object (File)
Jan 29 2025, 10:32 PM
Unknown Object (File)
Jan 29 2025, 10:32 PM
Unknown Object (File)
Jan 27 2025, 9:29 PM
Unknown Object (File)
Jan 26 2025, 7:30 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.