4 commits

Author	SHA1	Message	Date
epriestley	969a6a2b38	Fix custom field selector control in cases where the user doesn't edit anything ... Summary: The JS and PHP representations of state can differ; just have the JS write the state out immediately on page load. Test Plan: Saved `diffusion.fields` without making changes, reloaded, saw no effective change. Reviewers: joshuaspence Reviewed By: joshuaspence Subscribers: epriestley Differential Revision: https://secure.phabricator.com/D12998	2015-05-25 05:34:23 -07:00
Joshua Spence	a0d63322b1	Remove unused JavaScript variables. ... Summary: There are a bunch of unused variables in JavaScript files. These were identified with JSHint. Test Plan: It's pretty hard to test this thoroughly... on inspection, it seems that everything //should// be okay (unless we are doing weird things with the JavaScript). Reviewers: chad, #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: epriestley, Korvin Differential Revision: https://secure.phabricator.com/D9676	2014-06-24 03:27:47 +10:00
Joshua Spence	6270114767	Various linter fixes. ... Summary: - Removed trailing newlines. - Added newline at EOF. - Removed leading newlines. - Trimmed trailing whitespace. - Spelling fix. - Added newline at EOF Test Plan: N/A Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley CC: hach-que, chad, Korvin, epriestley, aran Differential Revision: https://secure.phabricator.com/D8344	2014-02-26 12:44:58 -08:00
epriestley	e5f200c654	Allow custom fields to be reordered and disabled from Config ... Summary: Ref T1703. Put a more reasonable UI than "blob of JSON" on top of this. Test Plan: Reordered, enabled and disabled user profile fields. {F49317} Reviewers: btrahan, chad Reviewed By: chad CC: aran Maniphest Tasks: T1703 Differential Revision: https://secure.phabricator.com/D6393	2013-07-09 08:27:19 -07:00