Skip to content
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
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
1 change: 1 addition & 0 deletions draftlogs/7057_fix.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
- Fix missing cursor for Ternary Plot[[#7057](https://github.com/plotly/plotly.js/pull/7057)]
14 changes: 14 additions & 0 deletions src/plots/ternary/index.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
'use strict';

var d3 = require('@plotly/d3');

var Ternary = require('./ternary');

var getSubplotCalcData = require('../../plots/get_data').getSubplotCalcData;
Expand Down Expand Up @@ -75,3 +77,15 @@ exports.clean = function(newFullData, newFullLayout, oldFullData, oldFullLayout)
}
}
};

exports.updateFx = function(gd) {
var fullLayout = gd._fullLayout;
var dragmode = fullLayout.dragmode;
var toplevel = d3.select(gd).selectAll('g.toplevel');

if(dragmode === 'pan') {
toplevel.style('cursor', 'move');
} else {
toplevel.style('cursor', 'crosshair');
}
};
11 changes: 11 additions & 0 deletions src/plots/ternary/ternary.js
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ proto.plot = function(ternaryCalcData, fullLayout) {

_this.updateLayers(ternaryLayout);
_this.adjustLayout(ternaryLayout, graphSize);
_this.updateFx(fullLayout.dragmode);
Plots.generalUpdatePerTraceModule(_this.graphDiv, _this, ternaryCalcData, ternaryLayout);
_this.layers.plotbg.select('path').call(Color.fill, ternaryLayout.bgcolor);
};
Expand Down Expand Up @@ -97,6 +98,16 @@ proto.makeFramework = function(fullLayout) {
Drawing.setClipUrl(_this.layers.grids, clipId, gd);
};

proto.updateFx = function(dragmode) {
var toplevel = this.plotContainer.selectAll('g.toplevel');

if(dragmode === 'pan') {
toplevel.style('cursor', 'move');
} else {
toplevel.style('cursor', 'crosshair');
}
};

proto.updateLayers = function(ternaryLayout) {
var _this = this;
var layers = _this.layers;
Expand Down