feat: Less indentation for deep trees with single child

[BYK]

Before:

After:

Before:

After:

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
spotlightjs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 3, 2025 8:59pm

[codecov]

Codecov Report

Attention: Patch coverage is 12.50000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 31.76%. Comparing base (63f2403) to head (9c0b936).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...entry/components/explore/traces/spans/SpanItem.tsx	0.00%	6 Missing ⚠️
...entry/components/explore/traces/spans/SpanTree.tsx	50.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #686   +/-   ##
=======================================
  Coverage   31.76%   31.76%           
=======================================
  Files          92       92           
  Lines        5519     5521    +2     
  Branches      100      100           
=======================================
+ Hits         1753     1754    +1     
- Misses       3766     3767    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dcramer]

I dont LOVE the solution, but I'm also struggling to come up w/ a better idea. We still have this issue on production Sentry too and its kind of a disaster...

[BYK]

TBH this is quite a hacky solution and I certainly don't like it much. However until we can do better design and span grouping, this solution is the best bang for the buck.

With the upcoming profiling work, it is a necessity as profiles can go deep fast with 1-child trees.

[dcramer]

I'm good w/ this - @vuluongj20 mentioned we tried an approach with a dashed line that would simply collapse a lot of these single child spans. I htink something akin to that is the right direction, but the dashed implementation IMO is still not obvious enough.

We'll follow up with something better in future.

[vuluongj20]

Here's a screenshot for reference

Agree that it's not super obvious what the dashed line means. We can consider other visual cues.

[BYK]

Created #686 for a proper follow up.