feature/20624-treemap-drill-to-last-leaf

[hubertkozik]

Added new option treemap.traverseToLeaf which allows to drill into last possible leaf of Treemap series.

---

• To see the specific tasks where the Asana app for GitHub is being used, see below:
  • https://app.asana.com/0/0/1206630380830692

[highsoft-bot]

File size comparison

No differences found

[highsoft-bot]

Visual test results - No difference found

---

Samples changed

Change type	Sample
Modified	samples/highcharts/plotoptions/treemap-traverse-to-leaf/demo.js
?	samples/highcharts/plotoptions/treemap-zoomtolastleaf/demo.js
Modified	samples/highcharts/plotoptions/treemap-zoomtolastleaf/demo.js
Added	samples/highcharts/plotoptions/treemap-zoomtolastleaf/demo.js

[MarkusBarstad]

Cool stuff, it works well! Just a couple of minor things

[bre1470]

Nice! 👍

[TorsteinHonsi]

Thanks! All good in the code, but some notes on the naming:

• In the description for the changelog, the name drillToLastLeaf is used.
• In the code, it is zoomToLastLeaf.
• In the past, we have tried to use the term traversing constantly for drilling into a tree map. This is because it is technically not the same operation as the drilldown of the Drilldown module. We wanted to avoid confusion between the two. While the Drilldown module exchanges a point for a whole new series, the traversing of a treemap just changes the root node of the tree. So this new feature, isn't it really about traversing? Zooming indicates a purely visual operation, that we just change the view without changing the tree itself. An option that is related to this new feature is allowTraversingTree.

[hubertkozik]

@TorsteinHonsi what do you think about changing the name of a new option to traverseToLastLeaf? I agree, that the word zoom can be wrong.

[TorsteinHonsi]

traverseToLastLeaf sounds good.

Or, wait a minute - isn't "last leaf" redundant? I mean a leaf is by definition the last node on a branch, right? So traverseToLeaf should be sufficient?

[TorsteinHonsi]

Except for the hands-on demo not working since the name change, everything looks good!

On a small note, Øystein and I discussed that this option does two things:

• Allows traversing to the leaf
• Changes clicking to go directly to the leaf (skipping intermediate levels)
  We agreed that in the future, if someone wants to split this functionality, we can introduce string/enum settings for the option. Something like

traverseToLeaf: boolean|'directly'|'stepwise'

[hubertkozik]

Whoops, sorry with the option name in the demo, too many changes of name, and here's the result 😁 About future, do you think we should create a Feature Request ticket with that idea or maybe simply add it to Asana?

[TorsteinHonsi]

About future, do you think we should create a Feature Request ticket with that idea or maybe simply add it to Asana?

I don't think we need to record it anywhere. Let's just wait until someone requests it, most likely they never will.