JS review changes - Removed window from window.gl in dispatcher

Added page:load event as ready isnt fired by turbolinks

Fix dropdown menu link click registration
This commit is contained in:
Luke Bennett 2016-10-06 18:00:06 +01:00 committed by Annabel Dunstone Gray
parent 113050c570
commit 6a7b673035
5 changed files with 8 additions and 3 deletions

View file

@ -127,7 +127,7 @@
}
break;
case 'projects:pipelines:show':
new window.gl.Pipelines();
new gl.Pipelines();
break;
case 'groups:activity':
new Activities();

View file

@ -4,6 +4,7 @@
constructor() {
$(document).off('click', '.toggle-pipeline-btn').on('click', '.toggle-pipeline-btn', this.toggleGraph);
$(document).off('ready.addMarginToBuildColumns').on('ready.addMarginToBuildColumns', this.addMarginToBuildColumns);
$(document).off('page:load.addMarginToBuildColumns').on('page:load.addMarginToBuildColumns', this.addMarginToBuildColumns);
}
toggleGraph() {

View file

@ -373,6 +373,9 @@
}
.build-content {
display: -ms-flexbox;
display: -webkit-flex;
display: flex;
width: 164px;
.ci-status-icon {
@ -403,6 +406,7 @@
width: auto;
padding: 0;
color: $gl-text-color-light;
flex-grow: 1;
.ci-status-text {
max-width: 112px;

View file

@ -10,5 +10,5 @@
- else
%li.build
.curve
.dropdown.inline.build-content{ type: 'button', data: { toggle: 'dropdown' } }
.dropdown.inline.build-content
= render "projects/commit/pipeline_status_group", name: group_name, subject: grouped_statuses

View file

@ -1,7 +1,7 @@
- group_status = CommitStatus.where(id: subject).status
%span.ci-status-icon
= render_status_with_link('build', group_status)
%button.dropdown-menu-toggle
%button.dropdown-menu-toggle{ type: 'button', data: { toggle: 'dropdown' } }
%span.ci-status-text
= name
%span.badge= subject.size