Treemap update

[alexpghayes]

This pull request is to address the following issues in the Highcharter function hctreemap:

• hctreemap() fails to render with cellnames repeated across levels hctreemap() fails to render with cellnames repeated across levels #310
• bug in treemap when index variables have same values bug in treemap when index variables have same values #32
• Does not work correctly for 3 level Treemaps Does not work correctly for 3 level Treemaps #117
• hc_add_series_treemap - naming of the color value variable hc_add_series_treemap - naming of the color value variable #98
• review/rework treemap fns review/rework treemap fns #110

I've created a markdown walkthrough dev/treemap2.Rmd documenting a new function hctreemap2 that I believe fixes these. Sorry this has taken me so long.

[jbkunst]

Hi @alexpghayes

I'm really happy to see this :)! I will take 1 or 2 days to see the changes, and see how the functions works (I'm new to rlang) and see other details.

Thank you very much! With this it's probably make a new release soon!

Best,

[jbkunst]

And don't say sorry 😅 !

[alexpghayes]

The first half of this blog post covers all of the new dplyr NSE I used. I'm happy to comment the code or rewrite any sections for clarity. Probably the most useful thing to do to understand the function is to print out treemap_df after it's generated.

[jbkunst]

Hi @alexpghayes !!

I will take 1 or 2 days to see the changes,

The longest days on the entire history! So sorry! Thanks to you I will start the new release! I'll add you to the contributors in the DESCRIPTION if you don't mind

[jbkunst]

If you create and id field, why is a problem have same labels across the grouping vars?

[alexpghayes]

If the top level of the treemap has categories in {A, B} and second level has categories in {B, C}, for cells on a third level of the treemap a parent id of B1_B2 is ambiguous.