Mercurial > public > mercurial-scm > hg
diff tests/test-hgweb-commands.t @ 35255:ad0de63e1d6a
hgweb: move common vertex code to Graph.prototype
Just to give some context to the return values: vertex() needs to return two
HTML elements as strings, <li> to be used as a background and a <li> to be
shown in foreground. The latter was made obsolete recently when changesets
started to be rendered server-side, but background elements are still useful
for now.
| author | Anton Shestakov <av6@dwimlabs.net> |
|---|---|
| date | Wed, 06 Dec 2017 12:10:16 +0800 |
| parents | 4a6c51f5ae4d |
| children | 27ab3150cd50 |
line wrap: on
line diff
--- a/tests/test-hgweb-commands.t Wed Dec 06 12:01:07 2017 +0800 +++ b/tests/test-hgweb-commands.t Wed Dec 06 12:10:16 2017 +0800 @@ -1821,22 +1821,8 @@ graph.scale(39); graph.vertex = function(x, y, radius, color, parity, cur) { - - this.ctx.beginPath(); - color = this.setColor(color, 0.25, 0.75); - this.ctx.arc(x, y, radius, 0, Math.PI * 2, true); - this.ctx.fill(); - - var bg = '<li class="bg parity' + parity + '"></li>'; - var left = (this.bg_height - this.box_size) + (this.columns + 1) * this.box_size; - - var item = document.querySelector('[data-node="' + cur.node + '"]'); - if (item) { - item.style.paddingLeft = left + 'px'; - } - - return [bg, '']; - + Graph.prototype.vertex.apply(this, arguments); + return ['<li class="bg parity' + parity + '"></li>', '']; } graph.render(data);