Skip to content

Add ability to rename grouped traces #1919

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
Aug 15, 2017
Merged

Add ability to rename grouped traces #1919

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
aef38e5
Add ability to rename grouped traces
rreusser Jul 27, 2017
d886c9e
Fix l egend naming
rreusser Jul 27, 2017
e6f5e13
Rename namepattern --> nameformat
rreusser Jul 27, 2017
7a2316d
Remove unused mock
rreusser Jul 27, 2017
4e6a0aa
Add custom prop names to keyedContainer
rreusser Jul 28, 2017
952e4ea
Fix groupby groupnames supplyDefaults
rreusser Jul 28, 2017
8f37fc7
Make groupby nameformat dependent on number of traces
rreusser Jul 28, 2017
dd035ef
Add registry methods for transforms
rreusser Aug 1, 2017
7298a0c
Cut out unnecessary redraw in legend update
rreusser Aug 1, 2017
64d2bcd
Remove unneeded inputTraceCount parameter
rreusser Aug 1, 2017
57dfe4a
Add templateString method to perform substitutions
rreusser Aug 1, 2017
c0089ea
Replace ad hoc solution with relinkPrivateKeys
rreusser Aug 1, 2017
42d76e0
Add test for editing legend
rreusser Aug 1, 2017
efcef43
Add comment for relinkPrivateKeys + transforms
rreusser Aug 2, 2017
daac59d
Enhance keyedContainer to accept nested props
rreusser Aug 8, 2017
6aefb41
Fix group -> target in groupby styles
rreusser Aug 8, 2017
be0e693
Extend keyedContainer to handle nested keys
rreusser Aug 11, 2017
4943e3a
Fix keyedContainer behavior to unset entries without removing
rreusser Aug 15, 2017
4b04131
Only blank string unsets legend items
rreusser Aug 15, 2017
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
5 changes: 1 addition & 4 deletions src/plots/plots.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -811,10 +811,7 @@ plots.supplyDataDefaults = function(dataIn, dataOut, layout, fullLayout) {
var expandedTrace = expandedTraces[j];
var fullExpandedTrace = plots.supplyTraceDefaults(expandedTrace, cnt, fullLayout, i);

// The group key gets cleared. If set, pass it forward
if(expandedTrace._group) {
fullExpandedTrace._group = expandedTrace._group;
}
relinkPrivateKeys(fullExpandedTrace, expandedTrace);
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add a comment here e.g.:

relink private (i.e. underscore) keys expanded trace to full expanded trace so that transform supply-default methods can set _ keys for future use.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

✅


// mutate uid here using parent uid and expanded index
// to promote consistency between update calls
Expand Down