Page MenuHomePhorge

Hide the "hidden" fields on custom form previews
Needs RevisionPublic

Authored by 20after4 on Apr 24 2022, 18:19.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 27, 21:46
Unknown Object (File)
Fri, Jan 27, 21:46
Unknown Object (File)
Wed, Jan 25, 05:47
Unknown Object (File)
Dec 19 2022, 21:48
Unknown Object (File)
Dec 19 2022, 21:45
Unknown Object (File)
Dec 18 2022, 19:05
Unknown Object (File)
Dec 16 2022, 17:15
Unknown Object (File)
Dec 16 2022, 17:15

Details

Summary
Test Plan

This just hides some fields via css. Tested in Wikimedia's fork. I believe it results in better UX.

Diff Detail

Repository
rP Phorge
Branch
phorge.it (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 66
Build 66: arc lint + arc unit

Event Timeline

avivey added a subscriber: avivey.

I'm good with this, but we have a policy of "no landed diffs w/o tickets", so please create one before landing.

This revision is now accepted and ready to land.Oct 28 2022, 10:28
In D25037#1857, @avivey wrote:

I'm good with this, but we have a policy of "no landed diffs w/o tickets", so please create one before landing.

This revision is part of T15081.

Matthew requested changes to this revision.Dec 8 2022, 22:34

This does not land cleanly, please merge from master and re-submit.

matthew@Tower phorge % arc land
[2022-12-08 23:33:09] EXCEPTION: (PhutilBootloaderException) The phutil library 'phorge' has not been loaded! at [<arcanist>/src/init/lib/PhutilBootloader.php:199]
PHP Fatal error:  Uncaught exception 'PhutilBootloaderException' with message 'The phutil library 'phorge' has not been loaded!' in /Users/matthew/PhpstormProjects/Phorge/arcanist/src/init/lib/PhutilBootloader.php:199
Stack trace:
#0 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/init/lib/moduleutils.php(5): PhutilBootloader->getLibraryRoot('phabricator')
#1 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(482): phutil_get_library_root('phabricator')
#2 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(317): PhutilErrorHandler::getLibraryVersions()
#3 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(300): PhutilErrorHandler::formatStacktrace(Array)
#4 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(417): PhutilErrorHandler::outputStacktrace(Array)
#5 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(282): PhutilErrorHandler::dispatchErrorMessage('exception', Object( in /Users/matthew/PhpstormProjects/Phorge/arcanist/src/init/lib/PhutilBootloader.php on line 199

Fatal error: Uncaught exception 'PhutilBootloaderException' with message 'The phutil library 'phorge' has not been loaded!' in /Users/matthew/PhpstormProjects/Phorge/arcanist/src/init/lib/PhutilBootloader.php:199
Stack trace:
#0 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/init/lib/moduleutils.php(5): PhutilBootloader->getLibraryRoot('phabricator')
#1 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(482): phutil_get_library_root('phabricator')
#2 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(317): PhutilErrorHandler::getLibraryVersions()
#3 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(300): PhutilErrorHandler::formatStacktrace(Array)
#4 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(417): PhutilErrorHandler::outputStacktrace(Array)
#5 /Users/matthew/PhpstormProjects/Phorge/arcanist/src/error/PhutilErrorHandler.php(282): PhutilErrorHandler::dispatchErrorMessage('exception', Object( in /Users/matthew/PhpstormProjects/Phorge/arcanist/src/init/lib/PhutilBootloader.php on line 199
This revision now requires changes to proceed.Dec 8 2022, 22:34