Sitemap treeview: Fix arrows/chevrons shown if not expandable

[florian-h05]

Regression from #2904.
Reported in #2970 (comment).

[relativeci]

#2695 Bundle Size — 10.98MiB (~+0.01%).

a8d2be5(current) vs e7d9b65 main#2689(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics   1 change

	Current #2695	Baseline #2689
Initial JS	1.9MiB	1.9MiB
Initial CSS	577.21KiB	577.21KiB
Cache Invalidation	17.51%	19.12%
Chunks	227	227
Assets	250	250
Modules	2950	2950
Duplicate Modules	154	154
Duplicate Code	1.8%	1.8%
Packages	98	98
Duplicate Packages	2	2

Bundle size by type   1 change 1 regression

	Current #2695	Baseline #2689
JS	9.19MiB (~+0.01%)	9.19MiB
CSS	867.02KiB	867.02KiB
Fonts	526.1KiB	526.1KiB
Media	295.6KiB	295.6KiB
IMG	140.74KiB	140.74KiB
HTML	1.38KiB	1.38KiB
Other	871B	871B

Bundle analysis report Branch florian-h05:sitemap-treeview-che... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[mherwege]

Thanks. I didn't test yet, but LGTM

[ghys]

Too late for 4.3.2?

[florian-h05]

#2904 is no part of 4.3.x at all, no need to backport this at all.

[ghys]

Indeed, sorry.

[mherwege]

@florian-h05 Did you test this? I tried replicating this logic for the model tree-view, but the expansion icons still show. I had to explicitely set the toggle parameter on the f7-treeview-item.

[florian-h05]

Yes, I tested this and it worked fine.

[mherwege]

OK, I now understand why this works here, but not in the model treeview. The difference is that I made sure there is an empty slot in the children in this case.
I tested this, and an alternative approach with a toggle that also works, which I have to use in the model treeview. My review proposal used in here uses this approach. It brings both in line, but yours will work as well. It is up to you what to choose.

[mherwege]

Remove v-if="canHaveChildren" from line 10.
Add :toggle="canHaveChildren" around line 6.

[florian-h05]

Let's align the implementation - I have just implemented your review comments.