Skip to content

Conversation

@dmt0
Copy link
Contributor

@dmt0 dmt0 commented Jun 27, 2018

Closes #499

@dmt0 dmt0 requested a review from nicolaskruchten June 27, 2018 17:19
@dmt0 dmt0 self-assigned this Jun 27, 2018
graphDiv.data[traceIndexes[0]].cells.valuessrc || null;
} else if (
update['cells.values'] === null &&
!graphDiv.data[traceIndexes[0]].header.valuessrc
Copy link
Contributor

Choose a reason for hiding this comment

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

@VeraZab do we need to use your special from/to functions here to extract an array of data source names?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be ok without any adjustments, as its the data selector itself that handles the from and to src conversions, everywhere else, the editor just uses its own format, its just in that component that we care about it, because that's where the fullValue is matched up to the dataSourceOptions

Copy link
Contributor

Choose a reason for hiding this comment

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

ok, great, thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

it's actually going to show very weird names in strembed, like ~:100:1000, this does nothing to prettify names, its just 2 functions that allow for custom src formatting between the editor and another app.

And those 2 functions are only used in the DataSelector, because that's the only component that has to interface between dataSourceOptions, which can be in a very custom format, and the react-chart-editor's way of handling srcs: array based for multi columns, and string based otherwise.

But these functions won't make the names of srcs prettier..

@dmt0 dmt0 requested a review from VeraZab June 27, 2018 18:14
@nicolaskruchten
Copy link
Contributor

💃

1 similar comment
@VeraZab
Copy link
Contributor

VeraZab commented Jun 27, 2018

💃

@dmt0 dmt0 merged commit d1abfe0 into master Jun 27, 2018
@dmt0 dmt0 deleted the table-column-defaults branch June 27, 2018 18:46
@nicolaskruchten nicolaskruchten restored the table-column-defaults branch July 18, 2018 17:16
@nicolaskruchten nicolaskruchten deleted the table-column-defaults branch July 18, 2018 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants