Skip to content

Implement layout.legend.orientation (closes #53) #535

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
May 17, 2016
Merged

Implement layout.legend.orientation (closes #53) #535

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
ff16df1
Move setup of legend trace toggle into a function
n-riesco Apr 28, 2016
ddc6d71
Simplify signature of drawTexts in legend
n-riesco Apr 28, 2016
f40dbd4
Legend: add attribute `orientation`
n-riesco May 12, 2016
92ff540
Legend: add helper.isVertical(legendLayout)
n-riesco May 12, 2016
869538c
Legend: refactor computation of text dimensions
n-riesco May 12, 2016
b2538e1
Legend: set tspan.line's x in the callback
n-riesco May 13, 2016
9dc2609
Legend: set tspan.line's x before expandMargin
n-riesco May 13, 2016
f68344b
Legend: compute size only after drawing texts
n-riesco May 13, 2016
c0dfc8a
Legend: move placing of groups and traces together
n-riesco May 13, 2016
e5049c9
Legend: implement layout.legend.orientation
n-riesco May 13, 2016
23e3972
Legend: add mocks with horizontal legends
n-riesco May 13, 2016
25c2970
Legend: Use D3 to invoke functions
n-riesco May 17, 2016
7d502fa
Legend: Use Lib.setTranslate
n-riesco May 17, 2016
107883a
Legend: replace .each() by a explicit loop
n-riesco May 17, 2016
65aa6c0
Legend: Remove legendItem from function signatures
n-riesco May 17, 2016
8c6d47a
Legend: set default position of horizontal legends
n-riesco May 17, 2016
bc20390
Legend: ensure all the legends sides are visible
n-riesco May 17, 2016
c1632d4
Legend: update baseline of legend_horizontal.json
n-riesco May 17, 2016
313976d
Legend: test defaults for horizontal legends
n-riesco May 17, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 41 additions & 0 deletions test/jasmine/tests/legend_test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
var Plots = require('@src/plots/plots');
var Lib = require('@src/lib');

var Legend = require('@src/components/legend');
var getLegendData = require('@src/components/legend/get_legend_data');
Expand Down Expand Up @@ -65,6 +66,46 @@ describe('Test legend:', function() {
expect(layoutOut.legend.traceorder).toEqual('grouped+reversed');
});

it('should default orientation to vertical', function() {
supplyLayoutDefaults(layoutIn, layoutOut, []);
expect(layoutOut.legend.orientation).toEqual('v');
});

describe('for horizontal legends', function() {
var layoutInForHorizontalLegends;

beforeEach(function() {
layoutInForHorizontalLegends = Lib.extendDeep({
legend: {
orientation: 'h'
},
xaxis: {
rangeslider: {
visible: false
}
}
}, layoutIn);
});

it('should default position to bottom left', function() {
supplyLayoutDefaults(layoutInForHorizontalLegends, layoutOut, []);
expect(layoutOut.legend.x).toEqual(0);
expect(layoutOut.legend.xanchor).toEqual('left');
expect(layoutOut.legend.y).toEqual(-0.1);
expect(layoutOut.legend.yanchor).toEqual('top');
});

it('should default position to top left if a range slider present', function() {
var mockLayoutIn = Lib.extendDeep({}, layoutInForHorizontalLegends);
mockLayoutIn.xaxis.rangeslider.visible = true;

supplyLayoutDefaults(mockLayoutIn, layoutOut, []);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great tests. Thanks

expect(layoutOut.legend.x).toEqual(0);
expect(layoutOut.legend.xanchor).toEqual('left');
expect(layoutOut.legend.y).toEqual(1.1);
expect(layoutOut.legend.yanchor).toEqual('bottom');
});
});
});

describe('getLegendData', function() {
Expand Down