Target all build links inside a dropdown. Use jQuery to handle event delegation.

This commit is contained in:
Filipa Lacerda 2017-03-09 18:51:52 +00:00
parent 12a0d5a2bc
commit a7a1a60421
2 changed files with 8 additions and 4 deletions

View File

@ -287,7 +287,7 @@ import MiniPipelineGraph from './mini_pipeline_graph_dropdown';
MergeRequestWidget.prototype.initMiniPipelineGraph = function() {
new MiniPipelineGraph({
container: '.js-pipeline-inline-mr-widget-graph',
container: '.js-pipeline-inline-mr-widget-graph:visible',
}).bindEvents();
};

View File

@ -40,9 +40,13 @@ export default class MiniPipelineGraph {
* target the click event of this component.
*/
stopDropdownClickPropagation() {
document.querySelector(`${this.container} .js-builds-dropdown-list a.mini-pipeline-graph-dropdown-item`).addEventListener('click', (e) => {
e.stopPropagation();
});
$(document).on(
'click',
`${this.container} .js-builds-dropdown-list a.mini-pipeline-graph-dropdown-item`,
(e) => {
e.stopPropagation();
},
);
}
/**