Conversation
| const selectedRows = R.uniq(R.pluck('row', selectedCells).sort((a, b) => a - b)); | ||
| const selectedCols: any = R.uniq(R.pluck('column', selectedCells).sort((a, b) => a - b)); | ||
|
|
||
| const transposedHeaders = createHeadings(R.pluck('name', columns), getHeaderRows(columns)); |
There was a problem hiding this comment.
When merging in with the hidden columns PR #508, will need to pass both the columns and visibleColumns -- visibleColumns for processing the data / creating the copy/paste info and columns (containing the hidden ones) to determine the number of header rows correctly
Marc-Andre-Rivet
left a comment
There was a problem hiding this comment.
Small comment about the incoming master / #508 merge.
Missing the toggle prop for this behavior as defined in the issue
This setting would be set by the Dash developer through a property like “include_headers_on_copy_and_paste=True”.
Marc-Andre-Rivet
left a comment
There was a problem hiding this comment.
💃, 👍for the extra cross-table test
Marc-Andre-Rivet
left a comment
There was a problem hiding this comment.
-- reblocking -- seeing some issues, want to make sure if these are regressions or were present before
Marc-Andre-Rivet
left a comment
There was a problem hiding this comment.
💃 unrelated problem, issue opened in #526
Closes #318