Thanks to visit codestin.com
Credit goes to github.com

Skip to content

A slightly less ambitious take on orphan subplots and 'delete last trace' #289

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 18 commits into from
Feb 29, 2016
Merged

A slightly less ambitious take on orphan subplots and 'delete last trace' #289

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
c4973f5
make Plotly.deleteTraces on 'surface' and 'mesh' traces work
etpinard Feb 19, 2016
48c6dfb
rm unnecessary hasCartesian check
etpinard Feb 19, 2016
1d4f7e6
make removing geo trace work:
etpinard Feb 19, 2016
fd5cab7
add geo and cartesian delete traces tests
etpinard Feb 19, 2016
d9ed496
generalize cleanScenes step:
etpinard Feb 19, 2016
26218f6
lint
etpinard Feb 23, 2016
e3cb6eb
write promise sequence as promise chains
etpinard Feb 22, 2016
1d2d9c3
assure that re-plots changing subplots pass through make framework
etpinard Feb 23, 2016
3a68023
assure that clearing cartesian axes in relayout lead to a re-calc
etpinard Feb 23, 2016
73817cb
add tests checking axis ranges after partial deletion
etpinard Feb 23, 2016
c75d8a2
add cartesian clean plot step to remove old pie traces
etpinard Feb 23, 2016
0d38cf7
rename Plots getSubplotIdsInData -> findSubplotIds
etpinard Feb 26, 2016
9da84df
standardize gl3d and geo layout default logic
etpinard Feb 26, 2016
07ca162
ensure that Plots getSubplotIds return ordered arrays of ids
etpinard Feb 26, 2016
ada31b5
robustify 'find axis' logic:
etpinard Feb 26, 2016
a71efa7
add jsdoc
etpinard Feb 26, 2016
de38b67
update geo delete trace test:
etpinard Feb 26, 2016
07efdc7
add trace type restyle and delete trace gl3d tests
etpinard Feb 26, 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
Prev Previous commit
Next Next commit
add cartesian clean plot step to remove old pie traces
  • Loading branch information
@etpinard
etpinard committed Feb 26, 2016
commit c75d8a22ef8f91c5c645846ca47886fd820342af
6 changes: 6 additions & 0 deletions src/plots/cartesian/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,3 +112,9 @@ exports.plot = function(gd) {
if(cdPie.length) Pie.plot(gd, cdPie);
}
};

exports.clean = function(newFullData, newFullLayout, oldFullData, oldFullLayout) {
if(oldFullLayout._hasPie && !newFullLayout._hasPie) {
oldFullLayout._pielayer.selectAll('g.trace').remove();
}
};
49 changes: 48 additions & 1 deletion test/jasmine/tests/plot_interact_test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -292,9 +292,23 @@ describe('Test plot structure', function() {

describe('pie traces', function() {
var mock = require('@mocks/pie_simple.json');
var gd;

function countPieTraces() {
return d3.select('g.pielayer').selectAll('g.trace').size();
}

function countBarTraces() {
return d3.selectAll('g.trace.bars').size();
}

beforeEach(function(done) {
Plotly.plot(createGraphDiv(), mock.data, mock.layout).then(done);
gd = createGraphDiv();

var mockData = Lib.extendDeep([], mock.data),
mockLayout = Lib.extendDeep({}, mock.layout);

Plotly.plot(gd, mockData, mockLayout).then(done);
});

it('has as many *slice* nodes as there are pie items', function() {
Expand All @@ -319,6 +333,39 @@ describe('Test plot structure', function() {
var testerSVG = d3.selectAll('#js-plotly-tester');
assertNamespaces(testerSVG.node());
});

it('should be able to get deleted', function(done) {
expect(countPieTraces()).toEqual(1);
expect(countSubplots()).toEqual(0);

Plotly.deleteTraces(gd, [0]).then(function() {
expect(countPieTraces()).toEqual(0);
expect(countSubplots()).toEqual(0);

done();
});
});

it('should be able to be restyled to a bar chart and back', function(done) {
expect(countPieTraces()).toEqual(1);
expect(countBarTraces()).toEqual(0);
expect(countSubplots()).toEqual(0);

Plotly.restyle(gd, 'type', 'bar').then(function() {
expect(countPieTraces()).toEqual(0);
expect(countBarTraces()).toEqual(1);
expect(countSubplots()).toEqual(1);

return Plotly.restyle(gd, 'type', 'pie');
}).then(function() {
expect(countPieTraces()).toEqual(1);
expect(countBarTraces()).toEqual(0);
expect(countSubplots()).toEqual(0);

done();
});

});
});
});

Expand Down