Skip to content

More layout meta support #3548

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.

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 4 commits into from
Feb 15, 2019
Merged
Show file tree
Hide file tree
Changes from all 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
12 changes: 11 additions & 1 deletion src/components/fx/hover.js
Original file line number Diff line number Diff line change
Expand Up @@ -738,6 +738,7 @@ function _hover(gd, evt, subplot, noHoverEvent) {
var EXTRA_STRING_REGEX = /<extra>([\s\S]*)<\/extra>/;

function createHoverText(hoverData, opts, gd) {
var fullLayout = gd._fullLayout;
var hovermode = opts.hovermode;
var rotateLabels = opts.rotateLabels;
var bgColor = opts.bgColor;
Expand Down Expand Up @@ -925,6 +926,10 @@ function createHoverText(hoverData, opts, gd) {
if(d.nameOverride !== undefined) d.name = d.nameOverride;

if(d.name) {
if(fullLayout.meta) {
d.name = Lib.templateString(d.name, {meta: fullLayout.meta});
}

name = svgTextUtils.plainText(d.name || '', {
len: d.nameLength,
allowedTags: ['br', 'sub', 'sup', 'b', 'i', 'em']
Expand Down Expand Up @@ -970,7 +975,12 @@ function createHoverText(hoverData, opts, gd) {
var hovertemplateLabels = d.hovertemplateLabels || d;
var eventData = d.eventData[0] || {};
if(hovertemplate) {
text = Lib.hovertemplateString(hovertemplate, hovertemplateLabels, eventData);
text = Lib.hovertemplateString(
hovertemplate,
hovertemplateLabels,
eventData,
{meta: fullLayout.meta}
);

text = text.replace(EXTRA_STRING_REGEX, function(match, extra) {
name = extra; // Assign name for secondary text label
Expand Down
6 changes: 5 additions & 1 deletion src/components/legend/draw.js
Original file line number Diff line number Diff line change
Expand Up @@ -404,9 +404,13 @@ function drawTexts(g, gd, maxLength) {
var trace = legendItem.trace;
var isPie = Registry.traceIs(trace, 'pie');
var traceIndex = trace.index;
var name = isPie ? legendItem.label : trace.name;
var isEditable = gd._context.edits.legendText && !isPie;

var name = isPie ? legendItem.label : trace.name;
if(fullLayout.meta) {
name = Lib.templateString(name, {meta: fullLayout.meta});
}

var textEl = Lib.ensureSingle(g, 'text', 'legendtext');

textEl.attr('text-anchor', 'start')
Expand Down
11 changes: 7 additions & 4 deletions src/components/rangeselector/draw.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ var FROM_BR = alignmentConstants.FROM_BR;
var constants = require('./constants');
var getUpdateObject = require('./get_update_object');


module.exports = function draw(gd) {
var fullLayout = gd._fullLayout;

Expand Down Expand Up @@ -152,12 +151,16 @@ function drawButtonText(button, selectorLayout, d, gd) {
});

text.call(Drawing.font, selectorLayout.font)
.text(getLabel(d))
.text(getLabel(d, gd._fullLayout.meta))
.call(textLayout);
}

function getLabel(opts) {
if(opts.label) return opts.label;
function getLabel(opts, meta) {
if(opts.label) {
return meta ?
Lib.templateString(opts.label, {meta: meta}) :
opts.label;
}

if(opts.step === 'all') return 'all';

Expand Down
12 changes: 11 additions & 1 deletion src/components/sliders/draw.js
Original file line number Diff line number Diff line change
Expand Up @@ -317,6 +317,10 @@ function drawCurrentValue(sliderGroup, sliderOpts, valueOverride) {
str += valueOverride;
} else {
var curVal = sliderOpts.steps[sliderOpts.active].label;
var meta = sliderOpts._gd._fullLayout.meta;
if(meta) {
curVal = Lib.templateString(curVal, {meta: meta});
}
str += curVal;
}

Expand Down Expand Up @@ -364,8 +368,14 @@ function drawLabel(item, data, sliderOpts) {
});
});

var tx = data.step.label;
var meta = sliderOpts._gd._fullLayout.meta;
if(meta) {
tx = Lib.templateString(tx, {meta: meta});
}

text.call(Drawing.font, sliderOpts.font)
.text(data.step.label)
.text(tx)
.call(svgTextUtils.convertToTspans, sliderOpts._gd);

return text;
Expand Down
8 changes: 7 additions & 1 deletion src/components/updatemenus/draw.js
Original file line number Diff line number Diff line change
Expand Up @@ -442,8 +442,14 @@ function drawItemText(item, menuOpts, itemOpts, gd) {
});
});

var tx = itemOpts.label;
var meta = gd._fullLayout.meta;
if(meta) {
tx = Lib.templateString(tx, {meta: meta});
}

text.call(Drawing.font, menuOpts.font)
.text(itemOpts.label)
.text(tx)
.call(svgTextUtils.convertToTspans, gd);
}

Expand Down
6 changes: 3 additions & 3 deletions src/lib/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -994,7 +994,7 @@ lib.numSeparate = function(value, separators, separatethousands) {
return x1 + x2;
};

var TEMPLATE_STRING_REGEX = /%{([^\s%{}:]*)(:[^}]*)?}/g;
lib.TEMPLATE_STRING_REGEX = /%{([^\s%{}:]*)(:[^}]*)?}/g;
var SIMPLE_PROPERTY_REGEX = /^\w*$/;

/**
Expand All @@ -1014,7 +1014,7 @@ lib.templateString = function(string, obj) {
// just in case it speeds things up *slightly*:
var getterCache = {};

return string.replace(TEMPLATE_STRING_REGEX, function(dummy, key) {
return string.replace(lib.TEMPLATE_STRING_REGEX, function(dummy, key) {
if(SIMPLE_PROPERTY_REGEX.test(key)) {
return obj[key] || '';
}
Expand Down Expand Up @@ -1047,7 +1047,7 @@ lib.hovertemplateString = function(string, labels) {
// just in case it speeds things up *slightly*:
var getterCache = {};

return string.replace(TEMPLATE_STRING_REGEX, function(match, key, format) {
return string.replace(lib.TEMPLATE_STRING_REGEX, function(match, key, format) {
var obj, value, i;
for(i = 2; i < args.length; i++) {
obj = args[i];
Expand Down
5 changes: 3 additions & 2 deletions src/plots/layout_attributes.js
Original file line number Diff line number Diff line change
Expand Up @@ -416,8 +416,9 @@ module.exports = {
editType: 'plot',
description: [
'Assigns extra meta information that can be used in various `text` attributes.',
'Attributes such as the graph, axis and colorbar `title.text` and annotation `text`',
'support `meta`. One can access `meta` fields using template strings:',
'Attributes such as the graph, axis and colorbar `title.text`, annotation `text`',
'`trace.name` in legend items, `rangeselector`, `updatemenues` and `sliders` `label` text',
'all support `meta`. One can access `meta` fields using template strings:',
'`%{meta[i]}` where `i` is the index of the `meta`',
'item in question.'
].join(' ')
Expand Down
Binary file modified test/image/baselines/layout_metatext.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
40 changes: 36 additions & 4 deletions test/image/mocks/layout_metatext.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
{
"data": [{
"y": [1, 2, 1]
"x": ["2019-01-01", "2019-02-10", "2019-03-24"],
"y": [1, 2, 1],
"name": "TRACE %{meta[3]}"
}, {
"type": "scatterpolar",
"r": [1, 2, 1]
Expand Down Expand Up @@ -29,12 +31,20 @@
"width": 700,
"height": 800,
"margin": {"b": 40},
"showlegend": false,

"title": {"text": "This graph is %{meta[2]}"},
"xaxis": {
"domain": {"row": 0, "column": 0},
"title": {"text": "Worth more than %{meta[4]} %{meta[1]}"}
"title": {"text": "Worth more than %{meta[4]} %{meta[1]}"},
"rangeselector": {
"buttons": [{
"step": "all"
}, {
"step": "month",
"count": 2,
"label": "backup %{meta[2]}"
}]
}
},
"yaxis": {
"domain": {"row": 0, "column": 0},
Expand Down Expand Up @@ -78,8 +88,30 @@
"text": "<i>N.B.</i> %{meta[2]}",
"xref": "x",
"yref": "y",
"x": 1,
"x": "2019-02-10",
"y": 2
}],

"updatemenus": [{
"buttons": [{
"label": "Btn-%{meta[0]}",
"method": "restyle",
"args": ["visible", false]
}],
"y": 1,
"yanchor": "bottom"
}],

"sliders": [{
"steps": [{
"label": "step-%{meta[0]}",
"method": "restyle",
"args": ["marker.color", "red"]
}, {
"label": "step-%{meta[1]}",
"method": "restyle",
"args": ["marker.color", "blue"]
}]
}]
}
}
45 changes: 45 additions & 0 deletions test/jasmine/tests/hover_label_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -1733,6 +1733,51 @@ describe('hover info', function() {
.catch(failTest)
.then(done);
});

it('should work with layout.meta references', function(done) {
var gd = document.getElementById('graph');

Plotly.update(gd,
{hovertemplate: 'TRACE -- %{meta[0]}<extra>%{meta[1]}</extra>'},
{meta: ['A', '$$$']}
).then(function() {
Fx.hover('graph', evt, 'xy');

assertHoverLabelContent({
nums: 'TRACE -- A',
name: '$$$',
axis: '0.388'
});
})
.catch(failTest)
.then(done);
});
});

it('should work with trace.name linked to layout.meta', function(done) {
var gd = createGraphDiv();

Plotly.plot(gd, [{
y: [1, 1, 1],
name: '%{meta[0]}',
marker: {size: 40}
}, {
y: [1]
}], {
meta: ['yo!'],
width: 400,
height: 400
})
.then(function() { _hoverNatural(gd, 200, 200); })
.then(function() {
assertHoverLabelContent({
nums: '1',
name: 'yo!',
axis: '2'
});
})
.catch(failTest)
.then(done);
});
});

Expand Down