Skip to content

Commit 70687cd

Browse files
hiroponzGitLab
authored and
GitLab
committed
Improve network graph
1 parent 65737bd commit 70687cd

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

lib/gitlab/graph/json_builder.rb

+6-4
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ def initialize project
1616

1717
@commits = collect_commits
1818
@days = index_commits
19+
@space = 0
1920
end
2021

2122
def to_json(*args)
@@ -97,8 +98,8 @@ def place_chain(commit, map, parent_time = nil)
9798
if leaves.empty?
9899
return
99100
end
100-
space = find_free_space(leaves, map)
101-
leaves.each{|l| l.space = space}
101+
@space = find_free_space(leaves, map)
102+
leaves.each{|l| l.space = @space}
102103
# and mark it as reserved
103104
min_time = leaves.last.time
104105
parents = leaves.last.parents.collect
@@ -115,7 +116,7 @@ def place_chain(commit, map, parent_time = nil)
115116
else
116117
max_time = parent_time - 1
117118
end
118-
mark_reserved(min_time..max_time, space)
119+
mark_reserved(min_time..max_time, @space)
119120

120121
# Visit branching chains
121122
leaves.each do |l|
@@ -139,9 +140,10 @@ def find_free_space(leaves, map)
139140
reserved += @_reserved[day]
140141
end
141142
space = base_space(leaves, map)
142-
while reserved.include? space do
143+
while (reserved.include? space) || (space == @space) do
143144
space += 1
144145
end
146+
145147
space
146148
end
147149

vendor/assets/javascripts/branch-graph.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@
121121
if (c.space == this.commits[i].space) {
122122
r.path([
123123
"M", x, y,
124-
"L", x - 20 * (c.time + 1), y
124+
"L", cx, cy
125125
]).attr({
126126
stroke: this.colors[c.space],
127127
"stroke-width": 2
@@ -351,4 +351,4 @@ function textWrap(t, width) {
351351
t.attr({
352352
"y": b.y + h
353353
});
354-
}
354+
}

0 commit comments

Comments
 (0)