mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-26 23:40:57 +01:00
2a39fd09eb
Summary: Javelin is currently embedded in Phabricator via copy-and-paste of prebuilt packages. This is not so great. Pull it in as a submodule instead and make all the Phabriator resources declare proper dependency trees. Add Javelin linting. Test Plan: I tried to run through pretty much all the JS functionality on the site. This is still a high-risk change, but I did a pretty thorough test Differential: inline comments, revealing diffs, list tokenizers, comment preview, editing/deleting comments, add review action. Maniphest: list tokenizer, comment actions Herald: rule editing, tokenizers, add/remove rows Reviewed By: tomo Reviewers: aran, tomo, mroch, jungejason, tuomaspelkonen CC: aran, tomo, epriestley Differential Revision: 223
35 lines
863 B
JavaScript
35 lines
863 B
JavaScript
/**
|
|
* @provides javelin-behavior-differential-add-reviewers
|
|
* @requires javelin-behavior
|
|
* javelin-dom
|
|
* javelin-tokenizer
|
|
* javelin-typeahead
|
|
* javelin-typeahead-preloaded-source
|
|
*/
|
|
|
|
JX.behavior('differential-add-reviewers', function(config) {
|
|
|
|
var root = JX.$(config.tokenizer);
|
|
var datasource = new JX.TypeaheadPreloadedSource(config.src);
|
|
|
|
var typeahead = new JX.Typeahead(root);
|
|
typeahead.setDatasource(datasource);
|
|
|
|
var tokenizer = new JX.Tokenizer(root);
|
|
tokenizer.setTypeahead(typeahead);
|
|
tokenizer.start();
|
|
|
|
JX.DOM.listen(
|
|
JX.$(config.select),
|
|
'change',
|
|
null,
|
|
function(e) {
|
|
if (JX.$(config.select).value == 'add_reviewers') {
|
|
JX.DOM.show(JX.$(config.row));
|
|
tokenizer.refresh();
|
|
} else {
|
|
JX.DOM.hide(JX.$(config.row));
|
|
}
|
|
});
|
|
});
|
|
|