1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-25 06:50:55 +01:00
phorge-phorge/webroot/rsrc/js/application/differential
Nick Harper 6039ca6fb5 Create option for differential custom fields to display warning on accept
Summary:
This adds support to differential fields to display warnings before a revision
gets accepted. Since lint and unit are differential fields, the code for their
warnings was moved into their respective field specification classes, so there
is only one code path for warnings (lint, unit, or custom).

Test Plan:
Select 'Accept' on a revision with lint/unit warnings and see messages appear
like they used to. Change it back to 'Comment' and they go away. Repeat with
a revision without lint/unit warnings and see no warnings appear. Checked
darkconsole to see no errors due to this.

Reviewers: jungejason, epriestley, vrana

Reviewed By: epriestley

CC: aran, Koolvin

Differential Revision: https://secure.phabricator.com/D2363
2012-05-02 14:30:21 -07:00
..
behavior-accept-with-errors.js Create option for differential custom fields to display warning on accept 2012-05-02 14:30:21 -07:00
behavior-add-reviewers-and-ccs.js Use Javelin placeholders and new sorting rules broadly; consolidate tokenizer construction code 2012-03-09 15:46:39 -08:00
behavior-comment-jump.js Added 'Next' and 'Previous' links to differential 2011-05-11 14:46:00 -07:00
behavior-comment-preview.js Improve inline comment previews 2012-03-28 10:11:41 -07:00
behavior-diff-radios.js Bring Javelin into Phabricator via git submodule, not copy-and-paste 2011-05-08 13:20:10 -07:00
behavior-dropdown-menus.js Add "View Options" dropdown to Differential / Audit 2012-03-19 19:57:41 -07:00
behavior-edit-inline-comments.js Fix reticle with edit inline comment 2012-04-12 11:42:50 -07:00
behavior-keyboard-nav.js Add "jump to table of content" keyboard shortcut 2012-04-28 18:05:08 -07:00
behavior-populate.js Show a tooltip about code coverage 2012-03-12 20:04:12 -07:00
behavior-show-all-comments.js Go to correct position after revealing comments 2012-02-26 13:13:51 -08:00
behavior-show-field-details.js Further improve unit/lint rendering 2012-05-01 10:15:56 -07:00
behavior-show-more.js Show a tooltip about code coverage 2012-03-12 20:04:12 -07:00
behavior-user-select.js Hide selection to provide a visual cue about clipboard JS magic in Differential 2012-04-24 18:53:48 -07:00
DifferentialInlineCommentEditor.js Don't provide Undo for empty text in Differential inline comment 2012-02-29 23:44:42 -08:00