Skip to content

Conversation

@svpace
Copy link

@svpace svpace commented Jul 21, 2017

CellRenderers can be used to define cell level customization to the default Table Renderer. Workaround for issue #707.

@nicolaskruchten
Copy link
Owner

Thanks for this PR, and apologies for taking so long to respond!

I'm weighing the complexity of merging this PR and maintaining the documentation etc, as compared to recommending that developers do something similar to the Heatmap and Barchart renderers, which basically walk the output of the Table Renderer and tweak the DOM.

@svpace
Copy link
Author

svpace commented Sep 22, 2017

I've struggled with this decision myself. I appreciate the separation of concerns of the layered renderers. But in the end, I was frequently writing encoded "hints" to the cells just remove and translate then to the good stuff in the next step. It felt inefficient and it was harder to reason about. Particularly when you start doing "fancy stuff" with aggregators and labels, using plain text to bridge the communication with the renderers was a pita.

I believe that the concept is easy enough to explain, its basically a hook to the last mile of the table renderer and the cell renderers are just functions to map some context to an html element. As soon as I figure out how to make contributions to the wiki I could write a couple of paragraphs explaining the feature.

@nicolaskruchten
Copy link
Owner

nicolaskruchten commented Sep 22, 2017 via email

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.

2 participants